[PATCH] powerpc: dts: keymile: Deduplicate binman code

[Simon Glass]

Hi Pali,

On Fri, 16 Dec 2022 at 15:00, Pali Rohár <pali at kernel.org> wrote:
>
> On Friday 16 December 2022 11:35:53 Simon Glass wrote:
> > Hi,
> >
> >
> > On Fri, 16 Dec 2022 at 11:16, Pali Rohár <pali at kernel.org> wrote:
> > >
> > > PING?
> >
> > Shouldn't these be dealt with by the PPC maintainer?
>
> I'm reminding patch for 4 months on the list and nobody is replying...
> So who can take and apply this patch? Who to contact? I do not know.
>
> > Regards,
> > Simon
> >
> > >
> > > On Monday 21 November 2022 18:40:41 Pali Rohár wrote:
> > > > + Tom
> > > >
> > > > On Tuesday 01 November 2022 23:57:08 Pali Rohár wrote:
> > > > > PING?
> > > > >
> > > > > On Sunday 09 October 2022 14:28:19 Pali Rohár wrote:
> > > > > > PING?
> > > > > >
> > > > > > On Wednesday 03 August 2022 13:20:49 Pali Rohár wrote:
> > > > > > > kmcent2-u-boot.dtsi file contains copy of powerpc u-boot.dtsi binman file.
> > > > > > > So remove code duplication and replace it by including u-boot.dtsi file.
> > > > > > >
> > > > > > > Signed-off-by: Pali Rohár <pali at kernel.org>
> > > > > > > ---
> > > > > > >  arch/powerpc/dts/kmcent2-u-boot.dtsi | 22 ++--------------------
> > > > > > >  1 file changed, 2 insertions(+), 20 deletions(-)
> > > > > > >
> > > > > > > diff --git a/arch/powerpc/dts/kmcent2-u-boot.dtsi b/arch/powerpc/dts/kmcent2-u-boot.dtsi
> > > > > > > index ab76a9f1226c..d0277627644b 100644
> > > > > > > --- a/arch/powerpc/dts/kmcent2-u-boot.dtsi
> > > > > > > +++ b/arch/powerpc/dts/kmcent2-u-boot.dtsi
> > > > > > > @@ -74,24 +74,6 @@
> > > > > > >                 compatible = "fsl,pcie-t104x";
> > > > > > >                 law_trgt_if = <0>;
> > > > > > >         };
> > > > > > > -
> > > > > > > -       binman {
> > > > > > > -               filename = "u-boot-with-dtb.bin";
> > > > > > > -               skip-at-start = <CONFIG_SYS_TEXT_BASE>;
> > > > > > > -               sort-by-offset;
> > > > > > > -               pad-byte = <0xff>;
> > > > > > > -               size = <CONFIG_SYS_MONITOR_LEN>;
> > > > > > > -
> > > > > > > -               u-boot-with-ucode-ptr {
> > > > > > > -                       offset = <CONFIG_SYS_TEXT_BASE>;
> > > > > > > -                       optional-ucode;
> > > > > > > -               };
> > > > > > > -
> > > > > > > -               u-boot-dtb-with-ucode {
> > > > > > > -                       align = <256>;
> > > > > > > -               };
> > > > > > > -               powerpc-mpc85xx-bootpg-resetvec {
> > > > > > > -                       offset = <(CONFIG_RESET_VECTOR_ADDRESS - 0xffc)>;
> > > > > > > -               };
> > > > > > > -       };
> > > > > > >  };
> > > > > > > +
> > > > > > > +#include "u-boot.dtsi"
> > > > > > > --
> > > > > > > 2.20.1
> > > > > > >

>From the MAINTAINERS file:

POWERPC MPC8XX
M: Christophe Leroy <christophe.leroy at csgroup.eu>
S: Maintained
T: git https://source.denx.de/u-boot/custodians/u-boot-mpc8xx.git
F: arch/powerpc/cpu/mpc8xx/

POWERPC MPC83XX
M: Mario Six <mario.six at gdsys.cc>
S: Maintained
T: git https://source.denx.de/u-boot/custodians/u-boot-mpc83xx.git
F: drivers/ram/mpc83xx_sdram.c
F: include/dt-bindings/memory/mpc83xx-sdram.h
F: drivers/sysreset/sysreset_mpc83xx.c
F: drivers/sysreset/sysreset_mpc83xx.h
F: drivers/clk/mpc83xx_clk.c
F: drivers/clk/mpc83xx_clk.h
F: include/dt-bindings/clk/mpc83xx-clk.h
F: drivers/timer/mpc83xx_timer.c
F: drivers/cpu/mpc83xx_cpu.c
F: drivers/cpu/mpc83xx_cpu.h
F: drivers/misc/mpc83xx_serdes.c
F: arch/powerpc/cpu/mpc83xx/
F: arch/powerpc/include/asm/arch-mpc83xx/

POWERPC MPC85XX
M: Marek Behún <kabel at kernel.org>
S: Maintained
T: git https://source.denx.de/u-boot/custodians/u-boot-mpc85xx.git
F: arch/powerpc/cpu/mpc85xx/

Regards,
Simon