mbox series

[v1,0/4] Convert SiFive drivers from SOC_FOO dependencies to ARCH_FOO

Message ID 20230406-undertake-stowing-50f45b90413a@spud
Headers show
Series Convert SiFive drivers from SOC_FOO dependencies to ARCH_FOO | expand

Message

Conor Dooley April 6, 2023, 8:57 p.m. UTC
From: Conor Dooley <conor.dooley@microchip.com>

RISC-V's SOC_FOO symbols for micro-archs are going away, and being
replaced with the more common ARCH_FOO pattern that is used by other
archs (and by vendors with a history outside of RISC-V).
I kicked the conversion off by converting the Microchip RISC-V bits to
use their replacement symbol, so here's round two: the various SiFive
drivers.

There are no dependencies here, everything can go via subsystem trees.
We've already added the replacement symbols to RISC-V's Kconfig bits.

Cheers,
Conor.

CC: Michael Turquette <mturquette@baylibre.com>
CC: Stephen Boyd <sboyd@kernel.org>
CC: Palmer Dabbelt <palmer@dabbelt.com>
CC: Paul Walmsley <paul.walmsley@sifive.com>
CC: Lorenzo Pieralisi <lpieralisi@kernel.org>
CC: Krzysztof WilczyƄski <kw@linux.com>
CC: Rob Herring <robh@kernel.org>
CC: Bjorn Helgaas <bhelgaas@google.com>
CC: Conor Dooley <conor@kernel.org>
CC: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
CC: Jiri Slaby <jirislaby@kernel.org>
CC: linux-clk@vger.kernel.org
CC: linux-riscv@lists.infradead.org
CC: linux-kernel@vger.kernel.org
CC: linux-pci@vger.kernel.org
CC: linux-serial@vger.kernel.org

Conor Dooley (4):
  clk: sifive: make SiFive clk drivers depend on ARCH_ symbols
  PCI: dwc: make SiFive's PCI driver depend on ARCH_ symbols
  soc: sifive: make SiFive's cache controller driver depend on ARCH_
    symbols
  serial: make SiFive serial drivers depend on ARCH_ symbols

 drivers/clk/sifive/Kconfig         | 6 +++---
 drivers/pci/controller/dwc/Kconfig | 2 +-
 drivers/soc/sifive/Kconfig         | 2 +-
 drivers/tty/serial/Kconfig         | 4 ++--
 4 files changed, 7 insertions(+), 7 deletions(-)

Comments

'Christoph Hellwig' April 8, 2023, 5:47 a.m. UTC | #1
On Thu, Apr 06, 2023 at 09:57:46PM +0100, Conor Dooley wrote:
> From: Conor Dooley <conor.dooley@microchip.com>
> 
> RISC-V's SOC_FOO symbols for micro-archs are going away, and being
> replaced with the more common ARCH_FOO pattern that is used by other
> archs (and by vendors with a history outside of RISC-V).

And this is still the wrong way around.  This ARCH_ madness must not
spread.  NAK.
Conor Dooley April 9, 2023, 8:58 a.m. UTC | #2
On 8 April 2023 06:47:59 IST, Christoph Hellwig <hch@infradead.org> wrote:
>On Thu, Apr 06, 2023 at 09:57:46PM +0100, Conor Dooley wrote:
>> From: Conor Dooley <conor.dooley@microchip.com>
>> 
>> RISC-V's SOC_FOO symbols for micro-archs are going away, and being
>> replaced with the more common ARCH_FOO pattern that is used by other
>> archs (and by vendors with a history outside of RISC-V).
>
>And this is still the wrong way around.  This ARCH_ madness must not
>spread.  NAK.

As I said when you complained last, we asked the arm lads and there was no interest there in doing things differently & that's just one architecture.
Nor was there in adding a SOC_ symbol from some of the people who do more than one ISA.

You're welcome to send reverts for my first round of conversions and patches converting all the ARCH_ stuff RISC-V has to SOC_.
I'll happily ack a complete conversion to SOC_.
I don't care if it is arch or soc, I just want things to be used consistently.