| Message ID | 20241214085833.8695-3-wsa+renesas@sang-engineering.com (mailing list archive) |
|---|---|
| State | New |
| Delegated to: | Geert Uytterhoeven |
| Headers | show |
| Series | add generic parity calculation for u8 | expand |
On Sat, Dec 14, 2024 at 09:58:32AM +0100, Wolfram Sang wrote: > Make use of the new generic helper for calculating the parity. > > Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com> Tested-by: Guenter Roeck <linux@roeck-us.net> > --- > drivers/hwmon/spd5118.c | 8 +------- > 1 file changed, 1 insertion(+), 7 deletions(-) > > diff --git a/drivers/hwmon/spd5118.c b/drivers/hwmon/spd5118.c > index 6cee48a3e5c3..07ab3a47b00c 100644 > --- a/drivers/hwmon/spd5118.c > +++ b/drivers/hwmon/spd5118.c > @@ -291,12 +291,6 @@ static umode_t spd5118_is_visible(const void *_data, enum hwmon_sensor_types typ > } > } > > -static inline bool spd5118_parity8(u8 w) > -{ > - w ^= w >> 4; > - return (0x6996 >> (w & 0xf)) & 1; > -} > - > /* > * Bank and vendor id are 8-bit fields with seven data bits and odd parity. > * Vendor IDs 0 and 0x7f are invalid. > @@ -304,7 +298,7 @@ static inline bool spd5118_parity8(u8 w) > */ > static bool spd5118_vendor_valid(u8 bank, u8 id) > { > - if (!spd5118_parity8(bank) || !spd5118_parity8(id)) > + if (get_parity8(bank) == 0 || get_parity8(id) == 0) > return false; > > id &= 0x7f; > -- > 2.45.2 >
On Sat, Dec 14, 2024 at 9:58 AM Wolfram Sang <wsa+renesas@sang-engineering.com> wrote: > Make use of the new generic helper for calculating the parity. > > Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> Gr{oetje,eeting}s, Geert
On Sat, Dec 14, 2024 at 09:58:32AM +0100, Wolfram Sang wrote: > Make use of the new generic helper for calculating the parity. > > Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com> Reviewed-by: Kuan-Wei Chiu <visitorckw@gmail.com> Regards, Kuan-Wei
diff --git a/drivers/hwmon/spd5118.c b/drivers/hwmon/spd5118.c index 6cee48a3e5c3..07ab3a47b00c 100644 --- a/drivers/hwmon/spd5118.c +++ b/drivers/hwmon/spd5118.c @@ -291,12 +291,6 @@ static umode_t spd5118_is_visible(const void *_data, enum hwmon_sensor_types typ } } -static inline bool spd5118_parity8(u8 w) -{ - w ^= w >> 4; - return (0x6996 >> (w & 0xf)) & 1; -} - /* * Bank and vendor id are 8-bit fields with seven data bits and odd parity. * Vendor IDs 0 and 0x7f are invalid. @@ -304,7 +298,7 @@ static inline bool spd5118_parity8(u8 w) */ static bool spd5118_vendor_valid(u8 bank, u8 id) { - if (!spd5118_parity8(bank) || !spd5118_parity8(id)) + if (get_parity8(bank) == 0 || get_parity8(id) == 0) return false; id &= 0x7f;
Make use of the new generic helper for calculating the parity. Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com> --- drivers/hwmon/spd5118.c | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-)