[2/2] dt-bindings: display: Add obsolete note to "ti,tilcdc,panel" binding
diff mbox series

Message ID eabe78184ea7125dcff50826a0ee12a892443017.1575481630.git.jsarha@ti.com
State New
Headers show
Series
  • drm/tilcdc: Remove "ti,tilcdc,tfp410" and mark "ti,tilcdc,panel" obsolete
Related show

Commit Message

Jyri Sarha Dec. 4, 2019, 5:53 p.m. UTC
The "ti,tilcdc,panel" binding should not be used anymore, since tilcdc
is fully capable of using generic drm panels like panel-simple and
panel-common binding. However, the obsolete binding is still widely
used in many mainline supported platforms that I do not have access to
and who knows how many custom platforms. So I am afraid we have to
keep the old bundled tilcdc panel driver around.

Signed-off-by: Jyri Sarha <jsarha@ti.com>
---
 Documentation/devicetree/bindings/display/tilcdc/panel.txt | 6 ++++++
 1 file changed, 6 insertions(+)

Comments

H. Nikolaus Schaller Dec. 5, 2019, 7:06 a.m. UTC | #1
> Am 04.12.2019 um 18:53 schrieb Jyri Sarha <jsarha@ti.com>:
> 
> The "ti,tilcdc,panel" binding should not be used anymore, since tilcdc
> is fully capable of using generic drm panels like panel-simple and
> panel-common binding.

Is there an example how to convert?

As far as I can see all arch/arm/boot/dts/am335x-*.dts use "ti,tilcdc,panel"

arch/arm/boot/dts/am335x-evm.dts would IMHO be a good candidate.

> However, the obsolete binding is still widely
> used in many mainline supported platforms that I do not have access to
> and who knows how many custom platforms. So I am afraid we have to
> keep the old bundled tilcdc panel driver around.
> 
> Signed-off-by: Jyri Sarha <jsarha@ti.com>
> ---
> Documentation/devicetree/bindings/display/tilcdc/panel.txt | 6 ++++++
> 1 file changed, 6 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/display/tilcdc/panel.txt b/Documentation/devicetree/bindings/display/tilcdc/panel.txt
> index 808216310ea2..54963f9173cc 100644
> --- a/Documentation/devicetree/bindings/display/tilcdc/panel.txt
> +++ b/Documentation/devicetree/bindings/display/tilcdc/panel.txt
> @@ -1,5 +1,11 @@
> Device-Tree bindings for tilcdc DRM generic panel output driver
> 
> +NOTE: This binding (and the related driver) is obsolete and should not
> +      be used anymore. Please refer to drm panel-common binding (and
> +      to a generic drm panel driver like panel-simple).
> +      Documentation/devicetree/bindings/display/panel/panel-common.yaml
> +      (drivers/gpu/drm/panel/panel-simple.c)
> +
> Required properties:
>  - compatible: value should be "ti,tilcdc,panel".
>  - panel-info: configuration info to configure LCDC correctly for the panel
> -- 
> Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki
>
Jyri Sarha Dec. 5, 2019, 7:13 a.m. UTC | #2
On 05/12/2019 09:06, H. Nikolaus Schaller wrote:
> 
>> Am 04.12.2019 um 18:53 schrieb Jyri Sarha <jsarha@ti.com>:
>>
>> The "ti,tilcdc,panel" binding should not be used anymore, since tilcdc
>> is fully capable of using generic drm panels like panel-simple and
>> panel-common binding.
> 
> Is there an example how to convert?
> 
> As far as I can see all arch/arm/boot/dts/am335x-*.dts use "ti,tilcdc,panel"
> 
> arch/arm/boot/dts/am335x-evm.dts would IMHO be a good candidate.
> 

Yes, there is. Here are the patches to convert am335x-evm.dts and
am335x-evmsk.dts:

https://lists.freedesktop.org/archives/dri-devel/2019-December/247301.html
https://lists.freedesktop.org/archives/dri-devel/2019-December/247594.html

I would have added a reference to the note, but since the patches are
not yet queued to mainline, I decided against it.

The first thing to do before conversion is to check if panel-simple (or
some other generic drm panel driver) already supports the used panel. If
yes, then the conversion should be quite straight forwards. If not, then
the first thing to do is to add support for the used panel into
panel-simple.c.

Best regards,
Jyri


>> However, the obsolete binding is still widely
>> used in many mainline supported platforms that I do not have access to
>> and who knows how many custom platforms. So I am afraid we have to
>> keep the old bundled tilcdc panel driver around.
>>
>> Signed-off-by: Jyri Sarha <jsarha@ti.com>
>> ---
>> Documentation/devicetree/bindings/display/tilcdc/panel.txt | 6 ++++++
>> 1 file changed, 6 insertions(+)
>>
>> diff --git a/Documentation/devicetree/bindings/display/tilcdc/panel.txt b/Documentation/devicetree/bindings/display/tilcdc/panel.txt
>> index 808216310ea2..54963f9173cc 100644
>> --- a/Documentation/devicetree/bindings/display/tilcdc/panel.txt
>> +++ b/Documentation/devicetree/bindings/display/tilcdc/panel.txt
>> @@ -1,5 +1,11 @@
>> Device-Tree bindings for tilcdc DRM generic panel output driver
>>
>> +NOTE: This binding (and the related driver) is obsolete and should not
>> +      be used anymore. Please refer to drm panel-common binding (and
>> +      to a generic drm panel driver like panel-simple).
>> +      Documentation/devicetree/bindings/display/panel/panel-common.yaml
>> +      (drivers/gpu/drm/panel/panel-simple.c)
>> +
>> Required properties:
>>  - compatible: value should be "ti,tilcdc,panel".
>>  - panel-info: configuration info to configure LCDC correctly for the panel
>> -- 
>> Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki
>>
>
Laurent Pinchart Dec. 5, 2019, 8:46 a.m. UTC | #3
Hi Jyri,

Thank you for the patch.

On Wed, Dec 04, 2019 at 07:53:11PM +0200, Jyri Sarha wrote:
> The "ti,tilcdc,panel" binding should not be used anymore, since tilcdc
> is fully capable of using generic drm panels like panel-simple and
> panel-common binding. However, the obsolete binding is still widely
> used in many mainline supported platforms that I do not have access to
> and who knows how many custom platforms. So I am afraid we have to
> keep the old bundled tilcdc panel driver around.
> 
> Signed-off-by: Jyri Sarha <jsarha@ti.com>
> ---
>  Documentation/devicetree/bindings/display/tilcdc/panel.txt | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/display/tilcdc/panel.txt b/Documentation/devicetree/bindings/display/tilcdc/panel.txt
> index 808216310ea2..54963f9173cc 100644
> --- a/Documentation/devicetree/bindings/display/tilcdc/panel.txt
> +++ b/Documentation/devicetree/bindings/display/tilcdc/panel.txt
> @@ -1,5 +1,11 @@
>  Device-Tree bindings for tilcdc DRM generic panel output driver
>  
> +NOTE: This binding (and the related driver) is obsolete and should not
> +      be used anymore. Please refer to drm panel-common binding (and
> +      to a generic drm panel driver like panel-simple).
> +      Documentation/devicetree/bindings/display/panel/panel-common.yaml
> +      (drivers/gpu/drm/panel/panel-simple.c)
> +

How about adding a WARN_ON() in the code when this binding is used ?
This could help getting those platforms converted.

>  Required properties:
>   - compatible: value should be "ti,tilcdc,panel".
>   - panel-info: configuration info to configure LCDC correctly for the panel
Jyri Sarha Dec. 5, 2019, 9:17 a.m. UTC | #4
On 05/12/2019 10:46, Laurent Pinchart wrote:
> Hi Jyri,
> 
> Thank you for the patch.
> 
> On Wed, Dec 04, 2019 at 07:53:11PM +0200, Jyri Sarha wrote:
>> The "ti,tilcdc,panel" binding should not be used anymore, since tilcdc
>> is fully capable of using generic drm panels like panel-simple and
>> panel-common binding. However, the obsolete binding is still widely
>> used in many mainline supported platforms that I do not have access to
>> and who knows how many custom platforms. So I am afraid we have to
>> keep the old bundled tilcdc panel driver around.
>>
>> Signed-off-by: Jyri Sarha <jsarha@ti.com>
>> ---
>>  Documentation/devicetree/bindings/display/tilcdc/panel.txt | 6 ++++++
>>  1 file changed, 6 insertions(+)
>>
>> diff --git a/Documentation/devicetree/bindings/display/tilcdc/panel.txt b/Documentation/devicetree/bindings/display/tilcdc/panel.txt
>> index 808216310ea2..54963f9173cc 100644
>> --- a/Documentation/devicetree/bindings/display/tilcdc/panel.txt
>> +++ b/Documentation/devicetree/bindings/display/tilcdc/panel.txt
>> @@ -1,5 +1,11 @@
>>  Device-Tree bindings for tilcdc DRM generic panel output driver
>>  
>> +NOTE: This binding (and the related driver) is obsolete and should not
>> +      be used anymore. Please refer to drm panel-common binding (and
>> +      to a generic drm panel driver like panel-simple).
>> +      Documentation/devicetree/bindings/display/panel/panel-common.yaml
>> +      (drivers/gpu/drm/panel/panel-simple.c)
>> +
> 
> How about adding a WARN_ON() in the code when this binding is used ?
> This could help getting those platforms converted.
> 

I don't see why not, I'll add that.

Thanks
Jyro

>>  Required properties:
>>   - compatible: value should be "ti,tilcdc,panel".
>>   - panel-info: configuration info to configure LCDC correctly for the panel
>

Patch
diff mbox series

diff --git a/Documentation/devicetree/bindings/display/tilcdc/panel.txt b/Documentation/devicetree/bindings/display/tilcdc/panel.txt
index 808216310ea2..54963f9173cc 100644
--- a/Documentation/devicetree/bindings/display/tilcdc/panel.txt
+++ b/Documentation/devicetree/bindings/display/tilcdc/panel.txt
@@ -1,5 +1,11 @@ 
 Device-Tree bindings for tilcdc DRM generic panel output driver
 
+NOTE: This binding (and the related driver) is obsolete and should not
+      be used anymore. Please refer to drm panel-common binding (and
+      to a generic drm panel driver like panel-simple).
+      Documentation/devicetree/bindings/display/panel/panel-common.yaml
+      (drivers/gpu/drm/panel/panel-simple.c)
+
 Required properties:
  - compatible: value should be "ti,tilcdc,panel".
  - panel-info: configuration info to configure LCDC correctly for the panel