All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2 0/9] iomap: Constify ioreadX() iomem argument
@ 2020-01-08 20:05 ` Krzysztof Kozlowski
  0 siblings, 0 replies; 110+ messages in thread
From: Krzysztof Kozlowski @ 2020-01-08 20:05 UTC (permalink / raw)
  To: Richard Henderson, Ivan Kokshaysky, Matt Turner, Alexey Brodkin,
	Vineet Gupta, James E.J. Bottomley, Helge Deller,
	Benjamin Herrenschmidt, Paul Mackerras, Michael Ellerman,
	Yoshinori Sato, Rich Felker, Dave Airlie, David Airlie,
	Daniel Vetter, Ben Skeggs, Mauro Carvalho Chehab, Jiri Slaby,
	Nick Kossifidis, Luis Chamberlain, Kalle Valo, David S. Miller,
	Dave Jiang, Jon Mason, Allen Hubbe, Michael S. Tsirkin,
	Jason Wang, Arnd Bergmann, Krzysztof Kozlowski,
	Geert Uytterhoeven, Thomas Gleixner, linux-alpha, linux-kernel,
	linux-snps-arc, linux-parisc, linuxppc-dev, linux-sh, dri-devel,
	nouveau, linux-media, linux-wireless, netdev, linux-ntb,
	virtualization, linux-arch

Hi,


Changes since v1
========
https://lore.kernel.org/lkml/1578415992-24054-1-git-send-email-krzk@kernel.org/
1. Constify also ioreadX_rep() and mmio_insX(),
2. Squash lib+alpha+powerpc+parisc+sh into one patch for bisectability,
3. Add Geert's review,
4. Re-order patches so all optional driver changes are at the end.


Description
=====The ioread8/16/32() and others have inconsistent interface among the
architectures: some taking address as const, some not.

It seems there is nothing really stopping all of them to take
pointer to const.

Patchset was not really tested on all affected architectures.
Build testing is in progress - I hope auto-builders will point any issues.


volatile
====
There is still interface inconsistency between architectures around
"volatile" qualifier:
 - include/asm-generic/io.h:static inline u32 ioread32(const volatile void __iomem *addr)
 - include/asm-generic/iomap.h:extern unsigned int ioread32(const void __iomem *);

This is still discussed and out of scope of this patchset.


Merging
===Multiple architectures are affected in first patch so acks are welcomed.

Patches 2-4 depend on first patch.
The rest is optional cleanup, without actual impact.


Best regards,
Krzysztof


Krzysztof Kozlowski (9):
  iomap: Constify ioreadX() iomem argument (as in generic
    implementation)
  net: wireless: rtl818x: Constify ioreadX() iomem argument (as in
    generic implementation)
  ntb: intel: Constify ioreadX() iomem argument (as in generic
    implementation)
  virtio: pci: Constify ioreadX() iomem argument (as in generic
    implementation)
  arc: Constify ioreadX() iomem argument (as in generic implementation)
  drm/mgag200: Constify ioreadX() iomem argument (as in generic
    implementation)
  drm/nouveau: Constify ioreadX() iomem argument (as in generic
    implementation)
  media: fsl-viu: Constify ioreadX() iomem argument (as in generic
    implementation)
  net: wireless: ath5k: Constify ioreadX() iomem argument (as in generic
    implementation)

 arch/alpha/include/asm/core_apecs.h           |  6 +-
 arch/alpha/include/asm/core_cia.h             |  6 +-
 arch/alpha/include/asm/core_lca.h             |  6 +-
 arch/alpha/include/asm/core_marvel.h          |  4 +-
 arch/alpha/include/asm/core_mcpcia.h          |  6 +-
 arch/alpha/include/asm/core_t2.h              |  2 +-
 arch/alpha/include/asm/io.h                   | 12 ++--
 arch/alpha/include/asm/io_trivial.h           | 16 ++---
 arch/alpha/include/asm/jensen.h               |  2 +-
 arch/alpha/include/asm/machvec.h              |  6 +-
 arch/alpha/kernel/core_marvel.c               |  2 +-
 arch/alpha/kernel/io.c                        | 12 ++--
 arch/arc/plat-axs10x/axs10x.c                 |  4 +-
 arch/parisc/include/asm/io.h                  |  4 +-
 arch/parisc/lib/iomap.c                       | 72 +++++++++----------
 arch/powerpc/kernel/iomap.c                   | 28 ++++----
 arch/sh/kernel/iomap.c                        | 22 +++---
 drivers/gpu/drm/mgag200/mgag200_drv.h         |  4 +-
 drivers/gpu/drm/nouveau/nouveau_bo.c          |  2 +-
 drivers/media/platform/fsl-viu.c              |  2 +-
 drivers/net/wireless/ath/ath5k/ahb.c          | 10 +--
 .../realtek/rtl818x/rtl8180/rtl8180.h         |  6 +-
 drivers/ntb/hw/intel/ntb_hw_gen1.c            |  2 +-
 drivers/ntb/hw/intel/ntb_hw_gen3.h            |  2 +-
 drivers/ntb/hw/intel/ntb_hw_intel.h           |  2 +-
 drivers/virtio/virtio_pci_modern.c            |  6 +-
 include/asm-generic/iomap.h                   | 28 ++++----
 include/linux/io-64-nonatomic-hi-lo.h         |  4 +-
 include/linux/io-64-nonatomic-lo-hi.h         |  4 +-
 lib/iomap.c                                   | 30 ++++----
 30 files changed, 156 insertions(+), 156 deletions(-)

-- 
2.17.1

^ permalink raw reply	[flat|nested] 110+ messages in thread

end of thread, other threads:[~2020-01-27  8:15 UTC | newest]

Thread overview: 110+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-01-08 20:05 [PATCH v2 0/9] iomap: Constify ioreadX() iomem argument Krzysztof Kozlowski
2020-01-08 20:05 ` Krzysztof Kozlowski
2020-01-08 20:05 ` Krzysztof Kozlowski
2020-01-08 20:05 ` Krzysztof Kozlowski
2020-01-08 20:05 ` [PATCH v2 1/9] iomap: Constify ioreadX() iomem argument (as in generic implementation) Krzysztof Kozlowski
2020-01-08 20:05 ` Krzysztof Kozlowski
2020-01-08 20:05   ` Krzysztof Kozlowski
2020-01-08 20:05   ` Krzysztof Kozlowski
2020-01-08 20:05   ` Krzysztof Kozlowski
2020-01-08 20:53   ` Arnd Bergmann
2020-01-08 20:53   ` Arnd Bergmann
2020-01-08 20:53     ` Arnd Bergmann
2020-01-08 20:53     ` Arnd Bergmann
2020-01-08 20:53     ` Arnd Bergmann
2020-01-08 20:53     ` Arnd Bergmann
2020-01-09 12:00   ` Geert Uytterhoeven
2020-01-09 12:00   ` Geert Uytterhoeven
2020-01-09 12:00     ` Geert Uytterhoeven
2020-01-09 12:00     ` Geert Uytterhoeven
2020-01-09 12:00     ` Geert Uytterhoeven
2020-01-09 12:00     ` Geert Uytterhoeven
2020-01-08 20:05 ` [PATCH v2 2/9] net: wireless: rtl818x: Constify ioreadX() iomem argument (as in generic implementati Krzysztof Kozlowski
2020-01-08 20:05   ` [PATCH v2 2/9] net: wireless: rtl818x: Constify ioreadX() iomem argument (as in generic implementation) Krzysztof Kozlowski
2020-01-08 20:05   ` Krzysztof Kozlowski
2020-01-08 20:05   ` Krzysztof Kozlowski
2020-01-10 13:32   ` Kalle Valo
2020-01-10 13:32     ` Kalle Valo
2020-01-10 13:32     ` Kalle Valo
2020-01-10 13:32     ` Kalle Valo
2020-01-26 15:11   ` Kalle Valo
2020-01-26 15:11     ` Kalle Valo
2020-01-26 15:11     ` Kalle Valo
2020-01-26 15:11   ` Kalle Valo
2020-01-26 15:11     ` Kalle Valo
2020-01-26 15:11     ` Kalle Valo
2020-01-26 15:11   ` [PATCH v2 2/9] net: wireless: rtl818x: Constify ioreadX() iomem argument (as in generic implemen Kalle Valo
2020-01-26 15:11     ` [PATCH v2 2/9] net: wireless: rtl818x: Constify ioreadX() iomem argument (as in generic implementation) Kalle Valo
2020-01-26 15:11     ` Kalle Valo
2020-01-26 15:11     ` Kalle Valo
2020-01-26 15:11   ` Kalle Valo
2020-01-26 15:11   ` Kalle Valo
2020-01-08 20:05 ` Krzysztof Kozlowski
     [not found] ` <20200108200528.4614-1-krzk-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
2020-01-08 20:05   ` [PATCH v2 1/9] iomap: " Krzysztof Kozlowski
2020-01-08 20:05   ` [PATCH v2 2/9] net: wireless: rtl818x: " Krzysztof Kozlowski
2020-01-08 20:05   ` [PATCH v2 3/9] ntb: intel: " Krzysztof Kozlowski
2020-01-08 20:05   ` [PATCH v2 4/9] virtio: pci: " Krzysztof Kozlowski
2020-01-08 20:05   ` [PATCH v2 5/9] arc: " Krzysztof Kozlowski
2020-01-08 20:05   ` [PATCH v2 6/9] drm/mgag200: " Krzysztof Kozlowski
2020-01-08 20:05   ` [PATCH v2 7/9] drm/nouveau: " Krzysztof Kozlowski
2020-01-08 20:05   ` [PATCH v2 8/9] media: fsl-viu: " Krzysztof Kozlowski
2020-01-08 20:05   ` [PATCH v2 9/9] net: wireless: ath5k: " Krzysztof Kozlowski
2020-01-08 20:05 ` [PATCH v2 3/9] ntb: intel: " Krzysztof Kozlowski
2020-01-08 20:05   ` Krzysztof Kozlowski
2020-01-08 20:05   ` Krzysztof Kozlowski
2020-01-08 20:05   ` Krzysztof Kozlowski
     [not found]   ` <20200108200528.4614-4-krzk-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
2020-01-08 20:29     ` Dave Jiang
2020-01-08 20:29   ` Dave Jiang
2020-01-08 20:29     ` Dave Jiang
2020-01-08 20:29     ` Dave Jiang
2020-01-08 20:29     ` Dave Jiang
2020-01-08 20:05 ` Krzysztof Kozlowski
2020-01-08 20:05 ` [PATCH v2 4/9] virtio: pci: " Krzysztof Kozlowski
2020-01-08 20:05 ` Krzysztof Kozlowski
2020-01-08 20:05   ` Krzysztof Kozlowski
2020-01-08 20:05   ` Krzysztof Kozlowski
2020-01-08 20:05   ` Krzysztof Kozlowski
2020-01-08 20:05 ` [PATCH v2 5/9] arc: " Krzysztof Kozlowski
2020-01-08 20:05   ` Krzysztof Kozlowski
2020-01-08 20:05   ` Krzysztof Kozlowski
2020-01-08 20:05   ` Krzysztof Kozlowski
2020-01-09  6:59   ` Alexey Brodkin
2020-01-09  6:59     ` Alexey Brodkin
2020-01-09  6:59     ` Alexey Brodkin
2020-01-09  6:59     ` Alexey Brodkin
2020-01-09  6:59     ` Alexey Brodkin
2020-01-08 20:05 ` Krzysztof Kozlowski
2020-01-08 20:05 ` [PATCH v2 6/9] drm/mgag200: " Krzysztof Kozlowski
2020-01-08 20:05   ` Krzysztof Kozlowski
2020-01-08 20:05   ` Krzysztof Kozlowski
2020-01-08 20:05   ` Krzysztof Kozlowski
     [not found]   ` <20200108200528.4614-7-krzk-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
2020-01-09  6:04     ` Thomas Zimmermann
2020-01-09  6:04   ` Thomas Zimmermann
2020-01-09  6:04   ` Thomas Zimmermann
2020-01-09  6:04     ` Thomas Zimmermann
2020-01-09  6:04     ` Thomas Zimmermann
2020-01-09  6:04     ` Thomas Zimmermann
2020-01-09  6:04   ` Thomas Zimmermann
2020-01-08 20:05 ` Krzysztof Kozlowski
2020-01-08 20:05 ` Krzysztof Kozlowski
2020-01-08 20:05 ` [PATCH v2 7/9] drm/nouveau: " Krzysztof Kozlowski
2020-01-08 20:05   ` Krzysztof Kozlowski
2020-01-08 20:05   ` Krzysztof Kozlowski
2020-01-08 20:05   ` Krzysztof Kozlowski
2020-01-08 20:05 ` Krzysztof Kozlowski
2020-01-08 20:05 ` Krzysztof Kozlowski
2020-01-08 20:05 ` [PATCH v2 8/9] media: fsl-viu: " Krzysztof Kozlowski
2020-01-08 20:05 ` Krzysztof Kozlowski
2020-01-08 20:05 ` Krzysztof Kozlowski
2020-01-08 20:05   ` Krzysztof Kozlowski
2020-01-08 20:05   ` Krzysztof Kozlowski
2020-01-08 20:05   ` Krzysztof Kozlowski
2020-01-08 20:05 ` [PATCH v2 9/9] net: wireless: ath5k: " Krzysztof Kozlowski
2020-01-08 20:05 ` [PATCH v2 9/9] net: wireless: ath5k: Constify ioreadX() iomem argument (as in generic implementation Krzysztof Kozlowski
2020-01-08 20:05   ` [PATCH v2 9/9] net: wireless: ath5k: Constify ioreadX() iomem argument (as in generic implementation) Krzysztof Kozlowski
2020-01-08 20:05   ` Krzysztof Kozlowski
2020-01-08 20:05   ` Krzysztof Kozlowski
2020-01-10 13:33   ` Kalle Valo
2020-01-10 13:33     ` Kalle Valo
2020-01-10 13:33     ` Kalle Valo
2020-01-10 13:33     ` Kalle Valo

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.