[PATCH v2 1/1] video: Handle HAS_IOPORT dependencies

2024-05-14 Thread Niklas Schnelle
-by: Arnd Bergmann Signed-off-by: Arnd Bergmann Signed-off-by: Niklas Schnelle --- Note: This patch does not depend any not-yet-mainline HAS_IOPORT changes and may be merged via subsystem specific trees at your earliest convenience. v1 -> v2: - Moved vga_mm_r(), vga_mm_w(), vga_mm_w_fast() ab

[PATCH v2 0/1] video: Handle HAS_IOPORT dependencies

2024-05-14 Thread Niklas Schnelle
ks, Niklas [0] https://lore.kernel.org/all/20230522105049.1467313-1-schne...@linux.ibm.com/ [1] https://git.kernel.org/pub/scm/linux/kernel/git/niks/linux.git/log/?h=has_ioport [2] https://lore.kernel.org/lkml/CAHk-=wg80je=k7madf4e7wrrnp37e3qh6y10svhdc7o8sz_...@mail.gmail.com/ Niklas Schnelle (1

Re: [PATCH 1/1] video: Handle HAS_IOPORT dependencies

2024-04-22 Thread Niklas Schnelle
On Thu, 2024-04-11 at 16:00 +0200, Helge Deller wrote: > * Niklas Schnelle : > > In a future patch HAS_IOPORT=n will disable inb()/outb() and friends at > > compile time. We thus need to #ifdef functions and their callsites which > > unconditionally use these I/O accessors.

[PATCH 0/1] video: Handle HAS_IOPORT dependencies

2024-04-10 Thread Niklas Schnelle
[0] https://lore.kernel.org/all/20230522105049.1467313-1-schne...@linux.ibm.com/ [1] https://git.kernel.org/pub/scm/linux/kernel/git/niks/linux.git/log/?h=has_ioport [2] https://lore.kernel.org/lkml/CAHk-=wg80je=k7madf4e7wrrnp37e3qh6y10svhdc7o8sz_...@mail.gmail.com/ Niklas Schnelle (1): vid

[PATCH 1/1] video: Handle HAS_IOPORT dependencies

2024-04-10 Thread Niklas Schnelle
-by: Arnd Bergmann Signed-off-by: Arnd Bergmann Signed-off-by: Niklas Schnelle --- Note: This patch does not depend any not-yet-mainline HAS_IOPORT changes and may be merged via subsystem specific trees at your earliest convenience. include/video/vga.h | 35 +-- 1

[PATCH v2 0/1] fbdev: Handle HAS_IOPORT dependencies

2024-04-10 Thread Niklas Schnelle
kernel.org/all/20230522105049.1467313-1-schne...@linux.ibm.com/ [1] https://git.kernel.org/pub/scm/linux/kernel/git/niks/linux.git/log/?h=has_ioport [2] https://lore.kernel.org/lkml/CAHk-=wg80je=k7madf4e7wrrnp37e3qh6y10svhdc7o8sz_...@mail.gmail.com/ v1 -> v2: - Add dependency for FB_ARC Nikla

[PATCH v2 1/1] fbdev: add HAS_IOPORT dependencies

2024-04-10 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 --- Note: This patch does not depend any

Re: [PATCH 1/1] fbdev: add HAS_IOPORT dependencies

2024-04-10 Thread Niklas Schnelle
On Wed, 2024-04-10 at 10:27 +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. > > Co-developed-by: Arnd Bergmann > Signed-off

[PATCH 1/1] fbdev: add HAS_IOPORT dependencies

2024-04-10 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 --- Note: This patch does not depend any

[PATCH 0/1] fbdev: Handle HAS_IOPORT dependencies

2024-04-10 Thread Niklas Schnelle
kernel.org/all/20230522105049.1467313-1-schne...@linux.ibm.com/ [1] https://git.kernel.org/pub/scm/linux/kernel/git/niks/linux.git/log/?h=has_ioport [2] https://lore.kernel.org/lkml/CAHk-=wg80je=k7madf4e7wrrnp37e3qh6y10svhdc7o8sz_...@mail.gmail.com/ Niklas Schnelle (1): fbdev: add HAS_IOPORT de

Re: [PATCH 1/1] vgacon: add HAS_IOPORT dependencies

2024-04-09 Thread Niklas Schnelle
On Fri, 2024-04-05 at 17:47 +0200, Arnd Bergmann wrote: > On Fri, Apr 5, 2024, at 17:43, 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 us

[PATCH 0/1] vgacon: Handle HAS_IOPORT dependencies

2024-04-05 Thread Niklas Schnelle
[0] https://lore.kernel.org/all/20230522105049.1467313-1-schne...@linux.ibm.com/ [1] https://git.kernel.org/pub/scm/linux/kernel/git/niks/linux.git/log/?h=has_ioport [2] https://lore.kernel.org/lkml/CAHk-=wg80je=k7madf4e7wrrnp37e3qh6y10svhdc7o8sz_...@mail.gmail.com/ Niklas Schnelle (1): vgacon: add

[PATCH 1/1] vgacon: add HAS_IOPORT dependencies

2024-04-05 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 --- Note: This patch does not depend any

[PATCH v5 41/44] video: Handle HAS_IOPORT dependencies

2023-05-22 Thread Niklas Schnelle
-developed-by: Arnd Bergmann Signed-off-by: Arnd Bergmann Signed-off-by: Niklas Schnelle --- drivers/video/fbdev/Kconfig | 3 ++- include/video/vga.h | 35 +-- 2 files changed, 27 insertions(+), 11 deletions(-) diff --git a/drivers/video/fbdev/Kconfig b/drivers

[PATCH v5 39/44] vgacon: add HAS_IOPORT dependencies

2023-05-22 Thread Niklas Schnelle
In a future patch HAS_IOPORT=n will result in inb()/outb() and friends not being declared. 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/video/console/Kconfig

[PATCH v5 40/44] fbdev: add HAS_IOPORT dependencies

2023-05-22 Thread Niklas Schnelle
In a future patch HAS_IOPORT=n will result in inb()/outb() and friends not being declared. 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/video/fbdev/Kconfig | 23

[PATCH v5 09/44] drm: handle HAS_IOPORT dependencies

2023-05-22 Thread Niklas Schnelle
Signed-off-by: Niklas Schnelle --- drivers/gpu/drm/qxl/Kconfig | 1 + drivers/gpu/drm/tiny/bochs.c | 17 + drivers/gpu/drm/tiny/cirrus.c | 2 ++ 3 files changed, 20 insertions(+) diff --git a/drivers/gpu/drm/qxl/Kconfig b/drivers/gpu/drm/qxl/Kconfig index ca3f51c2a8fe

Re: [PATCH v4 38/41] video: handle HAS_IOPORT dependencies

2023-05-17 Thread Niklas Schnelle
On Tue, 2023-05-16 at 19:21 +0200, Thomas Zimmermann wrote: > Hi > > Am 16.05.23 um 13:00 schrieb Niklas Schnelle: > > In a future patch HAS_IOPORT=n will result in inb()/outb() and friends > > not being declared. We thus need to add HAS_IOPORT as dependency for > &g

[PATCH v4 38/41] video: handle HAS_IOPORT dependencies

2023-05-16 Thread Niklas Schnelle
In a future patch HAS_IOPORT=n will result in inb()/outb() and friends not being declared. We thus need to add HAS_IOPORT as dependency for those drivers using them and guard inline code in headers. Co-developed-by: Arnd Bergmann Signed-off-by: Arnd Bergmann Signed-off-by: Niklas Schnelle

[PATCH v4 07/41] drm: handle HAS_IOPORT dependencies

2023-05-16 Thread Niklas Schnelle
Signed-off-by: Niklas Schnelle --- Note: The HAS_IOPORT Kconfig option was added in v6.4-rc1 so per-subsystem patches may be applied independently drivers/gpu/drm/qxl/Kconfig | 1 + drivers/gpu/drm/tiny/bochs.c | 17 + drivers/gpu/drm/tiny/cirrus.c | 2 ++ 3 files changed

[PATCH v4 37/41] fbdev: atyfb: Remove unused clock determination

2023-05-16 Thread Niklas Schnelle
org/all/zbx5alo5h546b...@intel.com/ Suggested-by: Ville Syrjälä Signed-off-by: Niklas Schnelle --- Note: The HAS_IOPORT Kconfig option was added in v6.4-rc1 so per-subsystem patches may be applied independently drivers/video/fbdev/aty/atyfb_base.c | 5 - 1 file changed, 5 deletions(-) d

Re: [PATCH v3 35/38] video: handle HAS_IOPORT dependencies

2023-05-08 Thread Niklas Schnelle
On Thu, 2023-03-23 at 18:08 +0200, Ville Syrjälä wrote: > On Thu, Mar 23, 2023 at 03:17:38PM +0100, Niklas Schnelle wrote: > > On Wed, 2023-03-15 at 12:19 +0200, Ville Syrjälä wrote: > > > On Wed, Mar 15, 2023 at 09:16:50AM +0100, Geert Uytterhoeven wrote:

Re: [PATCH v3 07/38] drm: handle HAS_IOPORT dependencies

2023-03-23 Thread Niklas Schnelle
On Wed, 2023-03-15 at 13:54 +0200, Jani Nikula wrote: > On Tue, 14 Mar 2023, Niklas Schnelle wrote: > > In a future patch HAS_IOPORT=n will result in inb()/outb() and friends > > not being declared. We thus need to add HAS_IOPORT as dependency for > > those drivers using them

Re: [PATCH v3 35/38] video: handle HAS_IOPORT dependencies

2023-03-23 Thread Niklas Schnelle
On Wed, 2023-03-15 at 12:19 +0200, Ville Syrjälä wrote: > On Wed, Mar 15, 2023 at 09:16:50AM +0100, Geert Uytterhoeven wrote: > > Hi Niklas, > > > > On Tue, Mar 14, 2023 at 1:13 PM Niklas Schnelle > > wrote: > > > In a future patch HAS_IOPORT=n wil

[PATCH v3 35/38] video: handle HAS_IOPORT dependencies

2023-03-14 Thread Niklas Schnelle
In a future patch HAS_IOPORT=n will result in inb()/outb() and friends not being declared. We thus need to add HAS_IOPORT as dependency for those drivers using them and guard inline code in headers. Co-developed-by: Arnd Bergmann Signed-off-by: Niklas Schnelle --- drivers/video/console/Kconfig

[PATCH v3 07/38] drm: handle HAS_IOPORT dependencies

2023-03-14 Thread Niklas Schnelle
. There is also a direct and hard coded use in cirrus.c which according to the comment is only necessary during resume. Let's just skip this as for example s390 which doesn't have I/O port support also doesen't support suspend/resume. Co-developed-by: Arnd Bergmann Signed-off-by: Niklas Schnelle

Re: [PATCH 8/8] iommu/s390: Push the gfp parameter to the kmem_cache_alloc()'s

2023-01-17 Thread Niklas Schnelle
letions(-) > ---8<--- > Looks good to me and I have no objections. Reviewed-by: Niklas Schnelle

[PATCH 31/37] drm: handle HAS_IOPORT dependencies

2022-05-02 Thread Niklas Schnelle
skip this as for example s390 which doesn't have I/O port support also doesen't support suspend/resume. Co-developed-by: Arnd Bergmann Signed-off-by: Niklas Schnelle --- drivers/gpu/drm/qxl/Kconfig | 1 + drivers/gpu/drm/tiny/Kconfig | 1 + drivers/gpu/drm/tiny/cirrus.c | 2 ++ 3 files changed

[RFC v2 36/39] video: handle HAS_IOPORT dependencies

2022-05-02 Thread Niklas Schnelle
In a future patch HAS_IOPORT=n will result in inb()/outb() and friends not being declared. We thus need to add HAS_IOPORT as dependency for those drivers using them and guard inline code in headers. Co-developed-by: Arnd Bergmann Signed-off-by: Niklas Schnelle --- drivers/video/fbdev/Kconfig

Re: [PATCH 25/37] video: handle HAS_IOPORT dependencies

2022-05-02 Thread Niklas Schnelle
On Fri, 2022-04-29 at 15:50 +0200, Niklas Schnelle wrote: > In a future patch HAS_IOPORT=n will result in inb()/outb() and friends > not being declared. We thus need to add HAS_IOPORT as dependency for > those drivers using them and guard inline code in headers. > > Co-developed-by

Re: [PATCH 31/37] drm: handle HAS_IOPORT dependencies

2022-05-02 Thread Niklas Schnelle
On Fri, 2022-04-29 at 15:50 +0200, Niklas Schnelle wrote: > In a future patch HAS_IOPORT=n will result in inb()/outb() and friends > not being declared. We thus need to add HAS_IOPORT as dependency for > those drivers using them. There is also a direct and hard coded use in > ci

[RFC v2 08/39] drm: handle HAS_IOPORT dependencies

2022-05-02 Thread Niklas Schnelle
. There is also a direct and hard coded use in cirrus.c which according to the comment is only necessary during resume. Let's just skip this as for example s390 which doesn't have I/O port support also doesen't support suspend/resume. Co-developed-by: Arnd Bergmann Signed-off-by: Niklas Schnelle

[PATCH 25/37] video: handle HAS_IOPORT dependencies

2022-05-02 Thread Niklas Schnelle
In a future patch HAS_IOPORT=n will result in inb()/outb() and friends not being declared. We thus need to add HAS_IOPORT as dependency for those drivers using them and guard inline code in headers. Co-developed-by: Arnd Bergmann Signed-off-by: Niklas Schnelle --- drivers/video/fbdev/Kconfig

Re: [RFC 01/32] Kconfig: introduce and depend on LEGACY_PCI

2022-01-12 Thread Niklas Schnelle
On Thu, 2022-01-06 at 17:41 +, John Garry wrote: > On 05/01/2022 19:47, Bjorn Helgaas wrote: > > > > > > ok if the PCI maintainers decide otherwise. > > > > > I don't really like the "LEGACY_PCI" Kconfig option. "Legacy" just > > > > > means something old and out of favor; it doesn't

Re: [RFC 01/32] Kconfig: introduce and depend on LEGACY_PCI

2021-12-30 Thread Niklas Schnelle
On Wed, 2021-12-29 at 10:03 -0600, Bjorn Helgaas wrote: > On Wed, Dec 29, 2021 at 01:12:07PM +0100, Mauro Carvalho Chehab wrote: > > Em Wed, 29 Dec 2021 12:45:38 +0100 > > Niklas Schnelle escreveu: > > > ... > > > I do think we agree that once done correctly there

[RFC 22/32] video: handle HAS_IOPORT dependencies

2021-12-30 Thread Niklas Schnelle
In a future patch HAS_IOPORT=n will result in inb()/outb() and friends not being declared. We thus need to add HAS_IOPORT as dependency for those drivers using them and guard inline code in headers. Co-developed-by: Arnd Bergmann Signed-off-by: Arnd Bergmann Signed-off-by: Niklas Schnelle

Re: [RFC 01/32] Kconfig: introduce and depend on LEGACY_PCI

2021-12-30 Thread Niklas Schnelle
On Tue, 2021-12-28 at 10:15 +0100, Mauro Carvalho Chehab wrote: > Em Tue, 28 Dec 2021 09:21:23 +0100 > Greg Kroah-Hartman escreveu: > > > On Mon, Dec 27, 2021 at 05:42:46PM +0100, Niklas Schnelle wrote: > > > --- a/drivers/pci/Kconfig > > > +++ b/drivers/

[RFC 26/32] drm: handle HAS_IOPORT dependencies

2021-12-30 Thread Niklas Schnelle
skip this as for example s390 which doesn't have I/O port support also doesen't support suspend/resume. Co-developed-by: Arnd Bergmann Signed-off-by: Arnd Bergmann Signed-off-by: Niklas Schnelle --- drivers/gpu/drm/qxl/Kconfig | 1 + drivers/gpu/drm/tiny/Kconfig | 1 + drivers/gpu/drm/tiny

Re: [RFC 01/32] Kconfig: introduce and depend on LEGACY_PCI

2021-12-30 Thread Niklas Schnelle
On Tue, 2021-12-28 at 18:12 +0100, Mauro Carvalho Chehab wrote: > Em Tue, 28 Dec 2021 16:06:44 +0100 > Niklas Schnelle escreveu: > > (on a side note: the c/c list of this patch is too long. I would try to > avoid using a too long list, as otherwise this e-mail may end being rej

[RFC 01/32] Kconfig: introduce and depend on LEGACY_PCI

2021-12-30 Thread Niklas Schnelle
for all PCI drivers which only target legacy PCI devices and ifdef legacy PCI specific functions in ata handling. Co-developed-by: Arnd Bergmann Signed-off-by: Arnd Bergmann Signed-off-by: Niklas Schnelle --- drivers/ata/Kconfig | 34 drivers/ata

Re: [RFC 01/32] Kconfig: introduce and depend on LEGACY_PCI

2021-12-30 Thread Niklas Schnelle
On Tue, 2021-12-28 at 13:54 +0100, Mauro Carvalho Chehab wrote: > ---8<--- > > > > > All you really care about is the "legacy" I/O spaces here, this isn't > > > > tied to PCI specifically at all, right? > > > > > > > > So why not just have a OLD_STYLE_IO config option or something like > > > >

Re: [PATCH v3 08/16] s390/pci: Remove races against pte updates

2020-10-22 Thread Niklas Schnelle
e > Cc: Dan Williams > Cc: Kees Cook > Cc: Andrew Morton > Cc: John Hubbard > Cc: Jérôme Glisse > Cc: Jan Kara > Cc: linux...@kvack.org > Cc: linux-arm-ker...@lists.infradead.org > Cc: linux-samsung-...@vger.kernel.org > Cc: linux-me...@vger.kernel.

Re: [PATCH v2 08/17] s390/pci: Remove races against pte updates

2020-10-22 Thread Niklas Schnelle
address) fixed and I'll happily apply. Best regards, Niklas Schnelle On 10/12/20 4:19 PM, Daniel Vetter wrote: > On Mon, Oct 12, 2020 at 04:03:28PM +0200, Niklas Schnelle wrote: ... snip >>> Cc: Jason Gunthorpe >>> Cc: Dan Williams >>> Cc: Kees Cook >&

Re: [PATCH v2 08/17] s390/pci: Remove races against pte updates

2020-10-13 Thread Niklas Schnelle
Hi Daniel, freshly back from my vacation I've just taken a look at your patch. First thanks for this fix and the detailed commit description. Definitely makes sense to fix this and you can add my Acked-by: Niklas Schnelle Content wise it all looks sane and clear and since Gerald did

Re: [PATCH v2 08/17] s390/pci: Remove races against pte updates

2020-10-13 Thread Niklas Schnelle
... snip ... >>> Cc: linux-me...@vger.kernel.org >>> Cc: Niklas Schnelle >>> Cc: Gerald Schaefer >>> Cc: linux-s...@vger.kernel.org >>> -- >>> v2: Move VM_IO | VM_PFNMAP checks around so they keep returning EINVAL >>> like before