Re: [PATCH v2 1/2] iio: imu: inv_icm42600: add WoM support

[Andy Shevchenko]

On Wed, Apr 16, 2025 at 02:40:50PM +0000, Jean-Baptiste Maneyrol wrote:
> OK, understood.
> 
> But for this particular patch, do I need to stay consistent with the existing
> driver by keeping the standard type or use kernel types and mix with standard
> types?

Yes, consistency with the existing code is priority, if the change is a fix.
If it's a feature, better to clean up first (i.e. switch the types elsewhere)
and then add a new feature.

> From: Andy Shevchenko <andy.shevchenko@xxxxxxxxx>
> Sent: Wednesday, April 16, 2025 15:06
> To: Jean-Baptiste Maneyrol <Jean-Baptiste.Maneyrol@xxxxxxx>
> On Wed, Apr 16, 2025 at 3:41 PM Jean-Baptiste Maneyrol
> <Jean-Baptiste.Maneyrol@xxxxxxx> wrote:
> >
> > concerning usage of kernel types, my understanding was that we should
> > conform to existing types usage in a driver. That's why I'm keeping the
> > standard types instead of kernel ones. I could change them in the patch,
> > but it would mix usage of both standard and kernel types.
> >
> > Another thing not related, by reading the coding style documentation I
> > would think that for new driver we can use whatever we prefer between
> > standard types and kernel types. Is it not the case?
> 
> It's not for the repositories Greg KH maintaining directly or indirectly:
> https://urldefense.com/v3/__https://lore.kernel.org/all/20170411140919.GC4388@xxxxxxxxx/__;!!FtrhtPsWDhZ6tw!ACor8Q-WyU0CorzLpw0OtWqNk59Wn2Z_s7KKOZuNnglKBzN4jTqb3c6oea83KLER97bFxkEIgSl8_IcKHTxvaRKCUwVCP5wunvo$[lore[.]kernel[.]org]

-- 
With Best Regards,
Andy Shevchenko