mbox series

[0/4] Synquacer DT schema fixes

Message ID 20221206-synquacer-dts-v1-0-73eb52e391e9@kernel.org
Headers show
Series Synquacer DT schema fixes | expand

Message

Rob Herring Dec. 6, 2022, 4:16 p.m. UTC
This is a series of DT fixes for the Synquacer. These issues were found 
running the dtschema tools.

I don't have a board, but Ilias has tested the changes for me. Thanks!

Signed-off-by: Rob Herring <robh@kernel.org>

---
Rob Herring (4):
      dts: synquacer: Drop CPU 'arm,armv8' compatibles
      dts: synquacer: Use generic node names
      dts: synquacer: Fix "arm,armv7-timer-mem" node address sizes
      dts: synquacer: Fix idle-states 'entry-method' value

 .../dts/synquacer-sc2a11-developerbox-u-boot.dtsi  |  2 +-
 arch/arm/dts/synquacer-sc2a11-developerbox.dts     |  2 +-
 arch/arm/dts/synquacer-sc2a11.dtsi                 | 70 +++++++++++-----------
 3 files changed, 37 insertions(+), 37 deletions(-)
---
base-commit: bebb393b340295edb9ba50a996fc0510cd1b6ac0
change-id: 20221206-synquacer-dts-521500f88a1d

Best regards,

Comments

Ilias Apalodimas Dec. 7, 2022, 6:57 a.m. UTC | #1
Hi Rob

On Tue, 6 Dec 2022 at 18:16, Rob Herring <robh@kernel.org> wrote:
>
> This is a series of DT fixes for the Synquacer. These issues were found
> running the dtschema tools.
>
> I don't have a board, but Ilias has tested the changes for me. Thanks!

I did and fwiw for the series
Tested-by: Ilias Apalodimas <ilias.apalodimas@linaro.org>
Reviewed-by: Ilias Apalodimas <ilias.apalodimas@linaro.org>

but I'd prefer having someone from Socionext config that.  Kojima-san
does it look ok to you as well?

Regards
/Ilias
>
> Signed-off-by: Rob Herring <robh@kernel.org>
>
> ---
> Rob Herring (4):
>       dts: synquacer: Drop CPU 'arm,armv8' compatibles
>       dts: synquacer: Use generic node names
>       dts: synquacer: Fix "arm,armv7-timer-mem" node address sizes
>       dts: synquacer: Fix idle-states 'entry-method' value
>
>  .../dts/synquacer-sc2a11-developerbox-u-boot.dtsi  |  2 +-
>  arch/arm/dts/synquacer-sc2a11-developerbox.dts     |  2 +-
>  arch/arm/dts/synquacer-sc2a11.dtsi                 | 70 +++++++++++-----------
>  3 files changed, 37 insertions(+), 37 deletions(-)
> ---
> base-commit: bebb393b340295edb9ba50a996fc0510cd1b6ac0
> change-id: 20221206-synquacer-dts-521500f88a1d
>
> Best regards,
> --
> Rob Herring <robh@kernel.org>
Masahisa Kojima Dec. 7, 2022, 12:51 p.m. UTC | #2
Hi Rob, Ilias,

On Wed, 7 Dec 2022 at 15:58, Ilias Apalodimas
<ilias.apalodimas@linaro.org> wrote:
>
> Hi Rob
>
> On Tue, 6 Dec 2022 at 18:16, Rob Herring <robh@kernel.org> wrote:
> >
> > This is a series of DT fixes for the Synquacer. These issues were found
> > running the dtschema tools.
> >
> > I don't have a board, but Ilias has tested the changes for me. Thanks!
>
> I did and fwiw for the series
> Tested-by: Ilias Apalodimas <ilias.apalodimas@linaro.org>
> Reviewed-by: Ilias Apalodimas <ilias.apalodimas@linaro.org>
>
> but I'd prefer having someone from Socionext config that.  Kojima-san
> does it look ok to you as well?

I have checked and tested this series and everything is fine.
Thank you for the modification.

Regards,
Masahisa Kojima

>
> Regards
> /Ilias
> >
> > Signed-off-by: Rob Herring <robh@kernel.org>
> >
> > ---
> > Rob Herring (4):
> >       dts: synquacer: Drop CPU 'arm,armv8' compatibles
> >       dts: synquacer: Use generic node names
> >       dts: synquacer: Fix "arm,armv7-timer-mem" node address sizes
> >       dts: synquacer: Fix idle-states 'entry-method' value
> >
> >  .../dts/synquacer-sc2a11-developerbox-u-boot.dtsi  |  2 +-
> >  arch/arm/dts/synquacer-sc2a11-developerbox.dts     |  2 +-
> >  arch/arm/dts/synquacer-sc2a11.dtsi                 | 70 +++++++++++-----------
> >  3 files changed, 37 insertions(+), 37 deletions(-)
> > ---
> > base-commit: bebb393b340295edb9ba50a996fc0510cd1b6ac0
> > change-id: 20221206-synquacer-dts-521500f88a1d
> >
> > Best regards,
> > --
> > Rob Herring <robh@kernel.org>