Re: [PATCH 2/2] dt-bindings: display: Add obsolete note to "ti,tilcdc,panel" binding

[Jyri Sarha <jsarha@ti.com>]

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
>>
> 


-- 
Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki.
Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki