Re: [PATCH v3 2/6] staging: r8188eu: add helper macro for printing registers

Greg KH
2021-08-26T10:38:02Z

On Tue, Aug 24, 2021 at 10:27:20AM +0300, Pavel Skripkin wrote:
> There are a lof of places, where DBG_88E() is used to print register
> value. Since following patches change _rtw_read*() family
> prototypes, we can wrap printing registers into useful macro to avoid
> open-coding error checking like this:
>
> u32 tmp;
> if (!rtw_read(&tmp))
> DBG("reg = %d\n", tmp);
>
> So, added DBG_88E_REG{8,16,32} macros for printing register values.

No, please work to remove these messy things, do not add new ones.

Why is this ever needed? And if it is, use the correct in-kernel debug
functions please.

thanks,

greg k-h