mbox series

[0/5] Add CDTech 4.3" and 7" to panel-simple

Message ID 20180730231117.5631-1-giulio.benetti@micronovasrl.com (mailing list archive)
Headers show
Series Add CDTech 4.3" and 7" to panel-simple | expand

Message

Giulio Benetti July 30, 2018, 11:11 p.m. UTC
Add CDTech 4.3" S043WQ26H-CT7 support
Add CDTech 7" S070WV95-CT16 support

Giulio Benetti (5):
  dt-bindings: Add vendor prefix for CDTech(H.K.) Electronics Limited
  drm/panel: add panel CDTech S070WV95-CT16 to panel-simple
  dt-bindings: Add CDTech S070WV95-CT16 panel bindings
  drm/panel: add panel CDTech S043WQ26H-CT7 to panel-simple
  dt-bindings: Add CDTech S043WQ26H-CT7 panel bindings

 .../display/panel/cdtech,s043wq26h-ct7.txt    | 12 ++++
 .../display/panel/cdtech,s070wv95-ct16.txt    | 12 ++++
 .../devicetree/bindings/vendor-prefixes.txt   |  1 +
 drivers/gpu/drm/panel/panel-simple.c          | 55 +++++++++++++++++++
 4 files changed, 80 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/display/panel/cdtech,s043wq26h-ct7.txt
 create mode 100644 Documentation/devicetree/bindings/display/panel/cdtech,s070wv95-ct16.txt

Comments

Thierry Reding Sept. 27, 2018, 11:59 a.m. UTC | #1
On Tue, Jul 31, 2018 at 01:11:12AM +0200, Giulio Benetti wrote:
> Add CDTech 4.3" S043WQ26H-CT7 support
> Add CDTech 7" S070WV95-CT16 support
> 
> Giulio Benetti (5):
>   dt-bindings: Add vendor prefix for CDTech(H.K.) Electronics Limited
>   drm/panel: add panel CDTech S070WV95-CT16 to panel-simple
>   dt-bindings: Add CDTech S070WV95-CT16 panel bindings
>   drm/panel: add panel CDTech S043WQ26H-CT7 to panel-simple
>   dt-bindings: Add CDTech S043WQ26H-CT7 panel bindings
> 
>  .../display/panel/cdtech,s043wq26h-ct7.txt    | 12 ++++
>  .../display/panel/cdtech,s070wv95-ct16.txt    | 12 ++++
>  .../devicetree/bindings/vendor-prefixes.txt   |  1 +
>  drivers/gpu/drm/panel/panel-simple.c          | 55 +++++++++++++++++++
>  4 files changed, 80 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/display/panel/cdtech,s043wq26h-ct7.txt
>  create mode 100644 Documentation/devicetree/bindings/display/panel/cdtech,s070wv95-ct16.txt

All applied, thanks. One small note, though: in the future please send
DT bindings patches before the driver patches. This is important because
checkpatch, which is run as pre-commit script, warns about undocumented
compatible strings if you send driver changes before the bindings.

Thierry
Giulio Benetti Sept. 27, 2018, 1:08 p.m. UTC | #2
Hello Thierry,

Il 27/09/2018 13:59, Thierry Reding ha scritto:
> On Tue, Jul 31, 2018 at 01:11:12AM +0200, Giulio Benetti wrote:
>> Add CDTech 4.3" S043WQ26H-CT7 support
>> Add CDTech 7" S070WV95-CT16 support
>>
>> Giulio Benetti (5):
>>    dt-bindings: Add vendor prefix for CDTech(H.K.) Electronics Limited
>>    drm/panel: add panel CDTech S070WV95-CT16 to panel-simple
>>    dt-bindings: Add CDTech S070WV95-CT16 panel bindings
>>    drm/panel: add panel CDTech S043WQ26H-CT7 to panel-simple
>>    dt-bindings: Add CDTech S043WQ26H-CT7 panel bindings
>>
>>   .../display/panel/cdtech,s043wq26h-ct7.txt    | 12 ++++
>>   .../display/panel/cdtech,s070wv95-ct16.txt    | 12 ++++
>>   .../devicetree/bindings/vendor-prefixes.txt   |  1 +
>>   drivers/gpu/drm/panel/panel-simple.c          | 55 +++++++++++++++++++
>>   4 files changed, 80 insertions(+)
>>   create mode 100644 Documentation/devicetree/bindings/display/panel/cdtech,s043wq26h-ct7.txt
>>   create mode 100644 Documentation/devicetree/bindings/display/panel/cdtech,s070wv95-ct16.txt
> 
> All applied, thanks. One small note, though: in the future please send
> DT bindings patches before the driver patches. This is important because
> checkpatch, which is run as pre-commit script, warns about undocumented
> compatible strings if you send driver changes before the bindings.

Ok, thanks for both applying and pointing me this important detail.