[PATCH 1/1] Revert "arm64: Layerscape: Survive LPI one-way reset workaround"

Rayagonda Kokatanur rayagonda.kokatanur at broadcom.com
Mon Sep 13 09:52:10 CEST 2021


On Mon, Sep 13, 2021 at 11:29 AM Priyanka Jain (OSS)
<priyanka.jain at oss.nxp.com> wrote:
>
>
>
> >-----Original Message-----
> >From: U-Boot <u-boot-bounces at lists.denx.de> On Behalf Of Z.Q. Hou
> >Sent: Friday, September 10, 2021 12:13 PM
> >To: Tom Rini <trini at konsulko.com>; u-boot at lists.denx.de;
> >rayagonda.kokatanur at broadcom.com
> >Cc: Priyanka Jain <priyanka.jain at nxp.com>
> >Subject: RE: [PATCH 1/1] Revert "arm64: Layerscape: Survive LPI one-way reset
> >workaround"
> >
> >+ author of gic_lpi_syscon driver
> >
> >Hi Rayagonda,
> >
> >Please add the binding for gic_lpi_syscon driver.
> >
> >Thanks,
> >Zhiqiang
> >
> >> -----Original Message-----
> >> From: Tom Rini <trini at konsulko.com>
> >> Sent: 2021年8月26日 5:05
> >> To: u-boot at lists.denx.de
> >> Cc: Z.Q. Hou <zhiqiang.hou at nxp.com>; Priyanka Jain
> >> <priyanka.jain at nxp.com>
> >> Subject: [PATCH 1/1] Revert "arm64: Layerscape: Survive LPI one-way
> >> reset workaround"
> >>
> >> Ad-hoc bindings that are not part of the upstream device tree /
> >> bindings are not allowed in-tree.  Only bindings that are in-progress
> >> with upstream and then re-synced once agreed upon are.
> >>
> >> This reverts commit af288cb291da3abef6be0875527729296f7de7a0.
> >>
> >> Cc: Hou Zhiqiang <Zhiqiang.Hou at nxp.com>
> >> Cc: Priyanka Jain <priyanka.jain at nxp.com>
> >> Reported-by: Michael Walle <michael at walle.cc>
> >> Signed-off-by: Tom Rini <trini at konsulko.com>
> >> ---
> >>  arch/arm/cpu/armv8/fsl-layerscape/soc.c | 17 +----------------
> >>  arch/arm/dts/fsl-ls1028a.dtsi           |  6 ------
> >>  arch/arm/dts/fsl-ls1088a.dtsi           |  6 ------
> >>  arch/arm/dts/fsl-ls2080a.dtsi           |  6 ------
> >>  arch/arm/dts/fsl-lx2160a.dtsi           |  6 ------
> >>  5 files changed, 1 insertion(+), 40 deletions(-)
> >>
> >> diff --git a/arch/arm/cpu/armv8/fsl-layerscape/soc.c
> >> b/arch/arm/cpu/armv8/fsl-layerscape/soc.c
> >> index 42a096854629..166662a64baf 100644
> >> --- a/arch/arm/cpu/armv8/fsl-layerscape/soc.c
> >> +++ b/arch/arm/cpu/armv8/fsl-layerscape/soc.c
> >> @@ -43,22 +43,7 @@ DECLARE_GLOBAL_DATA_PTR;  #ifdef
> >CONFIG_GIC_V3_ITS
> >> int ls_gic_rd_tables_init(void *blob)  {
> >> -    struct fdt_memory lpi_base;
> >> -    fdt_addr_t addr;
> >> -    fdt_size_t size;
> >> -    int offset, ret;
> >> -
> >> -    offset = fdt_path_offset(gd->fdt_blob, "/syscon at 0x80000000");
> >> -    addr = fdtdec_get_addr_size_auto_noparent(gd->fdt_blob, offset,
> >> "reg",
> >> -                                              0, &size, false);
> >> -
> >> -    lpi_base.start = addr;
> >> -    lpi_base.end = addr + size - 1;
> >> -    ret = fdtdec_add_reserved_memory(blob, "lpi_rd_table", &lpi_base,
> >> NULL, false);
> >> -    if (ret) {
> >> -            debug("%s: failed to add reserved memory\n", __func__);
> >> -            return ret;
> >> -    }
> >> +    int ret;
> >>
> >>      ret = gic_lpi_tables_init();
> >>      if (ret)
> >> diff --git a/arch/arm/dts/fsl-ls1028a.dtsi
> >> b/arch/arm/dts/fsl-ls1028a.dtsi index 50f9b527cde1..53b052ed3271
> >> 100644
> >> --- a/arch/arm/dts/fsl-ls1028a.dtsi
> >> +++ b/arch/arm/dts/fsl-ls1028a.dtsi
> >> @@ -44,12 +44,6 @@
> >>                                       IRQ_TYPE_LEVEL_LOW)>;
> >>      };
> >>
> >> -    gic_lpi_base: syscon at 0x80000000 {
> >> -            compatible = "gic-lpi-base";
> >> -            reg = <0x0 0x80000000 0x0 0x100000>;
> >> -            max-gic-redistributors = <2>;
> >> -    };
> >> -
> >>      timer {
> >>              compatible = "arm,armv8-timer";
> >>              interrupts = <GIC_PPI 13 (GIC_CPU_MASK_SIMPLE(2) | diff --git
> >> a/arch/arm/dts/fsl-ls1088a.dtsi b/arch/arm/dts/fsl-ls1088a.dtsi index
> >> 64caa600ad77..3a5a50fb8313 100644
> >> --- a/arch/arm/dts/fsl-ls1088a.dtsi
> >> +++ b/arch/arm/dts/fsl-ls1088a.dtsi
> >> @@ -27,12 +27,6 @@
> >>              interrupts = <1 9 0x4>;
> >>      };
> >>
> >> -    gic_lpi_base: syscon at 0x80000000 {
> >> -            compatible = "gic-lpi-base";
> >> -            reg = <0x0 0x80000000 0x0 0x100000>;
> >> -            max-gic-redistributors = <8>;
> >> -    };
> >> -
> >>      timer {
> >>              compatible = "arm,armv8-timer";
> >>              interrupts = <1 13 0x8>, /* Physical Secure PPI, active-low */ diff
> >> --git a/arch/arm/dts/fsl-ls2080a.dtsi b/arch/arm/dts/fsl-ls2080a.dtsi
> >> index 7374d580e07e..278daeeb6eea 100644
> >> --- a/arch/arm/dts/fsl-ls2080a.dtsi
> >> +++ b/arch/arm/dts/fsl-ls2080a.dtsi
> >> @@ -27,12 +27,6 @@
> >>              interrupts = <1 9 0x4>;
> >>      };
> >>
> >> -    gic_lpi_base: syscon at 0x80000000 {
> >> -            compatible = "gic-lpi-base";
> >> -            reg = <0x0 0x80000000 0x0 0x100000>;
> >> -            max-gic-redistributors = <8>;
> >> -    };
> >> -
> >>      timer {
> >>              compatible = "arm,armv8-timer";
> >>              interrupts = <1 13 0x8>, /* Physical Secure PPI, active-low */ diff
> >> --git a/arch/arm/dts/fsl-lx2160a.dtsi b/arch/arm/dts/fsl-lx2160a.dtsi
> >> index
> >> a6f0e9bc56be..3b5f0d119e76 100644
> >> --- a/arch/arm/dts/fsl-lx2160a.dtsi
> >> +++ b/arch/arm/dts/fsl-lx2160a.dtsi
> >> @@ -43,12 +43,6 @@
> >>              interrupts = <1 9 0x4>;
> >>      };
> >>
> >> -    gic_lpi_base: syscon at 0x80000000 {
> >> -            compatible = "gic-lpi-base";
> >> -            reg = <0x0 0x80000000 0x0 0x200000>;
> >> -            max-gic-redistributors = <16>;
> >> -    };
> >> -
> >>      timer {
> >>              compatible = "arm,armv8-timer";
> >>              interrupts = <1 13 0x8>, /* Physical Secure PPI, active-low */
> >> --
> >> 2.17.1
>
>
> Tom,
>
> I agree that bindings patch is essential for device-tree changes.
> But merging this patch will break Layerscape Platforms.
> We have requested Rayagonda to submit Binding patch to upstream for review.
>
> If first patch of binding gets submitted timely, can we defer this patch.
>
>  Regards
> Priyanka
>

++Bharat, Roman and Henrik.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 4230 bytes
Desc: S/MIME Cryptographic Signature
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20210913/01dc3739/attachment.bin>


More information about the U-Boot mailing list