Re: [PATCHv2 4/8] dt-bindings: panel: common: document orientation property

From: Rob Herring
Date: Tue Feb 20 2018 - 14:09:30 EST


On Mon, Feb 19, 2018 at 5:06 AM, Sebastian Reichel
<sebastian.reichel@xxxxxxxxxxxxxxx> wrote:
> Hi,
>
> On Sun, Feb 18, 2018 at 05:24:24PM -0600, Rob Herring wrote:
>> On Thu, Feb 08, 2018 at 07:30:31PM +0100, Sebastian Reichel wrote:
>> > Introduce new "orientation" property for describing in which
>> > orientation a panel has been mounted to the device. This can
>> > be used by the operating system to automatically rotate the
>> > display correctly.
>> >
>> > Signed-off-by: Sebastian Reichel <sebastian.reichel@xxxxxxxxxxxxxxx>
>> > ---
>> > .../devicetree/bindings/display/panel/panel-common.txt | 12 ++++++++++++
>> > include/dt-bindings/display/common.h | 14 ++++++++++++++
>> > 2 files changed, 26 insertions(+)
>> > create mode 100644 include/dt-bindings/display/common.h
>> >
>> > diff --git a/Documentation/devicetree/bindings/display/panel/panel-common.txt b/Documentation/devicetree/bindings/display/panel/panel-common.txt
>> > index 557fa765adcb..c646b8908458 100644
>> > --- a/Documentation/devicetree/bindings/display/panel/panel-common.txt
>> > +++ b/Documentation/devicetree/bindings/display/panel/panel-common.txt
>> > @@ -18,6 +18,18 @@ Descriptive Properties
>> > physical area where images are displayed. These properties are expressed in
>> > millimeters and rounded to the closest unit.
>> >
>> > +- orientation: The orientation property specifies the panel orientation
>> > + in relation to the device's casing. The following values are possible:
>> > +
>> > + * 0 = The top side of the panel matches the top side of the device's
>> > + casing.
>> > + * 1 = The top side of the panel matches the bottom side of the device's
>> > + casing. In other words the panel is mounted upside-down.
>> > + * 2 = The left side of the panel matches the top side of the device's
>> > + casing.
>> > + * 3 = The right side of the panel matches the top side of the device's
>> > + casing.
>>
>> The rotation property in panel.txt already handles this, right?
>
> Yes, looks like it can be used. I guess panel.txt and
> panel-common.txt should be merged, though?

Yes, please. Actually, I'd like to merge simple-panel.txt too because
that is really kernel driver, not a type of panel. But there's a lot
of references to it to fix.

Rob