RE: [PATCH] iio: chemical: atlas-ezo-sensor: Simplify probe()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Andy Shevchenko,

> Subject: Re: [PATCH] iio: chemical: atlas-ezo-sensor: Simplify probe()
> 
> On Sat, Aug 12, 2023 at 05:34:25PM +0100, Biju Das wrote:
> > Simplify the probe() by replacing device_get_match_data() and ID
> > lookup match by i2c_get_match_data() as we have similar I2C and
> > DT-based matching table.
> 
> > While at it, drop id variable from probe() as it is unused.
> 
> As somebody noticed, this "while at it" part is misleading.
> It's integral part of the change.

Agreed, will drop this and send v2.

Cheers,
Biju

> 
> With that fixed,
> Reviewed-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
> 
> --
> With Best Regards,
> Andy Shevchenko
> 





[Index of Archives]     [Linux Samsung SOC]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux