Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1739626 > unrolled thread

[PATCH v3 00/14] ARM: dts: sunxi: Fix DT build warnings

Back to article view | Back to linux.kernel

Contents

  [PATCH v3 00/14] ARM: dts: sunxi: Fix DT build warnings Corentin Labbe <clabbe.montjoie@gmail.com> - 2017-09-26 09:30 +0200
    Re: [PATCH v3 00/14] ARM: dts: sunxi: Fix DT build warnings Chen-Yu Tsai <wens@csie.org> - 2017-09-26 09:50 +0200
      Re: [PATCH v3 00/14] ARM: dts: sunxi: Fix DT build warnings Maxime Ripard <maxime.ripard@free-electrons.com> - 2017-09-26 11:00 +0200
        Re: [PATCH v3 00/14] ARM: dts: sunxi: Fix DT build warnings Chen-Yu Tsai <wens@csie.org> - 2017-09-26 15:10 +0200
          Re: [PATCH v3 00/14] ARM: dts: sunxi: Fix DT build warnings Chen-Yu Tsai <wens@csie.org> - 2017-09-27 17:20 +0200

#1739626 — [PATCH v3 00/14] ARM: dts: sunxi: Fix DT build warnings

Hello

The goal of this patch series is to have a clean DT build with W=1.

Regards

Changes since v1:
- moved i2c2 unit address change from patch #2 in patch #1

Changes since v2
- added two patch for fixing pin names

Corentin Labbe (14):
  ARM: dts: sunxi: h3/h5: Fix simple-bus unit address format error
  ARM: dts: sunxi: h3/h5: Fix i2c2 register address
  ARM: dts: sunxi: h3/h5: Fix node with unit name and no reg property
  ARM: dts: nanopi: Fix node with unit name and no reg property
  ARM: dts: orangepi2: Fix node with unit name and no reg property
  ARM: dts: sun8i: orangepi-lite: Fix node with unit name and no reg
    property
  ARM: dts: sun8i: orangepi one: Fix node with unit name and no reg
    property
  ARM: dts: sun8i: orangepipc: Fix node with unit name and no reg
    property
  ARM: dts: sun8i: orangepi-plus: Fix node with unit name and no reg
    property
  arm64: allwinner: a64: Fix simple-bus unit address format error
  arm64: allwinner: a64: Fix node with unit name and no reg property
  ARM: dts: sun8i: a83t: Fix simple-bus unit address format error
  arm64: allwinner: a64: fix pin name
  ARM: dts: sun8i: fix pin name

 arch/arm/boot/dts/sun8i-a83t.dtsi                  |  6 +-
 arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts  |  8 +-
 arch/arm/boot/dts/sun8i-h3-bananapi-m2-plus.dts    |  8 +-
 arch/arm/boot/dts/sun8i-h3-beelink-x2.dts          | 10 +--
 arch/arm/boot/dts/sun8i-h3-nanopi-m1-plus.dts      |  4 +-
 arch/arm/boot/dts/sun8i-h3-nanopi-m1.dts           |  2 +-
 arch/arm/boot/dts/sun8i-h3-nanopi-neo-air.dts      |  4 +-
 arch/arm/boot/dts/sun8i-h3-nanopi.dtsi             | 12 +--
 arch/arm/boot/dts/sun8i-h3-orangepi-2.dts          | 16 ++--
 arch/arm/boot/dts/sun8i-h3-orangepi-lite.dts       | 14 ++--
 arch/arm/boot/dts/sun8i-h3-orangepi-one.dts        | 10 +--
 arch/arm/boot/dts/sun8i-h3-orangepi-pc-plus.dts    |  2 +-
 arch/arm/boot/dts/sun8i-h3-orangepi-pc.dts         | 12 +--
 arch/arm/boot/dts/sun8i-h3-orangepi-plus.dts       |  2 +-
 arch/arm/boot/dts/sunxi-h3-h5.dtsi                 | 88 +++++++++++-----------
 .../boot/dts/allwinner/sun50i-a64-bananapi-m64.dts |  2 +-
 .../boot/dts/allwinner/sun50i-a64-nanopi-a64.dts   |  2 +-
 .../boot/dts/allwinner/sun50i-a64-olinuxino.dts    |  2 +-
 .../boot/dts/allwinner/sun50i-a64-orangepi-win.dts |  2 +-
 .../arm64/boot/dts/allwinner/sun50i-a64-pine64.dts |  2 +-
 .../dts/allwinner/sun50i-a64-sopine-baseboard.dts  |  2 +-
 arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi      | 26 +++----
 .../dts/allwinner/sun50i-h5-nanopi-neo-plus2.dts   |  6 +-
 .../boot/dts/allwinner/sun50i-h5-nanopi-neo2.dts   |  4 +-
 .../boot/dts/allwinner/sun50i-h5-orangepi-pc2.dts  |  6 +-
 .../dts/allwinner/sun50i-h5-orangepi-prime.dts     |  8 +-
 .../allwinner/sun50i-h5-orangepi-zero-plus2.dts    |  4 +-
 27 files changed, 132 insertions(+), 132 deletions(-)

-- 
2.13.5

[toc] | [next] | [standalone]

#1739638

On Tue, Sep 26, 2017 at 3:22 PM, Corentin Labbe
<clabbe.montjoie@gmail.com> wrote:
> Hello
>
> The goal of this patch series is to have a clean DT build with W=1.
>
> Regards
>
> Changes since v1:
> - moved i2c2 unit address change from patch #2 in patch #1
>
> Changes since v2
> - added two patch for fixing pin names

The first twelve patches are good. Maxime, I think we can go ahead and
merge them, instead of having them resent again in v3?

The last two patches are not what I asked for though.

ChenYu

[toc] | [prev] | [next] | [standalone]

#1739721

[Multipart message — attachments visible in raw view] — view raw

On Tue, Sep 26, 2017 at 07:48:17AM +0000, Chen-Yu Tsai wrote:
> On Tue, Sep 26, 2017 at 3:22 PM, Corentin Labbe
> <clabbe.montjoie@gmail.com> wrote:
> > Hello
> >
> > The goal of this patch series is to have a clean DT build with W=1.
> >
> > Regards
> >
> > Changes since v1:
> > - moved i2c2 unit address change from patch #2 in patch #1
> >
> > Changes since v2
> > - added two patch for fixing pin names
> 
> The first twelve patches are good. Maxime, I think we can go ahead and
> merge them, instead of having them resent again in v3?

Yep, that works for me. Can you push them?

Thanks!
Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

[toc] | [prev] | [next] | [standalone]

#1739869

On Tue, Sep 26, 2017 at 4:58 PM, Maxime Ripard
<maxime.ripard@free-electrons.com> wrote:
> On Tue, Sep 26, 2017 at 07:48:17AM +0000, Chen-Yu Tsai wrote:
>> On Tue, Sep 26, 2017 at 3:22 PM, Corentin Labbe
>> <clabbe.montjoie@gmail.com> wrote:
>> > Hello
>> >
>> > The goal of this patch series is to have a clean DT build with W=1.
>> >
>> > Regards
>> >
>> > Changes since v1:
>> > - moved i2c2 unit address change from patch #2 in patch #1
>> >
>> > Changes since v2
>> > - added two patch for fixing pin names
>>
>> The first twelve patches are good. Maxime, I think we can go ahead and
>> merge them, instead of having them resent again in v3?
>
> Yep, that works for me. Can you push them?

Patches 1 and 12 didn't apply cleanly. I fixed them.

Patches 4~9 do the same type of cleanup for various h3 board dts files.
Maybe we should just squash them together. Arnd seems to prefer squashing
identical changes to different boards together.


ChenYu

[toc] | [prev] | [next] | [standalone]

#1740818

On Tue, Sep 26, 2017 at 9:01 PM, Chen-Yu Tsai <wens@csie.org> wrote:
> On Tue, Sep 26, 2017 at 4:58 PM, Maxime Ripard
> <maxime.ripard@free-electrons.com> wrote:
>> On Tue, Sep 26, 2017 at 07:48:17AM +0000, Chen-Yu Tsai wrote:
>>> On Tue, Sep 26, 2017 at 3:22 PM, Corentin Labbe
>>> <clabbe.montjoie@gmail.com> wrote:
>>> > Hello
>>> >
>>> > The goal of this patch series is to have a clean DT build with W=1.
>>> >
>>> > Regards
>>> >
>>> > Changes since v1:
>>> > - moved i2c2 unit address change from patch #2 in patch #1
>>> >
>>> > Changes since v2
>>> > - added two patch for fixing pin names
>>>
>>> The first twelve patches are good. Maxime, I think we can go ahead and
>>> merge them, instead of having them resent again in v3?
>>
>> Yep, that works for me. Can you push them?
>
> Patches 1 and 12 didn't apply cleanly. I fixed them.

Patch 10 didn't apply cleanly either. I fixed it.

> Patches 4~9 do the same type of cleanup for various h3 board dts files.
> Maybe we should just squash them together. Arnd seems to prefer squashing
> identical changes to different boards together.

Squashed them and mentioned it in the commit message.

Please check the sunxi tree on git.kernel.org if the results are correct.

ChenYu

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web