Re: [PATCH v2 9/9] net: wireless: ath5k: Constify ioreadX() iomem argument (as in generic implementation)

From
Kalle Valo <hidden>
Date
2020-01-12 12:37:29
Also in
dri-devel, linux-alpha, linux-arch, nouveau
Reply to
Krzysztof Kozlowski, 2020-01-08
Krzysztof Kozlowski [off-list ref] writes:
The ioreadX() helpers have inconsistent interface.  On some architectures
void *__iomem address argument is a pointer to const, on some not.

Implementations of ioreadX() do not modify the memory under the address
so they can be converted to a "const" version for const-safety and
consistency among architectures.

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
 drivers/net/wireless/ath/ath5k/ahb.c | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)
No need to have "net: wireless: " in the title, otherwise looks good.

Acked-by: Kalle Valo <redacted>

-- 
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
Thread (18 messages) 18 messages, 7 authors, last reply 2020-01-26
Keyboard shortcuts
hback out one level
jnext message in thread / next series-diff position
kprevious message in thread / previous series-diff position
ldrill in
Escclose help / fold thread tree
?toggle this help