[U-Boot] [PATCH 5/8] tegra: usb: move controller init into start_port
Lucas Stach
dev at lynxeye.de
Tue Oct 30 13:44:12 CET 2012
Hi Marek,
Am Dienstag, den 30.10.2012, 13:33 +0100 schrieb Marek Vasut:
> Dear Lucas Stach,
>
> [...]
>
> > > > -static int add_port(struct fdt_usb *config)
> > >
> > > Fix the comment instead of removing it?
> >
> > I don't think that this comment adds any real value. The whole function
> > which this comment refers to is removed and it's content split between
> > board_usb_init and ehci_hcd_init, which are self explanatory.
>
> Then add a proper comment please. Call me a docu-nazi, but I'd really love u-
> boot nicely and properly documented, please.
>
I'm all in favour of adding proper documentation, but I'm opposed to add
it in the middle of this cleanup/movement series.
I'll send a patch on top of this series to add doc, so it doesn't
interfere with the review of this series.
Regards,
Lucas
More information about the U-Boot
mailing list