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

2023-11-29 Thread Tom Rini
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 Applied to u-boot/next, thanks! -- Tom signature.asc

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

2023-11-21 Thread Tom Rini
On Mon, Nov 20, 2023 at 07:16:03PM -0700, Simon Glass wrote: > Hi Tom, > > On Mon, 20 Nov 2023 at 08:26, Tom Rini wrote: > > > > On Mon, Nov 20, 2023 at 08:49:32AM -0600, Andrew Davis wrote: > > > On 11/17/23 4:40 PM, Tom Rini wrote: > > > > On Fri, Nov 17, 2023 at 04:38:28PM -0600, Andrew Davis

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

2023-11-20 Thread Simon Glass
Hi Tom, On Mon, 20 Nov 2023 at 08:26, Tom Rini wrote: > > On Mon, Nov 20, 2023 at 08:49:32AM -0600, Andrew Davis wrote: > > On 11/17/23 4:40 PM, Tom Rini wrote: > > > 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

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

2023-11-20 Thread Tom Rini
On Mon, Nov 20, 2023 at 08:49:32AM -0600, Andrew Davis wrote: > On 11/17/23 4:40 PM, Tom Rini wrote: > > 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

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

2023-11-20 Thread Andrew Davis
On 11/17/23 4:40 PM, Tom Rini wrote: 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 ---

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

2023-11-17 Thread Tom Rini
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 > --- > drivers/memory/ti-aemif.c| 1 + >

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

2023-11-17 Thread Andrew Davis
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 --- drivers/memory/ti-aemif.c| 1 + drivers/soc/ti/keystone_serdes.c | 1 +