[U-Boot] [PATCH v4 3/7] usb: host: xhci-omap: fix double weak board_usb_init functions
Uri Mashiach
uri.mashiach at compulab.co.il
Tue Feb 28 08:00:20 UTC 2017
Hi,
On 02/27/2017 06:22 PM, Roger Quadros wrote:
> Hi,
>
> On 23/02/17 15:39, Uri Mashiach wrote:
>> A weak version of the function board_usb_init is implemented in:
>> common/usb.c
>> drivers/usb/host/xhci-omap.c
>>
>> To fix the double implementations:
>> * Convert the board_usb_init function in drivers/usb/host/xhci-omap.c
>> normal (not weak).
>> * The function board_usb_init in drivers/usb/host/xhci-omap.c calls to
>> the weak function omap_xhci_board_usb_init.
>> * Rename board version of the function board_usb_init to
>> omap_xhci_board_usb_init.
>> Done only for boards that defines CONFIG_USB_XHCI_OMAP.
>>
>> To achieve the same flexibility with the function board_usb_cleanup:
>> * Add a normal (not weak) implementation of the function
>> board_usb_cleanup in drivers/usb/host/xhci-omap.c
>> * The function board_usb_cleanup in drivers/usb/host/xhci-omap.c calls
>> to the weak function omap_xhci_board_usb_cleanup.
>> * Rename board version of the function board_usb_cleanup to
>> omap_xhci_board_usb_cleanup.
>> Done only for boards that defines CONFIG_USB_XHCI_OMAP.
>>
>> Cc: Lokesh Vutla <lokeshvutla at ti.com>
>> Signed-off-by: Uri Mashiach <uri.mashiach at compulab.co.il>
>> Acked-by: Marek Vasut <marex at denx.de>
>> Reviewed-by: Tom Rini <trini at konsulko.com>
>> ---
>> V1 -> V2: Use __weak instead of attribute block
>> V2 -> V4: none
>>
>> board/compulab/cl-som-am57x/cl-som-am57x.c | 2 +-
>> board/ti/am43xx/board.c | 4 ++--
>> board/ti/am57xx/board.c | 4 ++--
>> board/ti/dra7xx/evm.c | 4 ++--
>> drivers/usb/host/xhci-omap.c | 17 +++++++++++++++--
>
> What about board/ti/omap5_uevm/evm.c ?
The symbol CONFIG_USB_XHCI_OMAP is not included in the file
include/configs/omap5_uevm.h, therefore:
The file drivers/usb/host/xhci-omap.c is not included in the compilation
- no double implementations to fix.
--
Thanks and regards,
Uri
More information about the U-Boot
mailing list