On top of what Peter said. Just some nitpicks...
[...]
+static struct iio_chan_spec z188_adc_iio_channels[] = {

const

+       Z188_ADC_CHANNEL(0),
+       Z188_ADC_CHANNEL(1),
+       Z188_ADC_CHANNEL(2),
+       Z188_ADC_CHANNEL(3),
+       Z188_ADC_CHANNEL(4),
+       Z188_ADC_CHANNEL(5),
+       Z188_ADC_CHANNEL(6),
+       Z188_ADC_CHANNEL(7),
+};
+
[...]
+static struct iio_info z188_adc_info = {

const

+       .read_raw = &z188_iio_read_raw,
+       .driver_module = THIS_MODULE,
+};

> +
> +static void men_z188_remove(struct mcb_device *dev)
> +{
[...]
> +  mcb_set_drvdata(dev, NULL);

This is already done by the device driver core.

> +}
[...]
> +static struct mcb_device_id men_z188_ids[] = {

const

> +  { .device = 0xbc },
> +};
> +MODULE_DEVICE_TABLE(mcb, men_z188_ids);
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to