[PATCH] imx8m*_venice: move venice to OF_UPSTREAM

Tim Harvey tharvey at gateworks.com
Wed Mar 13 17:31:14 CET 2024


On Wed, Mar 13, 2024 at 6:20 AM Sumit Garg <sumit.garg at linaro.org> wrote:
>
> On Wed, 13 Mar 2024 at 06:46, Fabio Estevam <festevam at gmail.com> wrote:
> >
> > Hi Tim,
> >
> > On Tue, Mar 12, 2024 at 4:05 PM Tim Harvey <tharvey at gateworks.com> wrote:
> > >
> > > Move to imx8m{m,n,p}-venice to OF_UPSTREAM:
> > >  - replace the non-upstream generic imx8m{m,n,p}-venice dt with one of the
> > >    dt's from the OF_LIST
> > >  - handle the fact that dtbs now have a 'freescale/' prefix
> > >  - imply OF_UPSTREAM
> > >  - remove rudundant files from arch/arm/dts leaving only the
> > >    *-u-boot.dtsi files
> > >
> > > Signed-off-by: Tim Harvey <tharvey at gateworks.com>
> > ...
> > >  33 files changed, 13 insertions(+), 10658 deletions(-)
> >
> > This diff looks great :-)
>
> +1
>
> Reviewed-by: Sumit Garg <sumit.garg at linaro.org>
>

Hi Sumit,

Thanks for your work on this - I imagine over time this will
de-duplicate a lot of work!

I have a couple of questions about OF_UPSTREAM I haven't found the
answer to yet:
1. how do you determine what the last sync point was in dts/upstream?
(ie what kernel version is it currently sync'd with)
2. how often will dts/upstream get re-synced (not the
devicetree-rebasing.git but u-boot dts/upstream), who do you suspect
will do be doing it?
3. how would one go about adding a new feature via dt to uboot when
the same feature has not yet landed in dts upstream? perhaps the
answer is it must land upstream first or do you suspect it would be ok
to put something in a u-boot.dtsi that can later get removed as
redundant? I ask mainly for being able to add things quickly to a
downstream U-Boot repo that lags behind upstream U-Boot

Best Regards,

Tim


More information about the U-Boot mailing list