[PATCH 2.6.12-rc4] PCA9539 - new driver

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

 



Hi Ben

> Please fix the driver so it works with -mm- kernels and maybe also the alphabetical issue.

Well (and this goes for your all your other -mm submissions too),
don't just fix it so it works with mm, fix it to take advantage of why
the changes (specifically dynamic sysfs callbacks) are there in the
first place.

> +#define store_data(enum_name)                                                \
> +static ssize_t store_##enum_name(struct device *dev, const char *buf, size_t count) \
and here too.
> +{                                                                    \
> +     pca9539_store_reg(dev, buf, enum_name);                         \
> +     return count;                                                   \
> +}

Things like this are exactly what the change is supposed to rid us of,
please see the previous discussion and example patches in this thread:

http://lkml.org/lkml/2005/5/17/75

Thanks,
Yani




[Index of Archives]     [Linux Kernel]     [Linux Hardware Monitoring]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux