* [PATCH] touchscreen: stmpe-ts: enable module autoload @ 2015-05-21 21:19 Heiner Kallweit 2015-05-21 23:06 ` Dmitry Torokhov 0 siblings, 1 reply; 7+ messages in thread From: Heiner Kallweit @ 2015-05-21 21:19 UTC (permalink / raw) To: Dmitry Torokhov; +Cc: linux-input Even with poper DT config the module didn't autoload. That's fixed by this additional alias. Tested successfully with a STMPE610-based Adafruit PITFT Plus 3.5" on RPI2 under Arch Linux ARM (kernel 3.18.13). Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com> --- drivers/input/touchscreen/stmpe-ts.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/input/touchscreen/stmpe-ts.c b/drivers/input/touchscreen/stmpe-ts.c index e4c3125..1940e3c 100644 --- a/drivers/input/touchscreen/stmpe-ts.c +++ b/drivers/input/touchscreen/stmpe-ts.c @@ -393,4 +393,5 @@ module_platform_driver(stmpe_ts_driver); MODULE_AUTHOR("Luotao Fu <l.fu@pengutronix.de>"); MODULE_DESCRIPTION("STMPEXXX touchscreen driver"); MODULE_LICENSE("GPL"); +MODULE_ALIAS("spi:" STMPE_TS_NAME); MODULE_ALIAS("platform:" STMPE_TS_NAME); -- 2.4.1 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH] touchscreen: stmpe-ts: enable module autoload 2015-05-21 21:19 [PATCH] touchscreen: stmpe-ts: enable module autoload Heiner Kallweit @ 2015-05-21 23:06 ` Dmitry Torokhov 2015-05-22 5:34 ` Heiner Kallweit 0 siblings, 1 reply; 7+ messages in thread From: Dmitry Torokhov @ 2015-05-21 23:06 UTC (permalink / raw) To: Heiner Kallweit; +Cc: linux-input Hi Heiner, On Thu, May 21, 2015 at 11:19:52PM +0200, Heiner Kallweit wrote: > Even with poper DT config the module didn't autoload. > That's fixed by this additional alias. > > Tested successfully with a STMPE610-based Adafruit PITFT Plus 3.5" > on RPI2 under Arch Linux ARM (kernel 3.18.13). > > Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com> > --- > drivers/input/touchscreen/stmpe-ts.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/input/touchscreen/stmpe-ts.c b/drivers/input/touchscreen/stmpe-ts.c > index e4c3125..1940e3c 100644 > --- a/drivers/input/touchscreen/stmpe-ts.c > +++ b/drivers/input/touchscreen/stmpe-ts.c > @@ -393,4 +393,5 @@ module_platform_driver(stmpe_ts_driver); > MODULE_AUTHOR("Luotao Fu <l.fu@pengutronix.de>"); > MODULE_DESCRIPTION("STMPEXXX touchscreen driver"); > MODULE_LICENSE("GPL"); > +MODULE_ALIAS("spi:" STMPE_TS_NAME); > MODULE_ALIAS("platform:" STMPE_TS_NAME); Bit this is not an SPI driver, it is a platform driver for a MFD cell. Why does adding SPI alias help? Thanks. -- Dmitry ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] touchscreen: stmpe-ts: enable module autoload 2015-05-21 23:06 ` Dmitry Torokhov @ 2015-05-22 5:34 ` Heiner Kallweit 2015-05-22 18:29 ` Dmitry Torokhov 0 siblings, 1 reply; 7+ messages in thread From: Heiner Kallweit @ 2015-05-22 5:34 UTC (permalink / raw) To: Dmitry Torokhov; +Cc: linux-input Am 22.05.2015 um 01:06 schrieb Dmitry Torokhov: > Hi Heiner, > > On Thu, May 21, 2015 at 11:19:52PM +0200, Heiner Kallweit wrote: >> Even with poper DT config the module didn't autoload. >> That's fixed by this additional alias. >> >> Tested successfully with a STMPE610-based Adafruit PITFT Plus 3.5" >> on RPI2 under Arch Linux ARM (kernel 3.18.13). >> >> Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com> >> --- >> drivers/input/touchscreen/stmpe-ts.c | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/drivers/input/touchscreen/stmpe-ts.c b/drivers/input/touchscreen/stmpe-ts.c >> index e4c3125..1940e3c 100644 >> --- a/drivers/input/touchscreen/stmpe-ts.c >> +++ b/drivers/input/touchscreen/stmpe-ts.c >> @@ -393,4 +393,5 @@ module_platform_driver(stmpe_ts_driver); >> MODULE_AUTHOR("Luotao Fu <l.fu@pengutronix.de>"); >> MODULE_DESCRIPTION("STMPEXXX touchscreen driver"); >> MODULE_LICENSE("GPL"); >> +MODULE_ALIAS("spi:" STMPE_TS_NAME); >> MODULE_ALIAS("platform:" STMPE_TS_NAME); > > Bit this is not an SPI driver, it is a platform driver for a MFD cell. > Why does adding SPI alias help? > > Thanks. > The STMPE610 in my case is used on a small touchscreen for the Raspberry Pi and touchscreen controller as well as display controller are connected via SPI + a few GPIOs to the RPI. The display controller is supported by the fbtft driver and the drivers for the respective display controllers have similar aliases. see e.g. drivers/staging/fbtft/fb_hx8353d.c MODULE_ALIAS("spi:hx8353d"); MODULE_ALIAS("platform:hx8353d"); Kernel module stmpe_ts is autoloaded only when it's found at enumerating the devices connected to SPI. Below is the relevant part from the DT overlay. Heiner fragment@2 { target = <&spi0>; __overlay__ { /* needed to avoid dtc warning */ #address-cells = <1>; #size-cells = <0>; pitft: pitft@0{ compatible = "himax,hx8357d"; reg = <0>; pinctrl-names = "default"; pinctrl-0 = <&pitft_pins>; spi-max-frequency = <32000000>; rotate = <90>; fps = <25>; bgr; buswidth = <8>; dc-gpios = <&gpio 25 0>; debug = <0>; }; pitft_ts@1 { #address-cells = <1>; #size-cells = <0>; compatible = "st,stmpe610"; reg = <1>; spi-max-frequency = <500000>; irq-gpio = <&gpio 24 0x2>; /* IRQF_TRIGGER_FALLING */ interrupts = <24 2>; /* high-to-low edge triggered */ interrupt-parent = <&gpio>; interrupt-controller; stmpe_touchscreen { compatible = "st,stmpe-ts"; st,sample-time = <4>; st,mod-12b = <1>; st,ref-sel = <0>; st,adc-freq = <2>; st,ave-ctrl = <3>; st,touch-det-delay = <4>; st,settling = <2>; st,fraction-z = <7>; st,i-drive = <0>; }; stmpe_gpio: stmpe_gpio { #gpio-cells = <2>; compatible = "st,stmpe-gpio"; /* * only GPIO2 is wired/available * and it is wired to the backlight */ st,norequest-mask = <0x7b>; }; }; }; ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] touchscreen: stmpe-ts: enable module autoload 2015-05-22 5:34 ` Heiner Kallweit @ 2015-05-22 18:29 ` Dmitry Torokhov 2015-05-22 19:01 ` Heiner Kallweit 0 siblings, 1 reply; 7+ messages in thread From: Dmitry Torokhov @ 2015-05-22 18:29 UTC (permalink / raw) To: Heiner Kallweit; +Cc: linux-input On Fri, May 22, 2015 at 07:34:43AM +0200, Heiner Kallweit wrote: > Am 22.05.2015 um 01:06 schrieb Dmitry Torokhov: > > Hi Heiner, > > > > On Thu, May 21, 2015 at 11:19:52PM +0200, Heiner Kallweit wrote: > >> Even with poper DT config the module didn't autoload. > >> That's fixed by this additional alias. > >> > >> Tested successfully with a STMPE610-based Adafruit PITFT Plus 3.5" > >> on RPI2 under Arch Linux ARM (kernel 3.18.13). > >> > >> Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com> > >> --- > >> drivers/input/touchscreen/stmpe-ts.c | 1 + > >> 1 file changed, 1 insertion(+) > >> > >> diff --git a/drivers/input/touchscreen/stmpe-ts.c b/drivers/input/touchscreen/stmpe-ts.c > >> index e4c3125..1940e3c 100644 > >> --- a/drivers/input/touchscreen/stmpe-ts.c > >> +++ b/drivers/input/touchscreen/stmpe-ts.c > >> @@ -393,4 +393,5 @@ module_platform_driver(stmpe_ts_driver); > >> MODULE_AUTHOR("Luotao Fu <l.fu@pengutronix.de>"); > >> MODULE_DESCRIPTION("STMPEXXX touchscreen driver"); > >> MODULE_LICENSE("GPL"); > >> +MODULE_ALIAS("spi:" STMPE_TS_NAME); > >> MODULE_ALIAS("platform:" STMPE_TS_NAME); > > > > Bit this is not an SPI driver, it is a platform driver for a MFD cell. > > Why does adding SPI alias help? > > > > Thanks. > > > The STMPE610 in my case is used on a small touchscreen for the Raspberry Pi > and touchscreen controller as well as display controller are connected via > SPI + a few GPIOs to the RPI. > > The display controller is supported by the fbtft driver and the drivers > for the respective display controllers have similar aliases. > see e.g. drivers/staging/fbtft/fb_hx8353d.c > > MODULE_ALIAS("spi:hx8353d"); > MODULE_ALIAS("platform:hx8353d"); I understand why both platform and spi aliases are needed for fb_hx8353d: FBTFT_REGISTER_DRIVER macro does register 2 separate drivers, one SPI and another platform, and so needs both aliases. You still have not explained why we need SPI alias for stmpe-ts which is MFD cell represented by a platform device/driver no matter what bus is used to connect stmpe itself, I2C or SPI. What is the modalias on the stmpe-ts device in sysfs? Thanks. -- Dmitry ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] touchscreen: stmpe-ts: enable module autoload 2015-05-22 18:29 ` Dmitry Torokhov @ 2015-05-22 19:01 ` Heiner Kallweit 2015-05-22 20:17 ` Dmitry Torokhov 0 siblings, 1 reply; 7+ messages in thread From: Heiner Kallweit @ 2015-05-22 19:01 UTC (permalink / raw) To: Dmitry Torokhov; +Cc: linux-input Am 22.05.2015 um 20:29 schrieb Dmitry Torokhov: > On Fri, May 22, 2015 at 07:34:43AM +0200, Heiner Kallweit wrote: >> Am 22.05.2015 um 01:06 schrieb Dmitry Torokhov: >>> Hi Heiner, >>> >>> On Thu, May 21, 2015 at 11:19:52PM +0200, Heiner Kallweit wrote: >>>> Even with poper DT config the module didn't autoload. >>>> That's fixed by this additional alias. >>>> >>>> Tested successfully with a STMPE610-based Adafruit PITFT Plus 3.5" >>>> on RPI2 under Arch Linux ARM (kernel 3.18.13). >>>> >>>> Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com> >>>> --- >>>> drivers/input/touchscreen/stmpe-ts.c | 1 + >>>> 1 file changed, 1 insertion(+) >>>> >>>> diff --git a/drivers/input/touchscreen/stmpe-ts.c b/drivers/input/touchscreen/stmpe-ts.c >>>> index e4c3125..1940e3c 100644 >>>> --- a/drivers/input/touchscreen/stmpe-ts.c >>>> +++ b/drivers/input/touchscreen/stmpe-ts.c >>>> @@ -393,4 +393,5 @@ module_platform_driver(stmpe_ts_driver); >>>> MODULE_AUTHOR("Luotao Fu <l.fu@pengutronix.de>"); >>>> MODULE_DESCRIPTION("STMPEXXX touchscreen driver"); >>>> MODULE_LICENSE("GPL"); >>>> +MODULE_ALIAS("spi:" STMPE_TS_NAME); >>>> MODULE_ALIAS("platform:" STMPE_TS_NAME); >>> >>> Bit this is not an SPI driver, it is a platform driver for a MFD cell. >>> Why does adding SPI alias help? >>> >>> Thanks. >>> >> The STMPE610 in my case is used on a small touchscreen for the Raspberry Pi >> and touchscreen controller as well as display controller are connected via >> SPI + a few GPIOs to the RPI. >> >> The display controller is supported by the fbtft driver and the drivers >> for the respective display controllers have similar aliases. >> see e.g. drivers/staging/fbtft/fb_hx8353d.c >> >> MODULE_ALIAS("spi:hx8353d"); >> MODULE_ALIAS("platform:hx8353d"); > > I understand why both platform and spi aliases are needed for > fb_hx8353d: FBTFT_REGISTER_DRIVER macro does register 2 separate > drivers, one SPI and another platform, and so needs both aliases. You > still have not explained why we need SPI alias for stmpe-ts which is MFD > cell represented by a platform device/driver no matter what bus is used > to connect stmpe itself, I2C or SPI. > > What is the modalias on the stmpe-ts device in sysfs? > > Thanks. > These are the stmpe-related entries under /sys/devices: ./soc/3f204000.spi/spi_master/spi0/spi0.1/stmpe-gpio ./soc/3f204000.spi/spi_master/spi0/spi0.1/stmpe-ts stmpe-ts/modalias is: of:Nstmpe_touchscreenT<NULL>Cst,stmpe-ts I also tried setting the of_match_table element of the driver struct to "st,stmpe-ts". But this didn't work help. Only defining this spi alias enables autoload for whatever reason. Heiner ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] touchscreen: stmpe-ts: enable module autoload 2015-05-22 19:01 ` Heiner Kallweit @ 2015-05-22 20:17 ` Dmitry Torokhov 2015-05-22 21:02 ` Heiner Kallweit 0 siblings, 1 reply; 7+ messages in thread From: Dmitry Torokhov @ 2015-05-22 20:17 UTC (permalink / raw) To: Heiner Kallweit; +Cc: linux-input On Fri, May 22, 2015 at 09:01:50PM +0200, Heiner Kallweit wrote: > Am 22.05.2015 um 20:29 schrieb Dmitry Torokhov: > > On Fri, May 22, 2015 at 07:34:43AM +0200, Heiner Kallweit wrote: > >> Am 22.05.2015 um 01:06 schrieb Dmitry Torokhov: > >>> Hi Heiner, > >>> > >>> On Thu, May 21, 2015 at 11:19:52PM +0200, Heiner Kallweit wrote: > >>>> Even with poper DT config the module didn't autoload. > >>>> That's fixed by this additional alias. > >>>> > >>>> Tested successfully with a STMPE610-based Adafruit PITFT Plus 3.5" > >>>> on RPI2 under Arch Linux ARM (kernel 3.18.13). > >>>> > >>>> Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com> > >>>> --- > >>>> drivers/input/touchscreen/stmpe-ts.c | 1 + > >>>> 1 file changed, 1 insertion(+) > >>>> > >>>> diff --git a/drivers/input/touchscreen/stmpe-ts.c b/drivers/input/touchscreen/stmpe-ts.c > >>>> index e4c3125..1940e3c 100644 > >>>> --- a/drivers/input/touchscreen/stmpe-ts.c > >>>> +++ b/drivers/input/touchscreen/stmpe-ts.c > >>>> @@ -393,4 +393,5 @@ module_platform_driver(stmpe_ts_driver); > >>>> MODULE_AUTHOR("Luotao Fu <l.fu@pengutronix.de>"); > >>>> MODULE_DESCRIPTION("STMPEXXX touchscreen driver"); > >>>> MODULE_LICENSE("GPL"); > >>>> +MODULE_ALIAS("spi:" STMPE_TS_NAME); > >>>> MODULE_ALIAS("platform:" STMPE_TS_NAME); > >>> > >>> Bit this is not an SPI driver, it is a platform driver for a MFD cell. > >>> Why does adding SPI alias help? > >>> > >>> Thanks. > >>> > >> The STMPE610 in my case is used on a small touchscreen for the Raspberry Pi > >> and touchscreen controller as well as display controller are connected via > >> SPI + a few GPIOs to the RPI. > >> > >> The display controller is supported by the fbtft driver and the drivers > >> for the respective display controllers have similar aliases. > >> see e.g. drivers/staging/fbtft/fb_hx8353d.c > >> > >> MODULE_ALIAS("spi:hx8353d"); > >> MODULE_ALIAS("platform:hx8353d"); > > > > I understand why both platform and spi aliases are needed for > > fb_hx8353d: FBTFT_REGISTER_DRIVER macro does register 2 separate > > drivers, one SPI and another platform, and so needs both aliases. You > > still have not explained why we need SPI alias for stmpe-ts which is MFD > > cell represented by a platform device/driver no matter what bus is used > > to connect stmpe itself, I2C or SPI. > > > > What is the modalias on the stmpe-ts device in sysfs? > > > > Thanks. > > > These are the stmpe-related entries under /sys/devices: > > ./soc/3f204000.spi/spi_master/spi0/spi0.1/stmpe-gpio > ./soc/3f204000.spi/spi_master/spi0/spi0.1/stmpe-ts > > stmpe-ts/modalias is: > of:Nstmpe_touchscreenT<NULL>Cst,stmpe-ts OK, that explains it. Does the patch below help? Thanks. -- Dmitry Input: stmpe-ts - fix automatic module loading From: Dmitry Torokhov <dmitry.torokhov@gmail.com> When STMPE is instantiated via device tree individual MFD cells rae formed with OF modaliases, not platform modaliases, and so we need to add OF device table to the driver if we want it to load automatically: of:Nstmpe_touchscreenT<NULL>Cst,stmpe-ts Reported-by: Heiner Kallweit <hkallweit1@gmail.com> Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com> --- drivers/input/touchscreen/stmpe-ts.c | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/drivers/input/touchscreen/stmpe-ts.c b/drivers/input/touchscreen/stmpe-ts.c index e4c3125..e4977c6 100644 --- a/drivers/input/touchscreen/stmpe-ts.c +++ b/drivers/input/touchscreen/stmpe-ts.c @@ -383,13 +383,21 @@ static int stmpe_ts_remove(struct platform_device *pdev) static struct platform_driver stmpe_ts_driver = { .driver = { - .name = STMPE_TS_NAME, - }, + .name = STMPE_TS_NAME, + }, .probe = stmpe_input_probe, .remove = stmpe_ts_remove, }; module_platform_driver(stmpe_ts_driver); +#ifdef CONFIG_OF +static const struct of_device_id stmpe_ts_ids[] = { + { .compatible = "st,stmpe-ts", }, + { }, +}; +MODULE_DEVICE_TABLE(of, stmpe_ts_ids); +#endif + MODULE_AUTHOR("Luotao Fu <l.fu@pengutronix.de>"); MODULE_DESCRIPTION("STMPEXXX touchscreen driver"); MODULE_LICENSE("GPL"); ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH] touchscreen: stmpe-ts: enable module autoload 2015-05-22 20:17 ` Dmitry Torokhov @ 2015-05-22 21:02 ` Heiner Kallweit 0 siblings, 0 replies; 7+ messages in thread From: Heiner Kallweit @ 2015-05-22 21:02 UTC (permalink / raw) To: Dmitry Torokhov; +Cc: linux-input Am 22.05.2015 um 22:17 schrieb Dmitry Torokhov: > On Fri, May 22, 2015 at 09:01:50PM +0200, Heiner Kallweit wrote: >> Am 22.05.2015 um 20:29 schrieb Dmitry Torokhov: >>> On Fri, May 22, 2015 at 07:34:43AM +0200, Heiner Kallweit wrote: >>>> Am 22.05.2015 um 01:06 schrieb Dmitry Torokhov: >>>>> Hi Heiner, >>>>> >>>>> On Thu, May 21, 2015 at 11:19:52PM +0200, Heiner Kallweit wrote: >>>>>> Even with poper DT config the module didn't autoload. >>>>>> That's fixed by this additional alias. >>>>>> >>>>>> Tested successfully with a STMPE610-based Adafruit PITFT Plus 3.5" >>>>>> on RPI2 under Arch Linux ARM (kernel 3.18.13). >>>>>> >>>>>> Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com> >>>>>> --- >>>>>> drivers/input/touchscreen/stmpe-ts.c | 1 + >>>>>> 1 file changed, 1 insertion(+) >>>>>> >>>>>> diff --git a/drivers/input/touchscreen/stmpe-ts.c b/drivers/input/touchscreen/stmpe-ts.c >>>>>> index e4c3125..1940e3c 100644 >>>>>> --- a/drivers/input/touchscreen/stmpe-ts.c >>>>>> +++ b/drivers/input/touchscreen/stmpe-ts.c >>>>>> @@ -393,4 +393,5 @@ module_platform_driver(stmpe_ts_driver); >>>>>> MODULE_AUTHOR("Luotao Fu <l.fu@pengutronix.de>"); >>>>>> MODULE_DESCRIPTION("STMPEXXX touchscreen driver"); >>>>>> MODULE_LICENSE("GPL"); >>>>>> +MODULE_ALIAS("spi:" STMPE_TS_NAME); >>>>>> MODULE_ALIAS("platform:" STMPE_TS_NAME); >>>>> >>>>> Bit this is not an SPI driver, it is a platform driver for a MFD cell. >>>>> Why does adding SPI alias help? >>>>> >>>>> Thanks. >>>>> >>>> The STMPE610 in my case is used on a small touchscreen for the Raspberry Pi >>>> and touchscreen controller as well as display controller are connected via >>>> SPI + a few GPIOs to the RPI. >>>> >>>> The display controller is supported by the fbtft driver and the drivers >>>> for the respective display controllers have similar aliases. >>>> see e.g. drivers/staging/fbtft/fb_hx8353d.c >>>> >>>> MODULE_ALIAS("spi:hx8353d"); >>>> MODULE_ALIAS("platform:hx8353d"); >>> >>> I understand why both platform and spi aliases are needed for >>> fb_hx8353d: FBTFT_REGISTER_DRIVER macro does register 2 separate >>> drivers, one SPI and another platform, and so needs both aliases. You >>> still have not explained why we need SPI alias for stmpe-ts which is MFD >>> cell represented by a platform device/driver no matter what bus is used >>> to connect stmpe itself, I2C or SPI. >>> >>> What is the modalias on the stmpe-ts device in sysfs? >>> >>> Thanks. >>> >> These are the stmpe-related entries under /sys/devices: >> >> ./soc/3f204000.spi/spi_master/spi0/spi0.1/stmpe-gpio >> ./soc/3f204000.spi/spi_master/spi0/spi0.1/stmpe-ts >> >> stmpe-ts/modalias is: >> of:Nstmpe_touchscreenT<NULL>Cst,stmpe-ts > > OK, that explains it. Does the patch below help? > > Thanks. > Yes, with your patch autoloading works. Thanks. ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2015-05-22 21:02 UTC | newest] Thread overview: 7+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2015-05-21 21:19 [PATCH] touchscreen: stmpe-ts: enable module autoload Heiner Kallweit 2015-05-21 23:06 ` Dmitry Torokhov 2015-05-22 5:34 ` Heiner Kallweit 2015-05-22 18:29 ` Dmitry Torokhov 2015-05-22 19:01 ` Heiner Kallweit 2015-05-22 20:17 ` Dmitry Torokhov 2015-05-22 21:02 ` Heiner Kallweit
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).