[PATCH 2/3] configs: keystone2: Do not include hardware.h

Tom Rini trini at konsulko.com
Fri Nov 17 23:40:44 CET 2023


On Fri, Nov 17, 2023 at 04:38:28PM -0600, Andrew Davis wrote:

> This is a hacky way to have this file included in all source files that
> include common.h, instead just include from the files that need it.
> 
> Signed-off-by: Andrew Davis <afd at ti.com>
> ---
>  drivers/memory/ti-aemif.c            | 1 +
>  drivers/soc/ti/keystone_serdes.c     | 1 +
>  include/configs/ti_armv7_keystone2.h | 1 -
>  3 files changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/memory/ti-aemif.c b/drivers/memory/ti-aemif.c
> index c4bc88c1510..41325eb0f94 100644
> --- a/drivers/memory/ti-aemif.c
> +++ b/drivers/memory/ti-aemif.c
> @@ -7,6 +7,7 @@
>   */
>  
>  #include <common.h>
> +#include <asm/arch/hardware.h>
>  #include <asm/ti-common/ti-aemif.h>
>  
>  #define AEMIF_WAITCYCLE_CONFIG		(KS2_AEMIF_CNTRL_BASE + 0x4)
> diff --git a/drivers/soc/ti/keystone_serdes.c b/drivers/soc/ti/keystone_serdes.c
> index 2ece1a8f647..0e1bf8ff39d 100644
> --- a/drivers/soc/ti/keystone_serdes.c
> +++ b/drivers/soc/ti/keystone_serdes.c
> @@ -8,6 +8,7 @@
>  
>  #include <errno.h>
>  #include <common.h>
> +#include <asm/io.h>
>  #include <asm/ti-common/keystone_serdes.h>
>  #include <linux/bitops.h>
>  

Is there going to be a follow-up to remove common.h from these files
then? Thanks.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20231117/83e0eeee/attachment.sig>


More information about the U-Boot mailing list