mbox series

[0/8] Fixup old unmatched patches, thus fix l2 cache

Message ID 20210326205803.157027-1-xnox@ubuntu.com
Headers show
Series Fixup old unmatched patches, thus fix l2 cache | expand

Message

Dimitri John Ledkov March 26, 2021, 8:57 p.m. UTC
Update old unmatched patches to new revisions.

This also fixes l2 cache failure as seen on cold boot.

I was unable to test this on neither my board, nor in qemu.

BugLink: https://bugs.launchpad.net/bugs/1920916

Dimitri John Ledkov (3):
  Revert "dt-bindings: riscv: Update DT binding docs to support SiFive
    FU740 SoC"
  Revert "RISC-V: sifive_l2_cache: Update L2 cache driver to support
    SiFive FU740"
  Revert "RISC-V: Update l2 cache DT documentation to add support for
    SiFive FU740"

Yash Shah (5):
  dt-bindings: riscv: Update DT binding docs to support SiFive FU740 SoC
  dt-bindings: pwm: Update DT binding docs to support SiFive FU740 SoC
  dt-bindings: gpio: Update DT binding docs to support SiFive FU740 SoC
  dt-bindings: riscv: Update l2 cache DT documentation to add support
    for SiFive FU740
  RISC-V: sifive_l2_cache: Update L2 cache driver to support SiFive
    FU740

 .../devicetree/bindings/gpio/sifive,gpio.yaml |  6 ++--
 .../devicetree/bindings/i2c/i2c-ocores.txt    | 10 +++----
 .../bindings/riscv/sifive-l2-cache.yaml       |  7 ++---
 drivers/soc/sifive/sifive_l2_cache.c          | 28 ++++---------------
 4 files changed, 16 insertions(+), 35 deletions(-)

Comments

Stefan Bader March 29, 2021, 6:08 a.m. UTC | #1
On 26.03.21 21:57, Dimitri John Ledkov wrote:
> Update old unmatched patches to new revisions.
> 
> This also fixes l2 cache failure as seen on cold boot.
> 
> I was unable to test this on neither my board, nor in qemu.
> 
> BugLink: https://bugs.launchpad.net/bugs/1920916
> 
> Dimitri John Ledkov (3):
>    Revert "dt-bindings: riscv: Update DT binding docs to support SiFive
>      FU740 SoC"
>    Revert "RISC-V: sifive_l2_cache: Update L2 cache driver to support
>      SiFive FU740"
>    Revert "RISC-V: Update l2 cache DT documentation to add support for
>      SiFive FU740"
> 
> Yash Shah (5):
>    dt-bindings: riscv: Update DT binding docs to support SiFive FU740 SoC
>    dt-bindings: pwm: Update DT binding docs to support SiFive FU740 SoC
>    dt-bindings: gpio: Update DT binding docs to support SiFive FU740 SoC
>    dt-bindings: riscv: Update l2 cache DT documentation to add support
>      for SiFive FU740
>    RISC-V: sifive_l2_cache: Update L2 cache driver to support SiFive
>      FU740
> 
>   .../devicetree/bindings/gpio/sifive,gpio.yaml |  6 ++--
>   .../devicetree/bindings/i2c/i2c-ocores.txt    | 10 +++----
>   .../bindings/riscv/sifive-l2-cache.yaml       |  7 ++---
>   drivers/soc/sifive/sifive_l2_cache.c          | 28 ++++---------------
>   4 files changed, 16 insertions(+), 35 deletions(-)
> 
No clue which of the many series that are supported this is for.

-Stefan
Colin Ian King March 29, 2021, 9:13 a.m. UTC | #2
On 26/03/2021 20:57, Dimitri John Ledkov wrote:
> Update old unmatched patches to new revisions.
> 
> This also fixes l2 cache failure as seen on cold boot.
> 
> I was unable to test this on neither my board, nor in qemu.
> 
> BugLink: https://bugs.launchpad.net/bugs/1920916
> 
> Dimitri John Ledkov (3):
>   Revert "dt-bindings: riscv: Update DT binding docs to support SiFive
>     FU740 SoC"
>   Revert "RISC-V: sifive_l2_cache: Update L2 cache driver to support
>     SiFive FU740"
>   Revert "RISC-V: Update l2 cache DT documentation to add support for
>     SiFive FU740"
> 
> Yash Shah (5):
>   dt-bindings: riscv: Update DT binding docs to support SiFive FU740 SoC
>   dt-bindings: pwm: Update DT binding docs to support SiFive FU740 SoC
>   dt-bindings: gpio: Update DT binding docs to support SiFive FU740 SoC
>   dt-bindings: riscv: Update l2 cache DT documentation to add support
>     for SiFive FU740
>   RISC-V: sifive_l2_cache: Update L2 cache driver to support SiFive
>     FU740
> 
>  .../devicetree/bindings/gpio/sifive,gpio.yaml |  6 ++--
>  .../devicetree/bindings/i2c/i2c-ocores.txt    | 10 +++----
>  .../bindings/riscv/sifive-l2-cache.yaml       |  7 ++---
>  drivers/soc/sifive/sifive_l2_cache.c          | 28 ++++---------------
>  4 files changed, 16 insertions(+), 35 deletions(-)
> 
I also see config check failures with this series:

check-config: FAIL (n != y): CONFIG_SIFIVE_L2 policy<{'riscv64': 'y'}>
check-config: FAIL (- != y): CONFIG_EDAC_SIFIVE policy<{'riscv64': 'y'}>
check-config: 6953/6955 checks passed -- exit 1
Dimitri John Ledkov April 1, 2021, 11:40 a.m. UTC | #3
On Mon, 29 Mar 2021 at 10:13, Colin Ian King <colin.king@canonical.com> wrote:
>
> On 26/03/2021 20:57, Dimitri John Ledkov wrote:
> > Update old unmatched patches to new revisions.
> >
> > This also fixes l2 cache failure as seen on cold boot.
> >
> > I was unable to test this on neither my board, nor in qemu.
> >
> > BugLink: https://bugs.launchpad.net/bugs/1920916
> >
> > Dimitri John Ledkov (3):
> >   Revert "dt-bindings: riscv: Update DT binding docs to support SiFive
> >     FU740 SoC"
> >   Revert "RISC-V: sifive_l2_cache: Update L2 cache driver to support
> >     SiFive FU740"
> >   Revert "RISC-V: Update l2 cache DT documentation to add support for
> >     SiFive FU740"
> >
> > Yash Shah (5):
> >   dt-bindings: riscv: Update DT binding docs to support SiFive FU740 SoC
> >   dt-bindings: pwm: Update DT binding docs to support SiFive FU740 SoC
> >   dt-bindings: gpio: Update DT binding docs to support SiFive FU740 SoC
> >   dt-bindings: riscv: Update l2 cache DT documentation to add support
> >     for SiFive FU740
> >   RISC-V: sifive_l2_cache: Update L2 cache driver to support SiFive
> >     FU740
> >
> >  .../devicetree/bindings/gpio/sifive,gpio.yaml |  6 ++--
> >  .../devicetree/bindings/i2c/i2c-ocores.txt    | 10 +++----
> >  .../bindings/riscv/sifive-l2-cache.yaml       |  7 ++---
> >  drivers/soc/sifive/sifive_l2_cache.c          | 28 ++++---------------
> >  4 files changed, 16 insertions(+), 35 deletions(-)
> >
> I also see config check failures with this series:
>
> check-config: FAIL (n != y): CONFIG_SIFIVE_L2 policy<{'riscv64': 'y'}>
> check-config: FAIL (- != y): CONFIG_EDAC_SIFIVE policy<{'riscv64': 'y'}>
> check-config: 6953/6955 checks passed -- exit 1
>

Interesting, the debian.riscv has these both set in both the config
and annotations. It was built for riscv64 arch out of the
linux-riscv/+git/hirsute tree right?

I will recheck this before resending these patches.