ARM: mvebu: fixes for v4.17 (#2)

GIT PULL Applied Gregory CLEMENT 3 participants 4 messages May 28, 2018 – Jun 23, 2018

lore

Share & tools

lore

Gregory CLEMENT

author

2018-05-28 15:10 UTC #0 lore raw

Hi,

Here is the second pull request for fixes for mvebu for v4.17. Nothing really critical but it needs to be fixed.

Gregory

The following changes since commit f43194c1447c9536efb0859c2f3f46f6bf2b9154:

ARM64: dts: marvell: armada-cp110: Add mg_core_clk for ethernet node (2018-04-27 17:47:24 +0200)

are available in the Git repository at:

git://git.infradead.org/linux-mvebu.git tags/mvebu-fixes-4.17-2

for you to fetch changes up to ac62cc9d9cd6fa4c79e171c13dc8d58c3862b678:

arm: dts: armada: Fix "#cooling-cells" property's name (2018-05-28 16:54:44 +0200)

----------------------------------------------------------------
mvebu fixes for 4.17 (part 2)

- Use correct size for ICU nodes (irq controller) on Armada 7K/8K
- Fix "#cooling-cells" property's name on Synology DS116 (Armada 385)

----------------------------------------------------------------
Miquel Raynal (1):
arm64: dts: marvell: fix CP110 ICU node size

Viresh Kumar (1):
arm: dts: armada: Fix "#cooling-cells" property's name

 arch/arm/boot/dts/armada-385-synology-ds116.dts | 2 +-
 arch/arm64/boot/dts/marvell/armada-cp110.dtsi   | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

Olof Johansson

2018-06-02 08:24 UTC #1 lore raw

Quoted 25 lines · Gregory CLEMENT

Hi,
Here is the second pull request for fixes for mvebu for v4.17. Nothing really critical but it needs to be fixed.
Gregory
The following changes since commit f43194c1447c9536efb0859c2f3f46f6bf2b9154:
ARM64: dts: marvell: armada-cp110: Add mg_core_clk for ethernet node (2018-04-27 17:47:24 +0200)
are available in the Git repository at:
git://git.infradead.org/linux-mvebu.git tags/mvebu-fixes-4.17-2
for you to fetch changes up to ac62cc9d9cd6fa4c79e171c13dc8d58c3862b678:
arm: dts: armada: Fix "#cooling-cells" property's name (2018-05-28 16:54:44 +0200)
----------------------------------------------------------------
mvebu fixes for 4.17 (part 2)
- Use correct size for ICU nodes (irq controller) on Armada 7K/8K
- Fix "#cooling-cells" property's name on Synology DS116 (Armada 385)

Quoted 6 lines

----------------------------------------------------------------
Miquel Raynal (1):
arm64: dts: marvell: fix CP110 ICU node size
Viresh Kumar (1):
arm: dts: armada: Fix "#cooling-cells" property's name

Subject here should be 'ARM: dts: ...'

The latter is definitely not 4.17 material by now. It's unclear on the ICU patch whether it's causing a real problem in reality, i.e. if it's an urgent regression fix or if it's just fixing up the register range to be correct per documentation. Let me know if it's a real issue and I can either cherry-pick it or send a separate pull request with just that fix.

-Olof

Gregory CLEMENT

2018-06-07 13:57 UTC #2 lore raw

Hi Olof,

Quoted 33 lines · Olof Johansson

Quoted 25 lines · Gregory CLEMENT
Hi,
Here is the second pull request for fixes for mvebu for v4.17. Nothing really critical but it needs to be fixed.
Gregory
The following changes since commit f43194c1447c9536efb0859c2f3f46f6bf2b9154:
ARM64: dts: marvell: armada-cp110: Add mg_core_clk for ethernet node (2018-04-27 17:47:24 +0200)
are available in the Git repository at:
git://git.infradead.org/linux-mvebu.git tags/mvebu-fixes-4.17-2
for you to fetch changes up to ac62cc9d9cd6fa4c79e171c13dc8d58c3862b678:
arm: dts: armada: Fix "#cooling-cells" property's name (2018-05-28 16:54:44 +0200)
----------------------------------------------------------------
mvebu fixes for 4.17 (part 2)
- Use correct size for ICU nodes (irq controller) on Armada 7K/8K
- Fix "#cooling-cells" property's name on Synology DS116 (Armada 385)
Quoted 6 lines
----------------------------------------------------------------
Miquel Raynal (1):
arm64: dts: marvell: fix CP110 ICU node size
Viresh Kumar (1):
arm: dts: armada: Fix "#cooling-cells" property's name

Quoted 7 lines

Subject here should be 'ARM: dts: ...'
The latter is definitely not 4.17 material by now. It's unclear on the ICU patch whether it's causing a real problem in reality, i.e. if it's an urgent regression fix or if it's just fixing up the register range to be correct per documentation. Let me know if it's a real issue and I can either cherry-pick it or send a separate pull request with just that fix.

Even the first patch did not fix a regression. But it is more just than just fixing the documentation as the extension of the size of the register set is needed to support more feature provided by ICU (it is part of series already submitted).

It's not a problem for me that this PR was not merged in 4.17, but could you add it for 4.18 in the current merge window?

Thanks,

Gregory

Quoted 4 lines

-Olof

Signature · 3 lines

Gregory Clement, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
http://bootlin.com

Olof Johansson

Applied

2018-06-23 17:14 UTC #3 lore raw

Quoted 45 lines · Gregory CLEMENT

Hi Olof,
Quoted 33 lines · Olof Johansson
Quoted 25 lines · Gregory CLEMENT
Hi,
Here is the second pull request for fixes for mvebu for v4.17. Nothing really critical but it needs to be fixed.
Gregory
The following changes since commit f43194c1447c9536efb0859c2f3f46f6bf2b9154:
ARM64: dts: marvell: armada-cp110: Add mg_core_clk for ethernet node (2018-04-27 17:47:24 +0200)
are available in the Git repository at:
git://git.infradead.org/linux-mvebu.git tags/mvebu-fixes-4.17-2
for you to fetch changes up to ac62cc9d9cd6fa4c79e171c13dc8d58c3862b678:
arm: dts: armada: Fix "#cooling-cells" property's name (2018-05-28 16:54:44 +0200)
----------------------------------------------------------------
mvebu fixes for 4.17 (part 2)
- Use correct size for ICU nodes (irq controller) on Armada 7K/8K
- Fix "#cooling-cells" property's name on Synology DS116 (Armada 385)
Quoted 6 lines
----------------------------------------------------------------
Miquel Raynal (1):
arm64: dts: marvell: fix CP110 ICU node size
Viresh Kumar (1):
arm: dts: armada: Fix "#cooling-cells" property's name
Quoted 7 lines
Subject here should be 'ARM: dts: ...'
The latter is definitely not 4.17 material by now. It's unclear on the ICU patch whether it's causing a real problem in reality, i.e. if it's an urgent regression fix or if it's just fixing up the register range to be correct per documentation. Let me know if it's a real issue and I can either cherry-pick it or send a separate pull request with just that fix.

Quoted 7 lines

Even the first patch did not fix a regression. But it is more just than just fixing the documentation as the extension of the size of the register set is needed to support more feature provided by ICU (it is part of series already submitted).
It's not a problem for me that this PR was not merged in 4.17, but could you add it for 4.18 in the current merge window?

Merged into fixes now, apologies for the delay.

-Olof