Re: [PATCH v2 1/2] drm/ttm: Change ttm_device_init to use a struct instead of multiple bools

2024-10-09 Thread Thomas Hellström
On Sat, 2024-10-05 at 04:14 +, Matthew Brost wrote: > On Fri, Oct 04, 2024 at 04:28:29PM +0200, Thomas Hellström wrote: > > On Wed, 2024-10-02 at 14:54 +0200, Thomas Hellström wrote: > > > On Wed, 2024-10-02 at 14:45 +0200, Christian König wrote: > > > > Am 02.10.24 um 14:24 schrieb Thomas Hell

[PATCH v7 0/5] treewide: Remove I/O port accessors for HAS_IOPORT=n

2024-10-09 Thread Niklas Schnelle
Hi All, This is a follow up in my long running effort of making inb()/outb() and similar I/O port accessors compile-time optional. After initially sending this as a treewide series with the latest revision at[0] we switched to per subsystem series. Now though as we're left with only 5 patches left

[PATCH v7 2/5] Bluetooth: add HAS_IOPORT dependencies

2024-10-09 Thread Niklas Schnelle
In a future patch HAS_IOPORT=n will disable inb()/outb() and friends at compile time. We thus need to add HAS_IOPORT as dependency for those drivers using them. Co-developed-by: Arnd Bergmann Signed-off-by: Arnd Bergmann Signed-off-by: Niklas Schnelle --- drivers/bluetooth/Kconfig | 6 +++---

[PATCH v8 2/5] Bluetooth: add HAS_IOPORT dependencies

2024-10-09 Thread Niklas Schnelle
In a future patch HAS_IOPORT=n will disable inb()/outb() and friends at compile time. We thus need to add HAS_IOPORT as dependency for those drivers using them. Co-developed-by: Arnd Bergmann Signed-off-by: Arnd Bergmann Signed-off-by: Niklas Schnelle --- drivers/bluetooth/Kconfig | 6 +++---

[PATCH v8 0/5] treewide: Remove I/O port accessors for HAS_IOPORT=n

2024-10-09 Thread Niklas Schnelle
Hi All, This is a follow up in my long running effort of making inb()/outb() and similar I/O port accessors compile-time optional. After initially sending this as a treewide series with the latest revision at[0] we switched to per subsystem series. Now though as we're left with only 5 patches left

[PATCH v8 3/5] drm: handle HAS_IOPORT dependencies

2024-10-09 Thread Niklas Schnelle
In a future patch HAS_IOPORT=n will disable inb()/outb() and friends at compile time. We thus need to add HAS_IOPORT as dependency for those drivers using them. In the bochs driver there is optional MMIO support detected at runtime, warn if this isn't taken when HAS_IOPORT is not defined. There is

[PATCH v7 3/5] drm: handle HAS_IOPORT dependencies

2024-10-09 Thread Niklas Schnelle
In a future patch HAS_IOPORT=n will disable inb()/outb() and friends at compile time. We thus need to add HAS_IOPORT as dependency for those drivers using them. In the bochs driver there is optional MMIO support detected at runtime, warn if this isn't taken when HAS_IOPORT is not defined. There is

[PATCH v7 1/5] hexagon: Don't select GENERIC_IOMAP without HAS_IOPORT support

2024-10-09 Thread Niklas Schnelle
In a future patch HAS_IOPORT=n will disable inb()/outb() and friends at compile time. As hexagon does not support I/O port access it also the GENERIC_IOMAP mechanism of dynamically choosing between I/O port and MMIO access doesn't work so don't select it. Reviewed-by: Brian Cain Co-developed-by:

[PATCH v8 1/5] hexagon: Don't select GENERIC_IOMAP without HAS_IOPORT support

2024-10-09 Thread Niklas Schnelle
In a future patch HAS_IOPORT=n will disable inb()/outb() and friends at compile time. As hexagon does not support I/O port access it also the GENERIC_IOMAP mechanism of dynamically choosing between I/O port and MMIO access doesn't work so don't select it. Reviewed-by: Brian Cain Co-developed-by:

Re: [PATCH v7 4/5] tty: serial: handle HAS_IOPORT dependencies

2024-10-09 Thread Niklas Schnelle
On Tue, 2024-10-08 at 13:32 +0200, Niklas Schnelle wrote: > In a future patch HAS_IOPORT=n will disable inb()/outb() and friends at > compile time. We thus need to add HAS_IOPORT as dependency for those > drivers using them unconditionally. Some 8250 serial drivers support > MMIO only use, so fence

[PATCH v8 5/5] asm-generic/io.h: Remove I/O port accessors for HAS_IOPORT=n

2024-10-09 Thread Niklas Schnelle
With all subsystems and drivers either declaring their dependence on HAS_IOPORT or fencing I/O port specific code sections we can finally make inb()/outb() and friends compile-time dependent on HAS_IOPORT as suggested by Linus in the linked mail. The main benefit of this is that on platforms such a

Re: [PATCH v6 4/5] tty: serial: handle HAS_IOPORT dependencies

2024-10-09 Thread Niklas Schnelle
On Tue, 2024-10-08 at 10:16 +0200, Niklas Schnelle wrote: > On Mon, 2024-10-07 at 22:09 +0100, Maciej W. Rozycki wrote: > > On Mon, 7 Oct 2024, Niklas Schnelle wrote: > > > > > diff --git a/drivers/tty/serial/8250/8250_pci.c > > > b/drivers/tty/serial/8250/8250_pci.c > > > index > > > 6709b6a5f3

[PATCH v7 4/5] tty: serial: handle HAS_IOPORT dependencies

2024-10-09 Thread Niklas Schnelle
In a future patch HAS_IOPORT=n will disable inb()/outb() and friends at compile time. We thus need to add HAS_IOPORT as dependency for those drivers using them unconditionally. Some 8250 serial drivers support MMIO only use, so fence only the parts requiring I/O ports and print an error message if

[PATCH v8 4/5] tty: serial: handle HAS_IOPORT dependencies

2024-10-09 Thread Niklas Schnelle
In a future patch HAS_IOPORT=n will disable inb()/outb() and friends at compile time. We thus need to add HAS_IOPORT as dependency for those drivers using them unconditionally. Some 8250 serial drivers support MMIO only use, so fence only the parts requiring I/O ports and print an error message if

[PATCH v7 5/5] asm-generic/io.h: Remove I/O port accessors for HAS_IOPORT=n

2024-10-09 Thread Niklas Schnelle
With all subsystems and drivers either declaring their dependence on HAS_IOPORT or fencing I/O port specific code sections we can finally make inb()/outb() and friends compile-time dependent on HAS_IOPORT as suggested by Linus in the linked mail. The main benefit of this is that on platforms such a