Re: CVS commit: src/sys/arch/arm/broadcom

[Nick Hudson <nick.hudson%gmx.co.uk@localhost>]

On 20/10/2018 08:19, Ryo Shimizu wrote:
> > On 20/10/2018 06:53, Ryo Shimizu wrote:
> > > Module Name:	src
> > > Committed By:	ryo
> > > Date:		Sat Oct 20 05:53:22 UTC 2018
> > >
> > > Modified Files:
> > > 	src/sys/arch/arm/broadcom: bcm283x_platform.c
> > >
> > > Log Message:
> > > - fix AP spinup on bcm283x
> > > - "brcm,bcm2836-smp" is a enable-method for only 32bit bcm2836.
> >
> > I think the dts should be fixed rather than #ifdef __arm__
> I agree that we need to change dts to eliminate 32bit enable method from
> the dtb for 64bit.

Cool

>   However, it is also strange that enable-method code
> for 32bit ARM remains in the 64bit binary as well.
Lots more code in GENERIC64 than this little snippet that won't ever get 
used, e.g. SOC_{RK*,TEGRA,SUN*,etc} on RPI3

Nick