Re: [PATCH v7 3/7] media: platform: amd: Add isp4 fw and hw interface

[Sakari Ailus]

Hi Bin,

On Wed, Jan 07, 2026 at 04:44:11PM +0800, Du, Bin wrote:
> > > +int isp4if_init(struct isp4_interface *ispif, struct device *dev, void __iomem *isp_mmio);
> > 
> > Could you run
> > 
> > 	$ ./scripts/checkpatch.pl --strict --max-line-length=80
> > 
> > on the set, please?
> > 
> 
> Oh, I will change --max-line-length in our checkpatch.pl from 100 to 80 and
> fix all the violations. I used to think that the requirement has been
> relaxed to 100 columns based on
> https://www.phoronix.com/news/Linux-Kernel-Deprecates-80-Col

The default checkpatch.pl warning has been changed but not the coding
style. There are valid reasons why you might want to have longer lines,
still checkpatch.pl not warning about those isn't one. :-)

-- 
Regards,

Sakari Ailus