Re: [PATCH 2/2] misc: Add iop driver for Sunplus SP7021

[Greg KH]

On Wed, Nov 17, 2021 at 02:48:07PM +0800, Tony Huang wrote:
> Add iop driver for Sunplus SP7021
> 
> Signed-off-by: Tony Huang <tony.huang@xxxxxxxxxxx>

Along with everything that Arnd said, one more thing:

> +#define IOP_KDBG_INFO
> +#define IOP_FUNC_DEBUG
> +#define IOP_KDBG_ERR
> +#ifdef IOP_KDBG_INFO
> +	#define FUNC_DEBUG()	pr_info("K_IOP: %s(%d)\n", __func__, __LINE__)
> +#else
> +	#define FUNC_DEBUG()
> +#endif

This should just be deleted entirely, use ftrace if you want to see what
your kernel driver is doing.

No single driver should have special debugging macros like this, use the
kernel-wide system please.

This driver code can get a lot smaller after you make these cleanups.
Also be sure to use scripts/checkpatch.pl to remove the issues that it
points out so we don't have to say the same thing.

thanks,

greg k-h