* [PATCH 07/10] input: Enable STMPE keypad driver for Device Tree [not found] <1349451107-8009-1-git-send-email-lee.jones@linaro.org> @ 2012-10-05 15:31 ` Lee Jones 2012-10-10 16:50 ` Dmitry Torokhov 2012-10-05 15:31 ` [PATCH 08/10] Documentation: Describe bindings for STMPE Keypad driver Lee Jones 1 sibling, 1 reply; 4+ messages in thread From: Lee Jones @ 2012-10-05 15:31 UTC (permalink / raw) To: linux-arm-kernel, linux-kernel Cc: arnd, linus.walleij, Lee Jones, Dmitry Torokhov, linux-input This patch allows the STMPE driver to be successfully probed and initialised when Device Tree support is enabled. Besides the usual platform data changes, we also separate the process of filling in the 'in use' pin bitmap, as we have to extract the information from Device Tree in the DT boot case. Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com> Cc: linux-input@vger.kernel.org Acked-by: Linus Walleij <linus.walleij@linaro.org> Signed-off-by: Lee Jones <lee.jones@linaro.org> --- drivers/input/keyboard/stmpe-keypad.c | 67 ++++++++++++++++++++++++++++----- drivers/mfd/stmpe.c | 1 + 2 files changed, 59 insertions(+), 9 deletions(-) diff --git a/drivers/input/keyboard/stmpe-keypad.c b/drivers/input/keyboard/stmpe-keypad.c index 470a877..c722d23 100644 --- a/drivers/input/keyboard/stmpe-keypad.c +++ b/drivers/input/keyboard/stmpe-keypad.c @@ -257,19 +257,73 @@ static int __devinit stmpe_keypad_chip_init(struct stmpe_keypad *keypad) (plat->debounce_ms << 1)); } +static int stmpe_keypad_fill_used_pins(struct platform_device *pdev, + struct stmpe_keypad *keypad, + struct stmpe_keypad_platform_data *plat) +{ + struct device_node *np = pdev->dev.of_node; + unsigned int proplen; + const __be32 *prop; + int i; + + if (np) { + prop = of_get_property(np, "linux,keymap", &proplen); + if (!prop) { + dev_err(&pdev->dev, + "linux,keymap property not defined\n"); + return -EINVAL; + } + + for (i = 0; i < proplen / sizeof(u32); i++) { + unsigned int key = be32_to_cpup(prop + i); + + keypad->cols |= 1 << KEY_COL(key); + keypad->rows |= 1 << KEY_ROW(key); + } + } else { + for (i = 0; i < plat->keymap_data->keymap_size; i++) { + unsigned int key = plat->keymap_data->keymap[i]; + + keypad->cols |= 1 << KEY_COL(key); + keypad->rows |= 1 << KEY_ROW(key); + } + } + + return 0; +} + +static void __devinit stmpe_keypad_of_probe(struct device_node *np, + struct stmpe_keypad_platform_data *plat) +{ + of_property_read_u32(np, "debounce-interval", &plat->debounce_ms); + of_property_read_u32(np, "stericsson,scan-count", &plat->scan_count); + + if (of_get_property(np, "stericsson,no-autorepeat", NULL)) + plat->no_autorepeat = true; +} + static int __devinit stmpe_keypad_probe(struct platform_device *pdev) { struct stmpe *stmpe = dev_get_drvdata(pdev->dev.parent); struct stmpe_keypad_platform_data *plat; + struct device_node *np = pdev->dev.of_node; struct stmpe_keypad *keypad; struct input_dev *input; int ret; int irq; - int i; plat = stmpe->pdata->keypad; - if (!plat) - return -ENODEV; + if (!plat) { + if (np) { + plat = devm_kzalloc(&pdev->dev, + sizeof(*plat), GFP_KERNEL); + if (!plat) + return -ENOMEM; + + stmpe_keypad_of_probe(np, plat); + } else + return -ENODEV; + } irq = platform_get_irq(pdev, 0); if (irq < 0) @@ -300,12 +354,7 @@ static int __devinit stmpe_keypad_probe(struct platform_device *pdev) if (!plat->no_autorepeat) __set_bit(EV_REP, input->evbit); - for (i = 0; i < plat->keymap_data->keymap_size; i++) { - unsigned int key = plat->keymap_data->keymap[i]; - - keypad->cols |= 1 << KEY_COL(key); - keypad->rows |= 1 << KEY_ROW(key); - } + stmpe_keypad_fill_used_pins(pdev, keypad, plat); keypad->stmpe = stmpe; keypad->plat = plat; diff --git a/drivers/mfd/stmpe.c b/drivers/mfd/stmpe.c index ba157d4..b03cc64 100644 --- a/drivers/mfd/stmpe.c +++ b/drivers/mfd/stmpe.c @@ -321,6 +321,7 @@ static struct resource stmpe_keypad_resources[] = { static struct mfd_cell stmpe_keypad_cell = { .name = "stmpe-keypad", + .of_compatible = "st,stmpe-keypad", .resources = stmpe_keypad_resources, .num_resources = ARRAY_SIZE(stmpe_keypad_resources), }; -- 1.7.9.5 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH 07/10] input: Enable STMPE keypad driver for Device Tree 2012-10-05 15:31 ` [PATCH 07/10] input: Enable STMPE keypad driver for Device Tree Lee Jones @ 2012-10-10 16:50 ` Dmitry Torokhov 2012-10-11 8:10 ` Lee Jones 0 siblings, 1 reply; 4+ messages in thread From: Dmitry Torokhov @ 2012-10-10 16:50 UTC (permalink / raw) To: Lee Jones Cc: linux-arm-kernel, linux-kernel, arnd, linus.walleij, linux-input Hi Lee, On Fri, Oct 05, 2012 at 04:31:43PM +0100, Lee Jones wrote: > This patch allows the STMPE driver to be successfully probed and > initialised when Device Tree support is enabled. Besides the usual > platform data changes, we also separate the process of filling in > the 'in use' pin bitmap, as we have to extract the information from > Device Tree in the DT boot case. This generally looks OK although I wonder if we could not unify DT and non-DT case by doing: for (row = 0; row < STMPE_KEYPAD_MAX_ROWS; row++) { if (col = 0; col < STMPE_KEYPAD_MAX_COLS; col++) { int code = MATRIX_SCAN_CODE(row, col, STMPE_KEYPAD_ROW_SHIFT); if (keypad->keymap[code] != KEY_RESERVED) { keypad->rows |= 1 << row; keypad->cols |= 1 << col; } } } BTW, am I supposed to merge it or ack it? Thanks! > > Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com> > Cc: linux-input@vger.kernel.org > Acked-by: Linus Walleij <linus.walleij@linaro.org> > Signed-off-by: Lee Jones <lee.jones@linaro.org> > --- > drivers/input/keyboard/stmpe-keypad.c | 67 ++++++++++++++++++++++++++++----- > drivers/mfd/stmpe.c | 1 + > 2 files changed, 59 insertions(+), 9 deletions(-) > > diff --git a/drivers/input/keyboard/stmpe-keypad.c b/drivers/input/keyboard/stmpe-keypad.c > index 470a877..c722d23 100644 > --- a/drivers/input/keyboard/stmpe-keypad.c > +++ b/drivers/input/keyboard/stmpe-keypad.c > @@ -257,19 +257,73 @@ static int __devinit stmpe_keypad_chip_init(struct stmpe_keypad *keypad) > (plat->debounce_ms << 1)); > } > > +static int stmpe_keypad_fill_used_pins(struct platform_device *pdev, > + struct stmpe_keypad *keypad, > + struct stmpe_keypad_platform_data *plat) > +{ > + struct device_node *np = pdev->dev.of_node; > + unsigned int proplen; > + const __be32 *prop; > + int i; > + > + if (np) { > + prop = of_get_property(np, "linux,keymap", &proplen); > + if (!prop) { > + dev_err(&pdev->dev, > + "linux,keymap property not defined\n"); > + return -EINVAL; > + } > + > + for (i = 0; i < proplen / sizeof(u32); i++) { > + unsigned int key = be32_to_cpup(prop + i); > + > + keypad->cols |= 1 << KEY_COL(key); > + keypad->rows |= 1 << KEY_ROW(key); > + } > + } else { > + for (i = 0; i < plat->keymap_data->keymap_size; i++) { > + unsigned int key = plat->keymap_data->keymap[i]; > + > + keypad->cols |= 1 << KEY_COL(key); > + keypad->rows |= 1 << KEY_ROW(key); > + } > + } > + > + return 0; > +} > + > +static void __devinit stmpe_keypad_of_probe(struct device_node *np, > + struct stmpe_keypad_platform_data *plat) > +{ > + of_property_read_u32(np, "debounce-interval", &plat->debounce_ms); > + of_property_read_u32(np, "stericsson,scan-count", &plat->scan_count); > + > + if (of_get_property(np, "stericsson,no-autorepeat", NULL)) > + plat->no_autorepeat = true; > +} > + > static int __devinit stmpe_keypad_probe(struct platform_device *pdev) > { > struct stmpe *stmpe = dev_get_drvdata(pdev->dev.parent); > struct stmpe_keypad_platform_data *plat; > + struct device_node *np = pdev->dev.of_node; > struct stmpe_keypad *keypad; > struct input_dev *input; > int ret; > int irq; > - int i; > > plat = stmpe->pdata->keypad; > - if (!plat) > - return -ENODEV; > + if (!plat) { > + if (np) { > + plat = devm_kzalloc(&pdev->dev, > + sizeof(*plat), GFP_KERNEL); > + if (!plat) > + return -ENOMEM; > + > + stmpe_keypad_of_probe(np, plat); > + } else > + return -ENODEV; > + } > > irq = platform_get_irq(pdev, 0); > if (irq < 0) > @@ -300,12 +354,7 @@ static int __devinit stmpe_keypad_probe(struct platform_device *pdev) > if (!plat->no_autorepeat) > __set_bit(EV_REP, input->evbit); > > - for (i = 0; i < plat->keymap_data->keymap_size; i++) { > - unsigned int key = plat->keymap_data->keymap[i]; > - > - keypad->cols |= 1 << KEY_COL(key); > - keypad->rows |= 1 << KEY_ROW(key); > - } > + stmpe_keypad_fill_used_pins(pdev, keypad, plat); > > keypad->stmpe = stmpe; > keypad->plat = plat; > diff --git a/drivers/mfd/stmpe.c b/drivers/mfd/stmpe.c > index ba157d4..b03cc64 100644 > --- a/drivers/mfd/stmpe.c > +++ b/drivers/mfd/stmpe.c > @@ -321,6 +321,7 @@ static struct resource stmpe_keypad_resources[] = { > > static struct mfd_cell stmpe_keypad_cell = { > .name = "stmpe-keypad", > + .of_compatible = "st,stmpe-keypad", > .resources = stmpe_keypad_resources, > .num_resources = ARRAY_SIZE(stmpe_keypad_resources), > }; > -- > 1.7.9.5 > -- Dmitry ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 07/10] input: Enable STMPE keypad driver for Device Tree 2012-10-10 16:50 ` Dmitry Torokhov @ 2012-10-11 8:10 ` Lee Jones 0 siblings, 0 replies; 4+ messages in thread From: Lee Jones @ 2012-10-11 8:10 UTC (permalink / raw) To: Dmitry Torokhov Cc: linux-arm-kernel, linux-kernel, arnd, linus.walleij, linux-input On Wed, 10 Oct 2012, Dmitry Torokhov wrote: > Hi Lee, > > On Fri, Oct 05, 2012 at 04:31:43PM +0100, Lee Jones wrote: > > This patch allows the STMPE driver to be successfully probed and > > initialised when Device Tree support is enabled. Besides the usual > > platform data changes, we also separate the process of filling in > > the 'in use' pin bitmap, as we have to extract the information from > > Device Tree in the DT boot case. > > > This generally looks OK although I wonder if we could not unify DT and > non-DT case by doing: > > for (row = 0; row < STMPE_KEYPAD_MAX_ROWS; row++) { > if (col = 0; col < STMPE_KEYPAD_MAX_COLS; col++) { > int code = MATRIX_SCAN_CODE(row, col, > STMPE_KEYPAD_ROW_SHIFT); > if (keypad->keymap[code] != KEY_RESERVED) { > keypad->rows |= 1 << row; > keypad->cols |= 1 << col; > } > } > } Looks like it could work. I have a quite a long TODO list at the moment, but I will add testing this to it. > BTW, am I supposed to merge it or ack it? If you Ack it, I can carry it for you no problem. > > Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com> > > Cc: linux-input@vger.kernel.org > > Acked-by: Linus Walleij <linus.walleij@linaro.org> > > Signed-off-by: Lee Jones <lee.jones@linaro.org> > > --- > > drivers/input/keyboard/stmpe-keypad.c | 67 ++++++++++++++++++++++++++++----- > > drivers/mfd/stmpe.c | 1 + > > 2 files changed, 59 insertions(+), 9 deletions(-) > > > > diff --git a/drivers/input/keyboard/stmpe-keypad.c b/drivers/input/keyboard/stmpe-keypad.c > > index 470a877..c722d23 100644 > > --- a/drivers/input/keyboard/stmpe-keypad.c > > +++ b/drivers/input/keyboard/stmpe-keypad.c > > @@ -257,19 +257,73 @@ static int __devinit stmpe_keypad_chip_init(struct stmpe_keypad *keypad) > > (plat->debounce_ms << 1)); > > } > > > > +static int stmpe_keypad_fill_used_pins(struct platform_device *pdev, > > + struct stmpe_keypad *keypad, > > + struct stmpe_keypad_platform_data *plat) > > +{ > > + struct device_node *np = pdev->dev.of_node; > > + unsigned int proplen; > > + const __be32 *prop; > > + int i; > > + > > + if (np) { > > + prop = of_get_property(np, "linux,keymap", &proplen); > > + if (!prop) { > > + dev_err(&pdev->dev, > > + "linux,keymap property not defined\n"); > > + return -EINVAL; > > + } > > + > > + for (i = 0; i < proplen / sizeof(u32); i++) { > > + unsigned int key = be32_to_cpup(prop + i); > > + > > + keypad->cols |= 1 << KEY_COL(key); > > + keypad->rows |= 1 << KEY_ROW(key); > > + } > > + } else { > > + for (i = 0; i < plat->keymap_data->keymap_size; i++) { > > + unsigned int key = plat->keymap_data->keymap[i]; > > + > > + keypad->cols |= 1 << KEY_COL(key); > > + keypad->rows |= 1 << KEY_ROW(key); > > + } > > + } > > + > > + return 0; > > +} > > + > > +static void __devinit stmpe_keypad_of_probe(struct device_node *np, > > + struct stmpe_keypad_platform_data *plat) > > +{ > > + of_property_read_u32(np, "debounce-interval", &plat->debounce_ms); > > + of_property_read_u32(np, "stericsson,scan-count", &plat->scan_count); > > + > > + if (of_get_property(np, "stericsson,no-autorepeat", NULL)) > > + plat->no_autorepeat = true; > > +} > > + > > static int __devinit stmpe_keypad_probe(struct platform_device *pdev) > > { > > struct stmpe *stmpe = dev_get_drvdata(pdev->dev.parent); > > struct stmpe_keypad_platform_data *plat; > > + struct device_node *np = pdev->dev.of_node; > > struct stmpe_keypad *keypad; > > struct input_dev *input; > > int ret; > > int irq; > > - int i; > > > > plat = stmpe->pdata->keypad; > > - if (!plat) > > - return -ENODEV; > > + if (!plat) { > > + if (np) { > > + plat = devm_kzalloc(&pdev->dev, > > + sizeof(*plat), GFP_KERNEL); > > + if (!plat) > > + return -ENOMEM; > > + > > + stmpe_keypad_of_probe(np, plat); > > + } else > > + return -ENODEV; > > + } > > > > irq = platform_get_irq(pdev, 0); > > if (irq < 0) > > @@ -300,12 +354,7 @@ static int __devinit stmpe_keypad_probe(struct platform_device *pdev) > > if (!plat->no_autorepeat) > > __set_bit(EV_REP, input->evbit); > > > > - for (i = 0; i < plat->keymap_data->keymap_size; i++) { > > - unsigned int key = plat->keymap_data->keymap[i]; > > - > > - keypad->cols |= 1 << KEY_COL(key); > > - keypad->rows |= 1 << KEY_ROW(key); > > - } > > + stmpe_keypad_fill_used_pins(pdev, keypad, plat); > > > > keypad->stmpe = stmpe; > > keypad->plat = plat; > > diff --git a/drivers/mfd/stmpe.c b/drivers/mfd/stmpe.c > > index ba157d4..b03cc64 100644 > > --- a/drivers/mfd/stmpe.c > > +++ b/drivers/mfd/stmpe.c > > @@ -321,6 +321,7 @@ static struct resource stmpe_keypad_resources[] = { > > > > static struct mfd_cell stmpe_keypad_cell = { > > .name = "stmpe-keypad", > > + .of_compatible = "st,stmpe-keypad", > > .resources = stmpe_keypad_resources, > > .num_resources = ARRAY_SIZE(stmpe_keypad_resources), > > }; > > -- > > 1.7.9.5 > > > > -- > Dmitry -- Lee Jones Linaro ST-Ericsson Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH 08/10] Documentation: Describe bindings for STMPE Keypad driver [not found] <1349451107-8009-1-git-send-email-lee.jones@linaro.org> 2012-10-05 15:31 ` [PATCH 07/10] input: Enable STMPE keypad driver for Device Tree Lee Jones @ 2012-10-05 15:31 ` Lee Jones 1 sibling, 0 replies; 4+ messages in thread From: Lee Jones @ 2012-10-05 15:31 UTC (permalink / raw) To: linux-arm-kernel, linux-kernel Cc: arnd, linus.walleij, Lee Jones, Dmitry Torokhov, linux-input Here we add the required documentation for the STMPE Device Tree bindings. It describes all of the bindings currently supported by the driver. Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com> Cc: linux-input@vger.kernel.org Acked-by: Linus Walleij <linus.walleij@linaro.org> Signed-off-by: Lee Jones <lee.jones@linaro.org> --- .../devicetree/bindings/input/stmpe-keypad.txt | 39 ++++++++++++++++++++ 1 file changed, 39 insertions(+) create mode 100644 Documentation/devicetree/bindings/input/stmpe-keypad.txt diff --git a/Documentation/devicetree/bindings/input/stmpe-keypad.txt b/Documentation/devicetree/bindings/input/stmpe-keypad.txt new file mode 100644 index 0000000..e18e2de --- /dev/null +++ b/Documentation/devicetree/bindings/input/stmpe-keypad.txt @@ -0,0 +1,39 @@ +* STMPE Keypad + +Required properties: + - compatible : "st,stmpe-keypad" + - linux,keymap : See ./matrix-keymap.txt + +Optional properties: + - debounce-interval : Debouncing interval time in milliseconds + - stericsson,scan-count : Scanning cycles elapsed before key data is updated + - stericsson,no-autorepeat : If specified device will not autorepeat + +Example: + + stmpe_keypad { + compatible = "st,stmpe-keypad"; + + debounce-interval = <64>; + stericsson,scan-count = <8>; + stericsson,no-autorepeat; + + linux,keymap = <0x205006b + 0x4010074 + 0x3050072 + 0x1030004 + 0x502006a + 0x500000a + 0x5008b + 0x706001c + 0x405000b + 0x6070003 + 0x3040067 + 0x303006c + 0x60400e7 + 0x602009e + 0x4020073 + 0x5050002 + 0x4030069 + 0x3020008>; + }; -- 1.7.9.5 ^ permalink raw reply related [flat|nested] 4+ messages in thread
end of thread, other threads:[~2012-10-11 8:10 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1349451107-8009-1-git-send-email-lee.jones@linaro.org>
2012-10-05 15:31 ` [PATCH 07/10] input: Enable STMPE keypad driver for Device Tree Lee Jones
2012-10-10 16:50 ` Dmitry Torokhov
2012-10-11 8:10 ` Lee Jones
2012-10-05 15:31 ` [PATCH 08/10] Documentation: Describe bindings for STMPE Keypad driver Lee Jones
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).