Message ID | 1354106642-4587-4-git-send-email-panto@antoniou-consulting.com |
---|---|
State | Superseded |
Delegated to: | Marek Vasut |
Headers | show |
Dear Pantelis Antoniou, > USB initialization shouldn't happen for all the boards. > > Signed-off-by: Pantelis Antoniou <panto@antoniou-consulting.com> > --- > common/cmd_dfu.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/common/cmd_dfu.c b/common/cmd_dfu.c > index 01d6b3a..327c738 100644 > --- a/common/cmd_dfu.c > +++ b/common/cmd_dfu.c > @@ -55,7 +55,10 @@ static int do_dfu(cmd_tbl_t *cmdtp, int flag, int argc, > char * const argv[]) goto done; > } > > +#ifdef CONFIG_TRATS > board_usb_init(); > +#endif > + It's common code: 1) Why is it called "board_usb_init()" ? Does this have anything to do with usb host? 2) Make it __weak, then if it's undefined for your board, something default will be called. > g_dnl_register(s); > while (1) { > if (ctrlc()) Best regards, Marek Vasut
Hi Marek, On Nov 28, 2012, at 4:45 AM, Marek Vasut wrote: > Dear Pantelis Antoniou, > >> USB initialization shouldn't happen for all the boards. >> >> Signed-off-by: Pantelis Antoniou <panto@antoniou-consulting.com> >> --- >> common/cmd_dfu.c | 3 +++ >> 1 file changed, 3 insertions(+) >> >> diff --git a/common/cmd_dfu.c b/common/cmd_dfu.c >> index 01d6b3a..327c738 100644 >> --- a/common/cmd_dfu.c >> +++ b/common/cmd_dfu.c >> @@ -55,7 +55,10 @@ static int do_dfu(cmd_tbl_t *cmdtp, int flag, int argc, >> char * const argv[]) goto done; >> } >> >> +#ifdef CONFIG_TRATS >> board_usb_init(); >> +#endif >> + > > It's common code: > > 1) Why is it called "board_usb_init()" ? Does this have anything to do with usb > host? > No idea. It makes no sense to me, but it was there from the original DFU poster. I don't have a TRATS board to test it anyway, but I didn't want to affect it. > 2) Make it __weak, then if it's undefined for your board, something default will > be called. > I see no reason why it should even exist. Perhaps we should ask the original poster. >> g_dnl_register(s); >> while (1) { >> if (ctrlc()) > > Best regards, > Marek Vasut Regards -- Pantelis
Dear Pantelis Antoniou, > Hi Marek, > > On Nov 28, 2012, at 4:45 AM, Marek Vasut wrote: > > Dear Pantelis Antoniou, > > > >> USB initialization shouldn't happen for all the boards. > >> > >> Signed-off-by: Pantelis Antoniou <panto@antoniou-consulting.com> > >> --- > >> common/cmd_dfu.c | 3 +++ > >> 1 file changed, 3 insertions(+) > >> > >> diff --git a/common/cmd_dfu.c b/common/cmd_dfu.c > >> index 01d6b3a..327c738 100644 > >> --- a/common/cmd_dfu.c > >> +++ b/common/cmd_dfu.c > >> @@ -55,7 +55,10 @@ static int do_dfu(cmd_tbl_t *cmdtp, int flag, int > >> argc, char * const argv[]) goto done; > >> > >> } > >> > >> +#ifdef CONFIG_TRATS > >> > >> board_usb_init(); > >> > >> +#endif > >> + > > > > It's common code: > > > > 1) Why is it called "board_usb_init()" ? Does this have anything to do > > with usb host? > > No idea. It makes no sense to me, but it was there from the original DFU > poster. I don't have a TRATS board to test it anyway, but I didn't want to > affect it. Can you please rename it to a more sensible name then? > > 2) Make it __weak, then if it's undefined for your board, something > > default will be called. > > I see no reason why it should even exist. Perhaps we should ask the > original poster. Please do it then > >> g_dnl_register(s); > >> while (1) { > >> > >> if (ctrlc()) > > > > Best regards, > > Marek Vasut > > Regards > > -- Pantelis Best regards, Marek Vasut
diff --git a/common/cmd_dfu.c b/common/cmd_dfu.c index 01d6b3a..327c738 100644 --- a/common/cmd_dfu.c +++ b/common/cmd_dfu.c @@ -55,7 +55,10 @@ static int do_dfu(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) goto done; } +#ifdef CONFIG_TRATS board_usb_init(); +#endif + g_dnl_register(s); while (1) { if (ctrlc())
USB initialization shouldn't happen for all the boards. Signed-off-by: Pantelis Antoniou <panto@antoniou-consulting.com> --- common/cmd_dfu.c | 3 +++ 1 file changed, 3 insertions(+)