* [PATCH] dt-bindings: touchscreen: silead_gsl1680: Document touchscreen-fw-name
@ 2016-08-23 15:04 ` Hans de Goede
0 siblings, 0 replies; 10+ messages in thread
From: Hans de Goede @ 2016-08-23 15:04 UTC (permalink / raw)
To: Dmitry Torokhov, Rob Herring, Chen-Yu Tsai
Cc: devicetree, Maxime Ripard, Hans de Goede, linux-arm-kernel, linux-input
The driver has supported touchscreen-fw-name to specify the firmware to
load since day 0, but this was omitted from the binding, document this.
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---
Documentation/devicetree/bindings/input/touchscreen/silead_gsl1680.txt | 1 +
1 file changed, 1 insertion(+)
diff --git a/Documentation/devicetree/bindings/input/touchscreen/silead_gsl1680.txt b/Documentation/devicetree/bindings/input/touchscreen/silead_gsl1680.txt
index ad7f41a..c9c1ddd 100644
--- a/Documentation/devicetree/bindings/input/touchscreen/silead_gsl1680.txt
+++ b/Documentation/devicetree/bindings/input/touchscreen/silead_gsl1680.txt
@@ -17,6 +17,7 @@ Required properties:
- touchscreen-size-y : See touchscreen.txt
Optional properties:
+- touchscreen-fw-name : Filename (string) for device specific firmware
- touchscreen-inverted-x : See touchscreen.txt
- touchscreen-inverted-y : See touchscreen.txt
- touchscreen-swapped-x-y : See touchscreen.txt
--
2.7.4
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH] dt-bindings: touchscreen: silead_gsl1680: Document touchscreen-fw-name
@ 2016-08-23 15:04 ` Hans de Goede
0 siblings, 0 replies; 10+ messages in thread
From: Hans de Goede @ 2016-08-23 15:04 UTC (permalink / raw)
To: linux-arm-kernel
The driver has supported touchscreen-fw-name to specify the firmware to
load since day 0, but this was omitted from the binding, document this.
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---
Documentation/devicetree/bindings/input/touchscreen/silead_gsl1680.txt | 1 +
1 file changed, 1 insertion(+)
diff --git a/Documentation/devicetree/bindings/input/touchscreen/silead_gsl1680.txt b/Documentation/devicetree/bindings/input/touchscreen/silead_gsl1680.txt
index ad7f41a..c9c1ddd 100644
--- a/Documentation/devicetree/bindings/input/touchscreen/silead_gsl1680.txt
+++ b/Documentation/devicetree/bindings/input/touchscreen/silead_gsl1680.txt
@@ -17,6 +17,7 @@ Required properties:
- touchscreen-size-y : See touchscreen.txt
Optional properties:
+- touchscreen-fw-name : Filename (string) for device specific firmware
- touchscreen-inverted-x : See touchscreen.txt
- touchscreen-inverted-y : See touchscreen.txt
- touchscreen-swapped-x-y : See touchscreen.txt
--
2.7.4
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH] dt-bindings: touchscreen: silead_gsl1680: Document touchscreen-fw-name
2016-08-23 15:04 ` Hans de Goede
@ 2016-08-29 0:37 ` Rob Herring
-1 siblings, 0 replies; 10+ messages in thread
From: Rob Herring @ 2016-08-29 0:37 UTC (permalink / raw)
To: Hans de Goede
Cc: Dmitry Torokhov, Chen-Yu Tsai, Maxime Ripard, linux-input,
linux-arm-kernel, devicetree
On Tue, Aug 23, 2016 at 05:04:48PM +0200, Hans de Goede wrote:
> The driver has supported touchscreen-fw-name to specify the firmware to
> load since day 0, but this was omitted from the binding, document this.
>
> Signed-off-by: Hans de Goede <hdegoede@redhat.com>
> ---
> Documentation/devicetree/bindings/input/touchscreen/silead_gsl1680.txt | 1 +
> 1 file changed, 1 insertion(+)
I'd really like to deprecate in favor of a standard property name. We've
somewhat agreed on 'firmware-name' though there is no common binding doc
yet. But given this is in use:
Acked-by: Rob Herring <robh@kernel.org>
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH] dt-bindings: touchscreen: silead_gsl1680: Document touchscreen-fw-name
@ 2016-08-29 0:37 ` Rob Herring
0 siblings, 0 replies; 10+ messages in thread
From: Rob Herring @ 2016-08-29 0:37 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Aug 23, 2016 at 05:04:48PM +0200, Hans de Goede wrote:
> The driver has supported touchscreen-fw-name to specify the firmware to
> load since day 0, but this was omitted from the binding, document this.
>
> Signed-off-by: Hans de Goede <hdegoede@redhat.com>
> ---
> Documentation/devicetree/bindings/input/touchscreen/silead_gsl1680.txt | 1 +
> 1 file changed, 1 insertion(+)
I'd really like to deprecate in favor of a standard property name. We've
somewhat agreed on 'firmware-name' though there is no common binding doc
yet. But given this is in use:
Acked-by: Rob Herring <robh@kernel.org>
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] dt-bindings: touchscreen: silead_gsl1680: Document touchscreen-fw-name
2016-08-29 0:37 ` Rob Herring
@ 2016-08-29 5:18 ` Maxime Ripard
-1 siblings, 0 replies; 10+ messages in thread
From: Maxime Ripard @ 2016-08-29 5:18 UTC (permalink / raw)
To: Rob Herring
Cc: Hans de Goede, Dmitry Torokhov, Chen-Yu Tsai, linux-input,
linux-arm-kernel, devicetree
[-- Attachment #1: Type: text/plain, Size: 1004 bytes --]
Hi Rob,
On Sun, Aug 28, 2016 at 07:37:25PM -0500, Rob Herring wrote:
> On Tue, Aug 23, 2016 at 05:04:48PM +0200, Hans de Goede wrote:
> > The driver has supported touchscreen-fw-name to specify the firmware to
> > load since day 0, but this was omitted from the binding, document this.
> >
> > Signed-off-by: Hans de Goede <hdegoede@redhat.com>
> > ---
> > Documentation/devicetree/bindings/input/touchscreen/silead_gsl1680.txt | 1 +
> > 1 file changed, 1 insertion(+)
>
> I'd really like to deprecate in favor of a standard property name. We've
> somewhat agreed on 'firmware-name' though there is no common binding doc
> yet. But given this is in use:
This has been introduced in 4.8-rc1, so it hasn't been used in any
stable version yet, and we don't have any users in tree so far.
If you want to something else, it feels like now is the time to change
it.
Maxime
--
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH] dt-bindings: touchscreen: silead_gsl1680: Document touchscreen-fw-name
@ 2016-08-29 5:18 ` Maxime Ripard
0 siblings, 0 replies; 10+ messages in thread
From: Maxime Ripard @ 2016-08-29 5:18 UTC (permalink / raw)
To: linux-arm-kernel
Hi Rob,
On Sun, Aug 28, 2016 at 07:37:25PM -0500, Rob Herring wrote:
> On Tue, Aug 23, 2016 at 05:04:48PM +0200, Hans de Goede wrote:
> > The driver has supported touchscreen-fw-name to specify the firmware to
> > load since day 0, but this was omitted from the binding, document this.
> >
> > Signed-off-by: Hans de Goede <hdegoede@redhat.com>
> > ---
> > Documentation/devicetree/bindings/input/touchscreen/silead_gsl1680.txt | 1 +
> > 1 file changed, 1 insertion(+)
>
> I'd really like to deprecate in favor of a standard property name. We've
> somewhat agreed on 'firmware-name' though there is no common binding doc
> yet. But given this is in use:
This has been introduced in 4.8-rc1, so it hasn't been used in any
stable version yet, and we don't have any users in tree so far.
If you want to something else, it feels like now is the time to change
it.
Maxime
--
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20160829/9aea5655/attachment.sig>
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] dt-bindings: touchscreen: silead_gsl1680: Document touchscreen-fw-name
2016-08-29 5:18 ` Maxime Ripard
@ 2016-08-29 9:35 ` Hans de Goede
-1 siblings, 0 replies; 10+ messages in thread
From: Hans de Goede @ 2016-08-29 9:35 UTC (permalink / raw)
To: Maxime Ripard, Rob Herring
Cc: Dmitry Torokhov, Chen-Yu Tsai, linux-input, linux-arm-kernel, devicetree
Hi,
On 29-08-16 07:18, Maxime Ripard wrote:
> Hi Rob,
>
> On Sun, Aug 28, 2016 at 07:37:25PM -0500, Rob Herring wrote:
>> On Tue, Aug 23, 2016 at 05:04:48PM +0200, Hans de Goede wrote:
>>> The driver has supported touchscreen-fw-name to specify the firmware to
>>> load since day 0, but this was omitted from the binding, document this.
>>>
>>> Signed-off-by: Hans de Goede <hdegoede@redhat.com>
>>> ---
>>> Documentation/devicetree/bindings/input/touchscreen/silead_gsl1680.txt | 1 +
>>> 1 file changed, 1 insertion(+)
>>
>> I'd really like to deprecate in favor of a standard property name. We've
>> somewhat agreed on 'firmware-name' though there is no common binding doc
>> yet. But given this is in use:
>
> This has been introduced in 4.8-rc1, so it hasn't been used in any
> stable version yet, and we don't have any users in tree so far.
>
> If you want to something else, it feels like now is the time to change
> it.
Ack, I can still change it to firmware-name if you want (Maxime has not yet
merged any dts changes using the touchscreen-fw-name). We should probably
also decide if the silead/ prefix we're using (so the firmware files are
/lib/firmware/silead/foo.fw under Linux) goes into the firmware-name
property as my current wip code is doing, or if we put the "silead/"
part in the driver and only put the part after it in the property.
I've a feeling that just putting only the base filename (so no "silead/"
prefix) in the property is best.
Regards,
Hans
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH] dt-bindings: touchscreen: silead_gsl1680: Document touchscreen-fw-name
@ 2016-08-29 9:35 ` Hans de Goede
0 siblings, 0 replies; 10+ messages in thread
From: Hans de Goede @ 2016-08-29 9:35 UTC (permalink / raw)
To: linux-arm-kernel
Hi,
On 29-08-16 07:18, Maxime Ripard wrote:
> Hi Rob,
>
> On Sun, Aug 28, 2016 at 07:37:25PM -0500, Rob Herring wrote:
>> On Tue, Aug 23, 2016 at 05:04:48PM +0200, Hans de Goede wrote:
>>> The driver has supported touchscreen-fw-name to specify the firmware to
>>> load since day 0, but this was omitted from the binding, document this.
>>>
>>> Signed-off-by: Hans de Goede <hdegoede@redhat.com>
>>> ---
>>> Documentation/devicetree/bindings/input/touchscreen/silead_gsl1680.txt | 1 +
>>> 1 file changed, 1 insertion(+)
>>
>> I'd really like to deprecate in favor of a standard property name. We've
>> somewhat agreed on 'firmware-name' though there is no common binding doc
>> yet. But given this is in use:
>
> This has been introduced in 4.8-rc1, so it hasn't been used in any
> stable version yet, and we don't have any users in tree so far.
>
> If you want to something else, it feels like now is the time to change
> it.
Ack, I can still change it to firmware-name if you want (Maxime has not yet
merged any dts changes using the touchscreen-fw-name). We should probably
also decide if the silead/ prefix we're using (so the firmware files are
/lib/firmware/silead/foo.fw under Linux) goes into the firmware-name
property as my current wip code is doing, or if we put the "silead/"
part in the driver and only put the part after it in the property.
I've a feeling that just putting only the base filename (so no "silead/"
prefix) in the property is best.
Regards,
Hans
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] dt-bindings: touchscreen: silead_gsl1680: Document touchscreen-fw-name
2016-08-29 9:35 ` Hans de Goede
@ 2016-08-30 16:34 ` Maxime Ripard
-1 siblings, 0 replies; 10+ messages in thread
From: Maxime Ripard @ 2016-08-30 16:34 UTC (permalink / raw)
To: Hans de Goede
Cc: Rob Herring, Dmitry Torokhov, Chen-Yu Tsai,
linux-input-u79uwXL29TY76Z2rM5mHXA,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, devicetree
[-- Attachment #1: Type: text/plain, Size: 1786 bytes --]
On Mon, Aug 29, 2016 at 11:35:47AM +0200, Hans de Goede wrote:
> Hi,
>
> On 29-08-16 07:18, Maxime Ripard wrote:
> >Hi Rob,
> >
> >On Sun, Aug 28, 2016 at 07:37:25PM -0500, Rob Herring wrote:
> >>On Tue, Aug 23, 2016 at 05:04:48PM +0200, Hans de Goede wrote:
> >>>The driver has supported touchscreen-fw-name to specify the firmware to
> >>>load since day 0, but this was omitted from the binding, document this.
> >>>
> >>>Signed-off-by: Hans de Goede <hdegoede-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
> >>>---
> >>> Documentation/devicetree/bindings/input/touchscreen/silead_gsl1680.txt | 1 +
> >>> 1 file changed, 1 insertion(+)
> >>
> >>I'd really like to deprecate in favor of a standard property name. We've
> >>somewhat agreed on 'firmware-name' though there is no common binding doc
> >>yet. But given this is in use:
> >
> >This has been introduced in 4.8-rc1, so it hasn't been used in any
> >stable version yet, and we don't have any users in tree so far.
> >
> >If you want to something else, it feels like now is the time to change
> >it.
>
> Ack, I can still change it to firmware-name if you want (Maxime has not yet
> merged any dts changes using the touchscreen-fw-name). We should probably
> also decide if the silead/ prefix we're using (so the firmware files are
> /lib/firmware/silead/foo.fw under Linux) goes into the firmware-name
> property as my current wip code is doing, or if we put the "silead/"
> part in the driver and only put the part after it in the property.
>
> I've a feeling that just putting only the base filename (so no "silead/"
> prefix) in the property is best.
Yes, I would agree too.
Maxime
--
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH] dt-bindings: touchscreen: silead_gsl1680: Document touchscreen-fw-name
@ 2016-08-30 16:34 ` Maxime Ripard
0 siblings, 0 replies; 10+ messages in thread
From: Maxime Ripard @ 2016-08-30 16:34 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, Aug 29, 2016 at 11:35:47AM +0200, Hans de Goede wrote:
> Hi,
>
> On 29-08-16 07:18, Maxime Ripard wrote:
> >Hi Rob,
> >
> >On Sun, Aug 28, 2016 at 07:37:25PM -0500, Rob Herring wrote:
> >>On Tue, Aug 23, 2016 at 05:04:48PM +0200, Hans de Goede wrote:
> >>>The driver has supported touchscreen-fw-name to specify the firmware to
> >>>load since day 0, but this was omitted from the binding, document this.
> >>>
> >>>Signed-off-by: Hans de Goede <hdegoede@redhat.com>
> >>>---
> >>> Documentation/devicetree/bindings/input/touchscreen/silead_gsl1680.txt | 1 +
> >>> 1 file changed, 1 insertion(+)
> >>
> >>I'd really like to deprecate in favor of a standard property name. We've
> >>somewhat agreed on 'firmware-name' though there is no common binding doc
> >>yet. But given this is in use:
> >
> >This has been introduced in 4.8-rc1, so it hasn't been used in any
> >stable version yet, and we don't have any users in tree so far.
> >
> >If you want to something else, it feels like now is the time to change
> >it.
>
> Ack, I can still change it to firmware-name if you want (Maxime has not yet
> merged any dts changes using the touchscreen-fw-name). We should probably
> also decide if the silead/ prefix we're using (so the firmware files are
> /lib/firmware/silead/foo.fw under Linux) goes into the firmware-name
> property as my current wip code is doing, or if we put the "silead/"
> part in the driver and only put the part after it in the property.
>
> I've a feeling that just putting only the base filename (so no "silead/"
> prefix) in the property is best.
Yes, I would agree too.
Maxime
--
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20160830/c9e82411/attachment.sig>
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2016-08-30 16:34 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-08-23 15:04 [PATCH] dt-bindings: touchscreen: silead_gsl1680: Document touchscreen-fw-name Hans de Goede
2016-08-23 15:04 ` Hans de Goede
2016-08-29 0:37 ` Rob Herring
2016-08-29 0:37 ` Rob Herring
2016-08-29 5:18 ` Maxime Ripard
2016-08-29 5:18 ` Maxime Ripard
2016-08-29 9:35 ` Hans de Goede
2016-08-29 9:35 ` Hans de Goede
[not found] ` <64d65f63-db47-d9c2-1c3f-cc544c2da2ab-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2016-08-30 16:34 ` Maxime Ripard
2016-08-30 16:34 ` Maxime Ripard
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.