mbox series

[0/2] Fix Arm Ltd board node name collisions

Message ID 20211109164650.2233507-1-robh@kernel.org (mailing list archive)
Headers show
Series Fix Arm Ltd board node name collisions | expand

Message

Rob Herring (Arm) Nov. 9, 2021, 4:46 p.m. UTC
The node name changes in commits 25b892b583cc ("ARM: dts: arm: Update 
register-bit-led nodes 'reg' and node names") and 2d3de197a818 ("ARM: 
dts: arm: Update ICST clock nodes 'reg' and node names") caused name 
collisions with the kernel's handling of ICST clock names and platform 
device names. This series fixes the kernel side to handle the DT 
changes.

This does break DT ABI compatibility which is partially mitigated if 
these changes are backported to stable. The alternative is reverting the 
referenced DT changes and coming up with different node names without 
unit-addresses. Ultimately, ABI issues are platform maintainers' 
decision, not mine.

Rob


Rob Herring (2):
  of: Support using 'mask' in making device bus id
  clk: versatile: clk-icst: Ensure clock names are unique

 drivers/clk/versatile/clk-icst.c |  6 ++++--
 drivers/of/platform.c            | 10 ++++++++--
 2 files changed, 12 insertions(+), 4 deletions(-)

Comments

Linus Walleij Nov. 9, 2021, 10:22 p.m. UTC | #1
On Tue, Nov 9, 2021 at 5:46 PM Rob Herring <robh@kernel.org> wrote:

> This does break DT ABI compatibility which is partially mitigated if
> these changes are backported to stable. The alternative is reverting the
> referenced DT changes and coming up with different node names without
> unit-addresses. Ultimately, ABI issues are platform maintainers'
> decision, not mine.

I don't mind breaking the ABI for these boards since everyone I
know rebuild the DT with the kernel and boot. I don't know if
other users flash it though, so check what Sudeep says.

Yours,
Linus Walleij