All of lore.kernel.org
 help / color / mirror / Atom feed
* [RESEND PATCH v2 0/9] iomap: Constify ioreadX() iomem argument
@ 2020-02-19 17:49 ` Krzysztof Kozlowski
  0 siblings, 0 replies; 148+ messages in thread
From: Krzysztof Kozlowski @ 2020-02-19 17:49 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, Geert Uytterhoeven, Andrew Morton,
	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
  Cc: Krzysztof Kozlowski

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 acks and reviews,
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 only compile tested on affected architectures.  No real
testing.


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.

1. All patches depend on first patch,
2. Patches 2-4 unify the interface also in few drivers,
3. PAtches 5-9 are optional cleanup, without actual impact.


Best regards,
Krzysztof


Krzysztof Kozlowski (9):
  iomap: Constify ioreadX() iomem argument (as in generic
    implementation)
  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)
  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] 148+ messages in thread

end of thread, other threads:[~2020-03-24  8:42 UTC | newest]

Thread overview: 148+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-02-19 17:49 [RESEND PATCH v2 0/9] iomap: Constify ioreadX() iomem argument Krzysztof Kozlowski
2020-02-19 17:49 ` Krzysztof Kozlowski
2020-02-19 17:49 ` Krzysztof Kozlowski
2020-02-19 17:49 ` Krzysztof Kozlowski
2020-02-19 17:49 ` Krzysztof Kozlowski
2020-02-19 17:49 ` [RESEND PATCH v2 1/9] iomap: Constify ioreadX() iomem argument (as in generic implementation) Krzysztof Kozlowski
2020-02-19 17:49   ` Krzysztof Kozlowski
2020-02-19 17:49   ` Krzysztof Kozlowski
2020-02-19 17:49   ` Krzysztof Kozlowski
     [not found]   ` <20200219175007.13627-2-krzk-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
2020-03-12 10:55     ` Michael Ellerman
2020-03-12 10:55   ` Michael Ellerman
2020-03-12 10:55   ` Michael Ellerman
2020-03-12 10:55   ` Michael Ellerman
2020-03-12 10:55     ` Michael Ellerman
2020-03-12 10:55     ` Michael Ellerman
2020-03-12 10:55     ` Michael Ellerman
2020-03-12 10:55     ` Michael Ellerman
2020-03-12 10:55     ` Michael Ellerman
2020-03-14 11:02   ` Krzysztof Kozlowski
2020-03-14 11:02     ` Krzysztof Kozlowski
2020-03-14 11:02     ` Krzysztof Kozlowski
2020-03-14 11:02     ` Krzysztof Kozlowski
2020-03-14 11:02     ` Krzysztof Kozlowski
2020-03-14 11:02     ` Krzysztof Kozlowski
2020-02-19 17:49 ` Krzysztof Kozlowski
2020-02-19 17:49 ` Krzysztof Kozlowski
2020-02-19 17:49 ` Krzysztof Kozlowski
2020-02-19 17:50 ` [RESEND PATCH v2 2/9] rtl818x: " Krzysztof Kozlowski
2020-02-19 17:50 ` Krzysztof Kozlowski
2020-02-19 17:50   ` Krzysztof Kozlowski
2020-02-19 17:50   ` Krzysztof Kozlowski
2020-02-19 17:50   ` Krzysztof Kozlowski
2020-02-19 17:50 ` Krzysztof Kozlowski
2020-02-19 17:50 ` Krzysztof Kozlowski
2020-02-19 17:50 ` [RESEND PATCH v2 3/9] ntb: intel: " Krzysztof Kozlowski
2020-02-19 17:50 ` Krzysztof Kozlowski
2020-02-19 17:50   ` Krzysztof Kozlowski
2020-02-19 17:50   ` Krzysztof Kozlowski
2020-02-19 17:50   ` Krzysztof Kozlowski
2020-02-19 17:50   ` Krzysztof Kozlowski
2020-02-19 17:50 ` Krzysztof Kozlowski
2020-02-19 17:50 ` [RESEND PATCH v2 4/9] virtio: pci: " Krzysztof Kozlowski
2020-02-19 17:50   ` Krzysztof Kozlowski
2020-02-19 17:50   ` Krzysztof Kozlowski
2020-02-19 17:50   ` Krzysztof Kozlowski
2020-02-19 17:50 ` Krzysztof Kozlowski
2020-02-19 17:50 ` Krzysztof Kozlowski
2020-02-19 17:50 ` Krzysztof Kozlowski
2020-02-19 17:50 ` [RESEND PATCH v2 5/9] arc: " Krzysztof Kozlowski
2020-02-19 17:50 ` Krzysztof Kozlowski
2020-02-19 17:50 ` Krzysztof Kozlowski
2020-02-19 17:50   ` Krzysztof Kozlowski
2020-02-19 17:50   ` Krzysztof Kozlowski
2020-02-19 17:50   ` Krzysztof Kozlowski
2020-02-19 17:50 ` Krzysztof Kozlowski
2020-02-19 17:50 ` [RESEND PATCH v2 6/9] drm/mgag200: " Krzysztof Kozlowski
2020-02-19 17:50 ` Krzysztof Kozlowski
2020-02-19 17:50 ` Krzysztof Kozlowski
2020-02-19 17:50 ` Krzysztof Kozlowski
2020-02-19 17:50   ` Krzysztof Kozlowski
2020-02-19 17:50   ` Krzysztof Kozlowski
2020-02-19 17:50   ` Krzysztof Kozlowski
2020-03-12 10:49   ` [RESEND PATCH v2 6/9] drm/mgag200: Constify ioreadX() iomem argument (as in generic implementati Thomas Zimmermann
2020-03-12 10:49     ` [RESEND PATCH v2 6/9] drm/mgag200: Constify ioreadX() iomem argument (as in generic implementation) Thomas Zimmermann
2020-03-12 10:49     ` Thomas Zimmermann
2020-03-12 10:49     ` Thomas Zimmermann
2020-03-12 10:49     ` Thomas Zimmermann
2020-03-14 10:59     ` [RESEND PATCH v2 6/9] drm/mgag200: Constify ioreadX() iomem argument (as in generic implementati Krzysztof Kozlowski
2020-03-14 10:59       ` [RESEND PATCH v2 6/9] drm/mgag200: Constify ioreadX() iomem argument (as in generic implementation) Krzysztof Kozlowski
2020-03-14 10:59       ` Krzysztof Kozlowski
2020-03-14 10:59       ` Krzysztof Kozlowski
2020-03-14 10:59       ` Krzysztof Kozlowski
2020-03-14 10:59       ` Krzysztof Kozlowski
2020-03-24  8:39       ` [RESEND PATCH v2 6/9] drm/mgag200: Constify ioreadX() iomem argument (as in generic implementati Thomas Zimmermann
2020-03-24  8:39         ` [RESEND PATCH v2 6/9] drm/mgag200: Constify ioreadX() iomem argument (as in generic implementation) Thomas Zimmermann
2020-03-24  8:39         ` Thomas Zimmermann
2020-03-24  8:39         ` Thomas Zimmermann
2020-03-24  8:39         ` Thomas Zimmermann
2020-03-24  8:39         ` Thomas Zimmermann
     [not found]   ` <20200219175007.13627-7-krzk-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
2020-03-12 10:49     ` Thomas Zimmermann
2020-03-12 10:49   ` Thomas Zimmermann
2020-03-12 10:49   ` Thomas Zimmermann
2020-02-19 17:50 ` [RESEND PATCH v2 7/9] drm/nouveau: " Krzysztof Kozlowski
2020-02-19 17:50 ` Krzysztof Kozlowski
2020-02-19 17:50   ` Krzysztof Kozlowski
2020-02-19 17:50   ` Krzysztof Kozlowski
2020-02-19 17:50   ` Krzysztof Kozlowski
2020-02-19 17:50   ` Krzysztof Kozlowski
2020-02-19 17:50 ` Krzysztof Kozlowski
2020-02-19 17:50 ` [RESEND PATCH v2 8/9] media: fsl-viu: " Krzysztof Kozlowski
2020-02-19 17:50 ` [RESEND PATCH v2 8/9] media: fsl-viu: Constify ioreadX() iomem argument (as in generic implementatio Krzysztof Kozlowski
2020-02-19 17:50   ` [RESEND PATCH v2 8/9] media: fsl-viu: Constify ioreadX() iomem argument (as in generic implementation) Krzysztof Kozlowski
2020-02-19 17:50   ` Krzysztof Kozlowski
2020-02-19 17:50   ` Krzysztof Kozlowski
2020-03-12  8:01   ` [RESEND PATCH v2 8/9] media: fsl-viu: Constify ioreadX() iomem argument (as in generic implement Hans Verkuil
2020-03-12  8:01     ` [RESEND PATCH v2 8/9] media: fsl-viu: Constify ioreadX() iomem argument (as in generic implementation) Hans Verkuil
2020-03-12  8:01     ` Hans Verkuil
2020-03-12  8:01     ` Hans Verkuil
2020-03-12  8:01     ` Hans Verkuil
2020-03-12  8:01   ` Hans Verkuil
     [not found]   ` <20200219175007.13627-9-krzk-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
2020-03-12  8:01     ` Hans Verkuil
2020-02-19 17:50 ` Krzysztof Kozlowski
2020-02-19 17:50 ` Krzysztof Kozlowski
2020-02-19 17:50 ` [RESEND PATCH v2 9/9] ath5k: " Krzysztof Kozlowski
2020-02-19 17:50 ` Krzysztof Kozlowski
2020-02-19 17:50   ` Krzysztof Kozlowski
2020-02-19 17:50   ` Krzysztof Kozlowski
2020-02-19 17:50   ` Krzysztof Kozlowski
2020-02-20  9:48   ` Jiri Slaby
2020-02-20  9:48   ` Jiri Slaby
2020-02-20  9:48     ` Jiri Slaby
2020-02-20  9:48     ` Jiri Slaby
2020-02-20  9:48     ` Jiri Slaby
2020-02-20  9:48     ` Jiri Slaby
2020-02-24 12:47     ` Krzysztof Kozlowski
2020-02-24 12:47       ` Krzysztof Kozlowski
2020-02-24 12:47       ` Krzysztof Kozlowski
2020-02-24 12:47       ` Krzysztof Kozlowski
2020-02-24 12:47       ` Krzysztof Kozlowski
2020-02-24 12:47       ` Krzysztof Kozlowski
2020-02-24 12:54       ` Geert Uytterhoeven
2020-02-24 12:54         ` Geert Uytterhoeven
2020-02-24 12:54         ` Geert Uytterhoeven
2020-02-24 12:54         ` Geert Uytterhoeven
2020-02-24 12:54         ` Geert Uytterhoeven
2020-02-24 14:00         ` David Laight
2020-02-24 14:00           ` David Laight
2020-02-24 14:00           ` David Laight
2020-02-24 14:00           ` David Laight
2020-02-24 14:00           ` David Laight
2020-02-24 14:44         ` Krzysztof Kozlowski
2020-02-24 14:44           ` Krzysztof Kozlowski
2020-02-24 14:44           ` Krzysztof Kozlowski
2020-02-24 14:44           ` Krzysztof Kozlowski
2020-02-24 14:44           ` Krzysztof Kozlowski
2020-02-20  9:48   ` Jiri Slaby
     [not found]   ` <20200219175007.13627-10-krzk-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
2020-02-20  9:48     ` Jiri Slaby
2020-02-19 17:50 ` Krzysztof Kozlowski
     [not found] ` <20200219175007.13627-1-krzk-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
2020-02-19 17:49   ` [RESEND PATCH v2 1/9] iomap: " Krzysztof Kozlowski
2020-02-19 17:50   ` [RESEND PATCH v2 2/9] rtl818x: " Krzysztof Kozlowski
2020-02-19 17:50   ` [RESEND PATCH v2 3/9] ntb: intel: " Krzysztof Kozlowski
2020-02-19 17:50   ` [RESEND PATCH v2 4/9] virtio: pci: " Krzysztof Kozlowski
2020-02-19 17:50   ` [RESEND PATCH v2 5/9] arc: " Krzysztof Kozlowski
2020-02-19 17:50   ` [RESEND PATCH v2 6/9] drm/mgag200: " Krzysztof Kozlowski
2020-02-19 17:50   ` [RESEND PATCH v2 7/9] drm/nouveau: " Krzysztof Kozlowski
2020-02-19 17:50   ` [RESEND PATCH v2 8/9] media: fsl-viu: " Krzysztof Kozlowski
2020-02-19 17:50   ` [RESEND PATCH v2 9/9] ath5k: " Krzysztof Kozlowski
2020-02-19 17:50 ` Krzysztof Kozlowski

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.