Re: [PATCH v7 1/9] iio: ssp_sensors: cleanup codestyle warning

From: Sanjay Chitroda

Date: Wed Apr 29 2026 - 14:15:26 EST




On 26 April 2026 7:23:12 pm IST, Jonathan Cameron <jic23@xxxxxxxxxx> wrote:
>On Sun, 26 Apr 2026 14:47:02 +0530
>Sanjay Chitroda <sanjayembeddedse@xxxxxxxxx> wrote:
>
>> From: Sanjay Chitroda <sanjayembeddedse@xxxxxxxxx>
>>
>> Reported by checkpatch:
>> FILE: drivers/iio/common/ssp_sensors/ssp_dev.c
>>
>> WARNING: Prefer __packed over __attribute__((__packed__))
>> +} __attribute__((__packed__));
>>
>> Signed-off-by: Sanjay Chitroda <sanjayembeddedse@xxxxxxxxx>
>Applied to the testing branch of iio.git.
>
>I'm trying to pick up some of the patches on list just
>to reduce how many are getting sent in new revisions and
>the time it takes people to look at them each time.
>
>Not sure how far I'll get in your series so look out for
>when I stop saying I've applied them.
>
>thanks,
>
>Jonathan
>

This was started with single change and later based on valuable input and guidance from you and Andy and during the internal review and development of driver some new change is getting appended and series is getting long even after taking few changes to iio.

Hoping to close this soon, so I can divert my focus in other open parts.

Thanks,
Sanjay Chitroda

>> ---
>> drivers/iio/common/ssp_sensors/ssp_dev.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/iio/common/ssp_sensors/ssp_dev.c b/drivers/iio/common/ssp_sensors/ssp_dev.c
>> index da09c9f3ceb6..7d07fae295fd 100644
>> --- a/drivers/iio/common/ssp_sensors/ssp_dev.c
>> +++ b/drivers/iio/common/ssp_sensors/ssp_dev.c
>> @@ -28,7 +28,7 @@ struct ssp_instruction {
>> __le32 a;
>> __le32 b;
>> u8 c;
>> -} __attribute__((__packed__));
>> +} __packed__;
>>
>> static const u8 ssp_magnitude_table[] = {110, 85, 171, 71, 203, 195, 0, 67,
>> 208, 56, 175, 244, 206, 213, 0, 92, 250, 0, 55, 48, 189, 252, 171,
>