Re: [PATCH V11 3/5] kbuild: Allow .dtso format for overlay source files

2021-04-09 Thread Masahiro Yamada
On Tue, Mar 16, 2021 at 5:01 PM Geert Uytterhoeven wrote: > > Hi Frank, > > On Tue, Mar 16, 2021 at 6:39 AM Frank Rowand wrote: > > On 3/15/21 5:12 PM, Laurent Pinchart wrote: > > > On Tue, Mar 16, 2021 at 02:43:45AM +0900, Masahiro Yamada wrote: > > >> But how can we fix

Re: [PATCH V11 3/5] kbuild: Allow .dtso format for overlay source files

2021-03-24 Thread Frank Rowand
Hi Viresh, On 3/24/21 2:34 AM, Frank Rowand wrote: > On 3/16/21 12:42 AM, Viresh Kumar wrote: >> On 16-03-21, 00:36, Frank Rowand wrote: >>> I should have looked at patch 3/5 more carefully instead of counting on >>> Masahiro to check it out and simply build testing. >>> >>> Patch 3/5 does not

Re: [PATCH V11 3/5] kbuild: Allow .dtso format for overlay source files

2021-03-24 Thread Frank Rowand
On 3/16/21 12:42 AM, Viresh Kumar wrote: > On 16-03-21, 00:36, Frank Rowand wrote: >> I should have looked at patch 3/5 more carefully instead of counting on >> Masahiro to check it out and simply build testing. >> >> Patch 3/5 does not seem correct. I'll look over all the makefile related >>

Re: [PATCH V11 3/5] kbuild: Allow .dtso format for overlay source files

2021-03-16 Thread Geert Uytterhoeven
Hi Frank, On Tue, Mar 16, 2021 at 6:39 AM Frank Rowand wrote: > On 3/15/21 5:12 PM, Laurent Pinchart wrote: > > On Tue, Mar 16, 2021 at 02:43:45AM +0900, Masahiro Yamada wrote: > >> But how can we fix drivers/gpu/drm/rcar-du/rcar_du_of_lvds_r8a779*.dts > >> if these are doing bad things. > >>

Re: [PATCH V11 3/5] kbuild: Allow .dtso format for overlay source files

2021-03-15 Thread Viresh Kumar
On 16-03-21, 00:36, Frank Rowand wrote: > I should have looked at patch 3/5 more carefully instead of counting on > Masahiro to check it out and simply build testing. > > Patch 3/5 does not seem correct. I'll look over all the makefile related > changes that have been accepted (if any) and patch

Re: [PATCH V11 3/5] kbuild: Allow .dtso format for overlay source files

2021-03-15 Thread Frank Rowand
On 3/15/21 5:12 PM, Laurent Pinchart wrote: > Hi Yamada-san, > > On Tue, Mar 16, 2021 at 02:43:45AM +0900, Masahiro Yamada wrote: >> On Mon, Mar 15, 2021 at 3:40 PM Viresh Kumar wrote: >>> On 14-03-21, 20:16, Frank Rowand wrote: On 3/12/21 11:11 PM, Frank Rowand wrote: > On 3/12/21 1:13

Re: [PATCH V11 3/5] kbuild: Allow .dtso format for overlay source files

2021-03-15 Thread Frank Rowand
On 3/15/21 1:40 AM, Viresh Kumar wrote: > On 14-03-21, 20:16, Frank Rowand wrote: >> On 3/12/21 11:11 PM, Frank Rowand wrote: >>> On 3/12/21 1:13 AM, Viresh Kumar wrote: On 12-03-21, 01:09, Frank Rowand wrote: > I suggested having the .dtso files include the .dts file because that is

Re: [PATCH V11 3/5] kbuild: Allow .dtso format for overlay source files

2021-03-15 Thread Viresh Kumar
On 16-03-21, 02:43, Masahiro Yamada wrote: > On Mon, Mar 15, 2021 at 3:40 PM Viresh Kumar wrote: > > On 14-03-21, 20:16, Frank Rowand wrote: > > What about doing this then in unittest's Makefile instead (which I > > already suggested earlier), that will make everything work just fine > > without

Re: [PATCH V11 3/5] kbuild: Allow .dtso format for overlay source files

2021-03-15 Thread Laurent Pinchart
Hi Yamada-san, On Tue, Mar 16, 2021 at 02:43:45AM +0900, Masahiro Yamada wrote: > On Mon, Mar 15, 2021 at 3:40 PM Viresh Kumar wrote: > > On 14-03-21, 20:16, Frank Rowand wrote: > > > On 3/12/21 11:11 PM, Frank Rowand wrote: > > > > On 3/12/21 1:13 AM, Viresh Kumar wrote: > > > >> On 12-03-21,

Re: [PATCH V11 3/5] kbuild: Allow .dtso format for overlay source files

2021-03-15 Thread Masahiro Yamada
On Mon, Mar 15, 2021 at 3:40 PM Viresh Kumar wrote: > > On 14-03-21, 20:16, Frank Rowand wrote: > > On 3/12/21 11:11 PM, Frank Rowand wrote: > > > On 3/12/21 1:13 AM, Viresh Kumar wrote: > > >> On 12-03-21, 01:09, Frank Rowand wrote: > > >>> I suggested having the .dtso files include the .dts

Re: [PATCH V11 3/5] kbuild: Allow .dtso format for overlay source files

2021-03-15 Thread Viresh Kumar
On 14-03-21, 20:16, Frank Rowand wrote: > On 3/12/21 11:11 PM, Frank Rowand wrote: > > On 3/12/21 1:13 AM, Viresh Kumar wrote: > >> On 12-03-21, 01:09, Frank Rowand wrote: > >>> I suggested having the .dtso files include the .dts file because that is > >>> a relatively > >>> small and easy change

Re: [PATCH V11 3/5] kbuild: Allow .dtso format for overlay source files

2021-03-14 Thread Frank Rowand
Hi VIresh, On 3/12/21 11:11 PM, Frank Rowand wrote: > On 3/12/21 1:13 AM, Viresh Kumar wrote: >> On 12-03-21, 01:09, Frank Rowand wrote: >>> I suggested having the .dtso files include the .dts file because that is a >>> relatively >>> small and easy change to test. What would probably make more

Re: [PATCH V11 3/5] kbuild: Allow .dtso format for overlay source files

2021-03-12 Thread Frank Rowand
On 3/12/21 1:13 AM, Viresh Kumar wrote: > On 12-03-21, 01:09, Frank Rowand wrote: >> I suggested having the .dtso files include the .dts file because that is a >> relatively >> small and easy change to test. What would probably make more sense is the >> rename >> the existing overlay .dts files

Re: [PATCH V11 3/5] kbuild: Allow .dtso format for overlay source files

2021-03-11 Thread Viresh Kumar
On 12-03-21, 01:09, Frank Rowand wrote: > I suggested having the .dtso files include the .dts file because that is a > relatively > small and easy change to test. What would probably make more sense is the > rename > the existing overlay .dts files to be .dtso files and then for each overlay >

Re: [PATCH V11 3/5] kbuild: Allow .dtso format for overlay source files

2021-03-11 Thread Frank Rowand
On 3/12/21 1:03 AM, Frank Rowand wrote: > Hi Viresh, > > On 3/11/21 10:47 PM, Viresh Kumar wrote: >> On 10-03-21, 20:24, Masahiro Yamada wrote: >>> Even without "-I dts", >>> >>>inform = guess_input_format(arg, "dts"); >>> >>> seems to fall back to "dts" anyway, >>> but I guess you wanted to

Re: [PATCH V11 3/5] kbuild: Allow .dtso format for overlay source files

2021-03-11 Thread Frank Rowand
Hi Viresh, On 3/11/21 10:47 PM, Viresh Kumar wrote: > On 10-03-21, 20:24, Masahiro Yamada wrote: >> Even without "-I dts", >> >>inform = guess_input_format(arg, "dts"); >> >> seems to fall back to "dts" anyway, >> but I guess you wanted to make this explicit, correct? > > >>> +# Required

Re: [PATCH V11 3/5] kbuild: Allow .dtso format for overlay source files

2021-03-11 Thread Viresh Kumar
On 10-03-21, 20:24, Masahiro Yamada wrote: > Even without "-I dts", > >inform = guess_input_format(arg, "dts"); > > seems to fall back to "dts" anyway, > but I guess you wanted to make this explicit, correct? > > +# Required for of unit-test files as they can't be renamed to .dtso > > If

Re: [PATCH V11 3/5] kbuild: Allow .dtso format for overlay source files

2021-03-11 Thread Viresh Kumar
On 11-03-21, 00:18, Masahiro Yamada wrote: > On Wed, Mar 10, 2021 at 11:48 PM Viresh Kumar wrote: > > > > On 10-03-21, 20:29, Masahiro Yamada wrote: > > > BTW, is the attached patch good for DTC? > > > > > > I do not know when '-O dtbo' is useful, > > > unless I am missing something. > > > > It

Re: [PATCH V11 3/5] kbuild: Allow .dtso format for overlay source files

2021-03-10 Thread Frank Rowand
On 3/10/21 9:15 AM, Masahiro Yamada wrote: > On Wed, Mar 10, 2021 at 11:47 PM Viresh Kumar wrote: >> >> On 10-03-21, 20:24, Masahiro Yamada wrote: >>> On Wed, Mar 10, 2021 at 2:35 PM Viresh Kumar >>> wrote: diff --git a/scripts/Makefile.lib b/scripts/Makefile.lib index

Re: [PATCH V11 3/5] kbuild: Allow .dtso format for overlay source files

2021-03-10 Thread Masahiro Yamada
On Wed, Mar 10, 2021 at 11:47 PM Viresh Kumar wrote: > > On 10-03-21, 20:24, Masahiro Yamada wrote: > > On Wed, Mar 10, 2021 at 2:35 PM Viresh Kumar > > wrote: > > > diff --git a/scripts/Makefile.lib b/scripts/Makefile.lib > > > index bc045a54a34e..59e86f67f9e0 100644 > > > ---

Re: [PATCH V11 3/5] kbuild: Allow .dtso format for overlay source files

2021-03-10 Thread Masahiro Yamada
On Wed, Mar 10, 2021 at 11:48 PM Viresh Kumar wrote: > > On 10-03-21, 20:29, Masahiro Yamada wrote: > > BTW, is the attached patch good for DTC? > > > > I do not know when '-O dtbo' is useful, > > unless I am missing something. > > It is useful if we are sending the -O option all the time (I have

Re: [PATCH V11 3/5] kbuild: Allow .dtso format for overlay source files

2021-03-10 Thread Viresh Kumar
On 10-03-21, 20:29, Masahiro Yamada wrote: > BTW, is the attached patch good for DTC? > > I do not know when '-O dtbo' is useful, > unless I am missing something. It is useful if we are sending the -O option all the time (I have already given more details to your patch) as outform will not be

Re: [PATCH V11 3/5] kbuild: Allow .dtso format for overlay source files

2021-03-10 Thread Viresh Kumar
On 10-03-21, 20:24, Masahiro Yamada wrote: > On Wed, Mar 10, 2021 at 2:35 PM Viresh Kumar wrote: > > diff --git a/scripts/Makefile.lib b/scripts/Makefile.lib > > index bc045a54a34e..59e86f67f9e0 100644 > > --- a/scripts/Makefile.lib > > +++ b/scripts/Makefile.lib > > @@ -339,7 +339,7 @@

Re: [PATCH V11 3/5] kbuild: Allow .dtso format for overlay source files

2021-03-10 Thread Masahiro Yamada
On Wed, Mar 10, 2021 at 8:24 PM Masahiro Yamada wrote: > > On Wed, Mar 10, 2021 at 2:35 PM Viresh Kumar wrote: > > > > Since the overlays dtb files are now named as .dtbo, there is a lot of > > interest in similarly naming the overlay source dts files as .dtso. > > > > This patch makes the

Re: [PATCH V11 3/5] kbuild: Allow .dtso format for overlay source files

2021-03-10 Thread Masahiro Yamada
On Wed, Mar 10, 2021 at 2:35 PM Viresh Kumar wrote: > > Since the overlays dtb files are now named as .dtbo, there is a lot of > interest in similarly naming the overlay source dts files as .dtso. > > This patch makes the necessary changes to allow .dtso format for overlay > source files. > >

[PATCH V11 3/5] kbuild: Allow .dtso format for overlay source files

2021-03-09 Thread Viresh Kumar
Since the overlays dtb files are now named as .dtbo, there is a lot of interest in similarly naming the overlay source dts files as .dtso. This patch makes the necessary changes to allow .dtso format for overlay source files. Note that we still support generating .dtbo files from .dts files.