Hi Tom,
Thank you for the patch
On mer., mai 01, 2024 at 19:31, Tom Rini <tr...@konsulko.com> wrote:
> Remove <common.h> from this driver directory and when needed
> add missing include files directly.
>
> Signed-off-by: Tom Rini <tr...@konsulko.com>
[...]
> diff --git a/drivers/usb/host/xhci-rcar.c b/drivers/usb/host/xhci-rcar.c
> index fedcf7869295..38c5928faed5 100644
> --- a/drivers/usb/host/xhci-rcar.c
> +++ b/drivers/usb/host/xhci-rcar.c
> @@ -5,7 +5,6 @@
> * Renesas RCar USB HOST xHCI Controller
> */
>
> -#include <common.h>
> #include <clk.h>
> #include <dm.h>
> #include <fdtdec.h>
> diff --git a/drivers/usb/host/xhci-ring.c b/drivers/usb/host/xhci-ring.c
> index 910c5f3352b8..1360a5940fa0 100644
> --- a/drivers/usb/host/xhci-ring.c
> +++ b/drivers/usb/host/xhci-ring.c
> @@ -13,7 +13,6 @@
> * Vikas Sajjan <vikas.saj...@samsung.com>
> */
>
> -#include <common.h>
This generates the following build warning with
khadas-vim3_android_defconfig:
drivers/usb/host/xhci-ring.c: In function 'xhci_wait_for_event':
drivers/usb/host/xhci-ring.c:464:28: warning: implicit declaration of function
'get_timer'; did you mean 'get_mem'? [-Wimplicit-function-declaration]
464 | unsigned long ts = get_timer(0);
| ^~~~~~~~~
| get_mem
Adding: "#include <time.h>" fixes the warning.
With the above fix included:
Reviewed-by: Mattijs Korpershoek <mkorpersh...@baylibre.com>
> #include <cpu_func.h>
> #include <log.h>
> #include <asm/byteorder.h>
> diff --git a/drivers/usb/host/xhci.c b/drivers/usb/host/xhci.c
> index 741e186ee05b..d30725d3fcaa 100644
> --- a/drivers/usb/host/xhci.c
> +++ b/drivers/usb/host/xhci.c
> @@ -19,7 +19,6 @@
> * The quirk devices support hasn't been given yet.
> */
>
> -#include <common.h>
> #include <cpu_func.h>
> #include <dm.h>
> #include <dm/device_compat.h>
[...]
