[PATCH 12/13] ARM: OMAP5: Add the build support
Tony Lindgren
tony at atomide.com
Tue May 8 11:57:07 EDT 2012
* Cousson, Benoit <b-cousson at ti.com> [120508 02:23]:
> Hi Tony,
>
> On 5/7/2012 7:37 PM, Tony Lindgren wrote:
> >* R, Sricharan<r.sricharan at ti.com> [120506 20:39]:
> >>>>+config MACH_OMAP5_SEVM
> >>>>+ bool "OMAP5 sevm Board"
> >>>>+ depends on ARCH_OMAP5
> >>>>+
> >>>> config OMAP3_EMU
> >>>> bool "OMAP3 debugging peripherals"
> >>>> depends on ARCH_OMAP3
> >>>
> >>>No need for it here either. Actually, I think this whole chunk
> >>>can be now left out since it's DT based?
> >>>he
> >> ok, but the concern here was that without this macro
> >> the print from compress and subsequently early
> >>prints appear broken.
> >>
> >>machine_is_omap5_sevm becomes zero without this config and
> >>machine_is_xxxx is used by the macro _DEBUG_LL_ENTRY
> >>uncompress.h.
> >
> >Ah I see.
>
> Mmm, still, I guess this mechanism should be deprecated with DT?
> We cannot keep creating fake board entry since we are not supposed
> to have MACHINE_ID anymore.
> That will break DEBUG_LL for sure, but we knew about that current
> limitation with DT.
For most usecases for DEBUG_LL we can get away in two steps:
1. Debug output for uncompress code based on DT parsing (I think
this is already there)
2. Early console/earlyprintk parsed from DT that gets set up in
init_early
This should be enough for most users, then the people who need
to debug early kernel code until init_early is done can patch
it in manually.
> The point is that we should fix the code to stop relying on a
> machine id going forward.
> And maybe we can parse the FDT at that time?
Yes that should be trivial for most use cases, we just need
to initialize the earlycon/earlyprintk based on the DT.
See kernel/early_printk.c for more info. We just need to
set up the hardware specific function for early_write instead
of printch.
Regards,
Tony
More information about the linux-arm-kernel
mailing list