On 07.07.2015 01:25, Dmitry Torokhov wrote:
Hi Dirk, On Fri, Jul 03, 2015 at 11:07:49AM +0200, Dirk Behme wrote:diff --git a/include/linux/platform_data/zforce_ts.h b/include/linux/platform_data/zforce_ts.h index 44cd90f..1794268 100644 --- a/include/linux/platform_data/zforce_ts.h +++ b/include/linux/platform_data/zforce_ts.h @@ -15,12 +15,9 @@ #ifndef _LINUX_INPUT_ZFORCE_TS_H #define _LINUX_INPUT_ZFORCE_TS_H -#include <linux/of_gpio.h> - struct zforce_ts_platdata { - int gpio_int; - int gpio_rst; - enum of_gpio_flags reset_active_low; + struct gpio_desc *gpio_int; + struct gpio_desc *gpio_rst; unsigned int x_max; unsigned int y_max;I do not think we should be plumbing GPIO descriptors directly into platform data. If board doe snot want to use DT/ACPI they will simply have to attach GPIO lookup table to the device (via gpiod_add_lookup_table). So I intend to apply the version below...
While it's technically fine, it seems to mix the two parts I've done in separate patches (adding the missing polarity handling and converting to gpiod) in one patch. Which might be less obvious to reviewers and in case of regressions. But anyway, if you like this, I'm fine with it.
Thanks! Best regards Dirk -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
