Lars-Peter Clausen <lars@xxxxxxxxxx> wrote:
>On 01/02/2012 06:25 PM, Jonathan Cameron wrote:
>> Trivial usecase in which just having two different attr
>> groups covers all options. Slightly more code, but a simpler
>> to follow result.
>>
>
>> static int __devinit ad9834_probe(struct spi_device *spi)
>> {
>> struct ad9834_platform_data *pdata = spi->dev.platform_data;
>> @@ -344,7 +347,10 @@ static int __devinit ad9834_probe(struct
>spi_device *spi)
>> st->reg = reg;
>> indio_dev->dev.parent = &spi->dev;
>> indio_dev->name = spi_get_device_id(spi)->name;
>> - indio_dev->info = &ad9834_info;
>> + if ((st->devid == ID_AD9833) || (st->devid == ID_AD9837))
>
>Maybe use a switch statement.
Good idea
>
>> + indio_dev->info = &ad9833_info;
>> + else
>> + indio_dev->info = &ad9834_info;
>> indio_dev->modes = INDIO_DIRECT_MODE;
>>
>> /* Setup default messages */
>
>--
>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
--
Sent from my Android phone with K-9 Mail. Please excuse my brevity.
--
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
