Re: [PATCH v2] wil6210: Replace %pK with %p

[Jeff Johnson <jeff.johnson@oss.qualcomm.com>]

On 9/18/2026 1:11 PM, Sebastian Andrzej Siewior wrote:

commit subject is missing "wifi: " prefix

> Commit 71338aa7d050c ("net: convert %p usage to %pK") which is from
> 2011 and changed the %p annotation for pointer to %pK. Back then the
> default behaviour for %p was to print the pointer. The %pK modifier was
> introduced to able to control the behaviour of specific pointer values
> without changing the behaviour of %p for everyone. It was dedicated to
> avoid leaking pointers via /proc.
> 
> Things changed over time. The default behaviour for %p is now to print a
> hash pointer which does not leak the address but allows to correlate if
> two pointers are equal. I intend to remove %pK from the library.
> 
> Replace %pK with %pK in order to retain the pointer output for

s/with %pK/with %p/

> debugging.
> 
> Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
> ---
> v1…v2: https://lore.kernel.org/all/20260918105632.pDhc9gIB@linutronix.de/
>   - Keep existing %p, just replace %pK.
> 
>  drivers/net/wireless/ath/wil6210/debugfs.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/net/wireless/ath/wil6210/debugfs.c b/drivers/net/wireless/ath/wil6210/debugfs.c
> index e06e45323272c..79f5785697081 100644
> --- a/drivers/net/wireless/ath/wil6210/debugfs.c
> +++ b/drivers/net/wireless/ath/wil6210/debugfs.c
> @@ -200,7 +200,7 @@ static void wil_print_sring(struct seq_file *s, struct wil6210_priv *wil,
>  	seq_printf(s, "Status Ring %s [ %d ] = {\n",
>  		   sring->is_rx ? "RX" : "TX", sring_idx);
>  	seq_printf(s, "  pa     = %pad\n", &sring->pa);
> -	seq_printf(s, "  va     = 0x%pK\n", sring->va);
> +	seq_printf(s, "  va     = 0x%p\n", sring->va);
>  	seq_printf(s, "  size   = %d\n", sring->size);
>  	seq_printf(s, "  elem_size   = %zu\n", sring->elem_size);
>  	seq_printf(s, "  swhead = %d\n", sring->swhead);

No need to repost to address the nits -- I'll fix them in 'pending'