[PATCH 02/15] arm: sunxi: Remove most SoC-specific config header files

Tom Rini trini at konsulko.com
Thu Mar 10 15:13:08 CET 2022


On Thu, Mar 10, 2022 at 02:04:31PM +0000, Andre Przywara wrote:
> On Thu, 10 Mar 2022 08:32:11 -0500
> Tom Rini <trini at konsulko.com> wrote:
> 
> > On Thu, Mar 10, 2022 at 10:58:50AM +0000, Andre Przywara wrote:
> > > On Wed,  9 Mar 2022 20:43:51 -0500
> > > Tom Rini <trini at konsulko.com> wrote:
> > > 
> > > Hi Tom,
> > >   
> > > > At this point, sunxi-common.h is used for all systems except for some
> > > > MACH_SUN50I variants.  Remove the now basically empty header files and
> > > > update CONFIG_SYS_CONFIG_NAME to use sunxi-common.h directly.  
> > > 
> > > So is this #include <include/configs/$CONFIG_SYS_CONFIG_NAME.h> the only
> > > user of this symbol? I was on the brink of removing those files several
> > > times already, but wasn't sure if that has side effects.
> > > I think buildman names use that as well, so we can't run "./buildman
> > > sun7i" anymore?
> > > If that's the only thing, I am fine with this.  
> > 
> > Yup, that's the only place it's used.
> > 
> > > Actually the content of sun50i.h is bogus, since we don't define
> > > CONFIG_GICV2, so don't need the GIC addresses.
> > > So this can go as well, and SYS_CONFIG_NAME can maybe just become "sunxi"?
> > > 
> > > Samuel, does this torpedo any RISC-V efforts?  
> > 
> > I'll do the move for v2 and keep an eye out if that's a problem for
> > RISC-V efforts.
> 
> Thanks. I guess we can bring it back, if needed, or turn it upside down
> by including "sunxi-riscv.h" from sunxi.h, inside an #ifdef.

Or pick the conflicting symbol as something to migrate to Kconfig? :)

-- 
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/20220310/52148b06/attachment.sig>


More information about the U-Boot mailing list