[PATCH 1/2] mach-snapdragon/mach-ipq40xx: fix, merge and refactoring

[Robert Marko]

On Tue, Nov 8, 2022 at 4:20 PM Tom Rini <trini at konsulko.com> wrote:
>
> On Sun, Nov 06, 2022 at 02:52:56AM +0100, Andrey VOLKOV wrote:
> > Hi Robert, Tom,
> >
> > Updated patches are following, could you review/accept them?
> >
> > Le 04/11/2022 à 12:05, Robert Marko a écrit :
> > > On Tue, Oct 25, 2022 at 3:31 AM Andrey Volkov<andrey at volkov.fr>  wrote:
> > >> SoC: qcom: Add missing Qualcomm's SOCs definitions to the arch KConfig
> > >>
> > >> From: Andrey VOLKOV<andrey at volkov.fr>
> > >>
> > >> Add APQ8016/APQ8096/QCS40X decls to the KConfig, convert SDM845 declaration to the hidden one (selected by target),
> > >> and update mach-snapdragon/Makefile accordingly.
> > >>
> > >> Also move 'CONFIG_ARM64' choice to the new declarations, since 'Snapdragon' contains Cortex A7A members too.
> > >>
> > >> Signed-off-by: Andrey VOLKOV<andrey at volkov.fr>
> > > I like this cleanup effort, however, can you rebase this to apply on the master?
> > > I would like to use this as a base of moving IPQ40xx to use Linux DTS
> > > and keep it in sync.
> >
> > It's nice idea, except that full blown DTS is huge enough (+some KiB at least),
> > and i'm afraid it's an old, old story about failing to make it :(.
> > Probably will be better if we will have some option that allowed to import and
> > reuse DTS files directly from the kernel.
> > Something like CONFIG_DTS_FILE_PATH/CONFIG_DTS_INCLUDES/...
> >
> > @Tom, any additional comments/objections?
> >
> > Btw As far as I remember, this topic (reuse kernel's DTS) has been discussed
> > repeatedly since the mid-2000s, but I don't remember why it was not implemented yet.
>
> Is the problem that the full DTS would make the resulting binary too
> large to fit in some restricted and can't be changed easily space?
> Generally we do want to keep the DTS files in-sync with the kernel.

Most IPQ40xx devices have a slot of 512kB left for the bootloader
which cannot be easily
expanded, however, that should still be enough for synced DTS anyway.

I am now more familiar with U-boot and really dont like the IPQ40xx
target as it is and
would like to rework it, including using upstream DTS.

Regards,
Robert
>
> --
> Tom



-- 
Robert Marko
Staff Embedded Linux Engineer
Sartura Ltd.
Lendavska ulica 16a
10000 Zagreb, Croatia
Email: robert.marko at sartura.hr
Web: www.sartura.hr