Re: [PATCH 07/22] staging:iio:ad7793: Remove unused platform_data from device state struct

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

 



On 08/14/2012 09:39 PM, Jonathan Cameron wrote:
> On 08/10/2012 05:36 PM, Lars-Peter Clausen wrote:
>> The platform data for the device is only used from within the drivers probe
>> callback, so there is no need to keep it around in the devices state struct.
>>
>> While we are at it mark the platform data struct as const.
>>
>> Signed-off-by: Lars-Peter Clausen <lars@xxxxxxxxxx>
>> ---
>>  drivers/staging/iio/adc/ad7793.c |   14 ++++++--------
>>  1 file changed, 6 insertions(+), 8 deletions(-)
>>
>> diff --git a/drivers/staging/iio/adc/ad7793.c b/drivers/staging/iio/adc/ad7793.c
>> index 112e2b7..0ba598e 100644
>> --- a/drivers/staging/iio/adc/ad7793.c
>> +++ b/drivers/staging/iio/adc/ad7793.c
>> @@ -44,7 +44,6 @@ struct ad7793_state {
>>  	struct iio_trigger		*trig;
>>  	const struct ad7793_chip_info	*chip_info;
>>  	struct regulator		*reg;
>> -	struct ad7793_platform_data	*pdata;
>>  	wait_queue_head_t		wq_data_avail;
>>  	bool				done;
>>  	bool				irq_dis;
>> @@ -253,7 +252,8 @@ out:
>>  	return ret;
>>  }
>>  
>> -static int ad7793_setup(struct ad7793_state *st)
>> +static int ad7793_setup(struct ad7793_state *st
>> +	const struct ad7793_platform_data *pdata)
> Really odd.  There's a missing comma between the arguments here...
> I'll fixup.

I think this happened during a rebase and I never spotted it because it was
fixed up in a later commit. Reminds me to run a built tests on each individual
patch and not just the whole series together.

Thanks,
- Lars
--
To unsubscribe from this list: send the line "unsubscribe linux-iio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Input]     [Linux Kernel]     [Linux SCSI]     [X.org]

  Powered by Linux