Re: [PATCH v4 3/3] drm/panel: add support for Innolux TV123WAM panel

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Apr 27, 2018 at 03:09:42PM +0530, Sandeep Panda wrote:
> Innolux TV123WAM is a 12.3" eDP display panel with
> 2160x1440 resolution, which can be supported by simple
> panel driver.
> 
> Changes in v1:
>  - Make use of simple panel driver instead of creating
>    a new driver for this panel (Sean Paul).
>  - Combine dt-binding and driver changes into one patch
>    as done by other existing panel support changes.

While allowed, having them separate is preferred.

> 
> Signed-off-by: Sandeep Panda <spanda@xxxxxxxxxxxxxx>
> ---
>  .../bindings/display/panel/innolux,tv123wam.txt    |  7 ++++++
>  drivers/gpu/drm/panel/panel-simple.c               | 27 ++++++++++++++++++++++
>  2 files changed, 34 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/display/panel/innolux,tv123wam.txt
> 
> diff --git a/Documentation/devicetree/bindings/display/panel/innolux,tv123wam.txt b/Documentation/devicetree/bindings/display/panel/innolux,tv123wam.txt
> new file mode 100644
> index 0000000..c12e714
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/display/panel/innolux,tv123wam.txt
> @@ -0,0 +1,7 @@
> +Innolux TV123WAM 12.3 inch eDP 2K display panel
> +
> +Required properties:
> +- compatible: should be "innolux,tv123wam"
> +
> +This binding is compatible with the simple-panel binding, which is specified
> +in simple-panel.txt in this directory.

You need to state exactly what properties from this apply.

Rob
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux