[U-Boot] [PATCH v2 2/6] am33xx: board: Refactor USB initialization into separate function

Tom Rini trini at konsulko.com
Tue Feb 7 18:23:48 UTC 2017


On Mon, Feb 06, 2017 at 07:17:33PM -0800, Alexandru Gagniuc wrote:

> The declaration of otg*_plat and otg*_board_data is guarded by
> CONFIG_USB_MUSB_*, but their use in arch_misc_init is not. The
> ifdef flow goes something like:
> 
> if (CONFIG_USB_MUSB_* && other_conditions)
> 	declare usb_data
> if (other_conditions)
> 	use usb_data
> 
> Thus when CONFIG_USB_MUSB_* is not declared, we try to use the
> data structures, but these structures aren't defined.
> 
> To fix this, move the USB initialization code into the same #ifdef
> which guards the declaration of the data structures. Since the DM_USB
> vs legacy cases are completely different, use two versions of
> arch_misc_init(), for readability.
> 
> Signed-off-by: Alexandru Gagniuc <alex.g at adaptrum.com>

Reviewed-by: Tom Rini <trini at konsulko.com>

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170207/3f90a810/attachment.sig>


More information about the U-Boot mailing list