linux-kernel - RE: [PATCH 1/2] dt-bindings: drm/bridge: MHDP8546 bridge binding changes for HDCP

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <DM5PR07MB3196D92300EDCB16E8F2F724C19A9@DM5PR07MB3196.namprd07.prod.outlook.com>
Date:   Mon, 1 Mar 2021 08:49:40 +0000
From:   Parshuram Raju Thombare <pthombar@...ence.com>
To:     Robert Foss <robert.foss@...aro.org>
CC:     Rob Herring <robh+dt@...nel.org>,
        Laurent Pinchart <laurent.pinchart@...asonboard.com>,
        David Airlie <airlied@...ux.ie>,
        Daniel Vetter <daniel@...ll.ch>,
        dri-devel <dri-devel@...ts.freedesktop.org>,
        "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" 
        <devicetree@...r.kernel.org>,
        linux-kernel <linux-kernel@...r.kernel.org>,
        Andrzej Hajda <a.hajda@...sung.com>,
        Neil Armstrong <narmstrong@...libre.com>,
        "nikhil.nd@...com" <nikhil.nd@...com>,
        "kishon@...com" <kishon@...com>,
        Swapnil Kashinath Jakhade <sjakhade@...ence.com>,
        Milind Parab <mparab@...ence.com>
Subject: RE: [PATCH 1/2] dt-bindings: drm/bridge: MHDP8546 bridge binding
 changes for HDCP

Hi Robert,

Thanks for your review comments.
I will fix this in next version of patch set.

Regards,
Parshuram Thombare

>-----Original Message-----
>From: Robert Foss <robert.foss@...aro.org>
>Sent: Friday, February 26, 2021 10:06 PM
>To: Parshuram Raju Thombare <pthombar@...ence.com>
>Cc: Rob Herring <robh+dt@...nel.org>; Laurent Pinchart
><laurent.pinchart@...asonboard.com>; David Airlie <airlied@...ux.ie>; Daniel
>Vetter <daniel@...ll.ch>; dri-devel <dri-devel@...ts.freedesktop.org>; open
>list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS
><devicetree@...r.kernel.org>; linux-kernel <linux-kernel@...r.kernel.org>;
>Andrzej Hajda <a.hajda@...sung.com>; Neil Armstrong
><narmstrong@...libre.com>; nikhil.nd@...com; kishon@...com; Swapnil
>Kashinath Jakhade <sjakhade@...ence.com>; Milind Parab
><mparab@...ence.com>
>Subject: Re: [PATCH 1/2] dt-bindings: drm/bridge: MHDP8546 bridge binding
>changes for HDCP
>
>EXTERNAL MAIL
>
>
>Hey Parshuram,
>
>Thanks for submitting this.
>
>This series rebased on upstream-drm-misc/for-linux-next fails dt
>binding verification.
>$ make dt_binding_check
>DT_SCHEMA_FILES=Documentation/devicetree/bindings/display/bridge/cdns,mh
>dp8546.yaml
>
>
>On Fri, 26 Feb 2021 at 17:18, Parshuram Thombare <pthombar@...ence.com>
>wrote:
>>
>> Add binding changes for HDCP in the MHDP8546 DPI/DP bridge binding.
>>
>> Signed-off-by: Parshuram Thombare <pthombar@...ence.com>
>> ---
>>  .../display/bridge/cdns,mhdp8546.yaml         | 20 +++++++++++++------
>>  1 file changed, 14 insertions(+), 6 deletions(-)
>>
>> diff --git
>a/Documentation/devicetree/bindings/display/bridge/cdns,mhdp8546.yaml
>b/Documentation/devicetree/bindings/display/bridge/cdns,mhdp8546.yaml
>> index 63427878715e..89b4bf783c53 100644
>> --- a/Documentation/devicetree/bindings/display/bridge/cdns,mhdp8546.yaml
>> +++
>b/Documentation/devicetree/bindings/display/bridge/cdns,mhdp8546.yaml
>> @@ -18,7 +18,7 @@ properties:
>>
>>    reg:
>>      minItems: 1
>> -    maxItems: 2
>> +    maxItems: 3
>>      items:
>>        - description:
>>            Register block of mhdptx apb registers up to PHY mapped area
>(AUX_CONFIG_P).
>> @@ -26,13 +26,16 @@ properties:
>>            included in the associated PHY.
>>        - description:
>>            Register block for DSS_EDP0_INTG_CFG_VP registers in case of TI J7
>SoCs.
>> +      - description:
>> +          Register block of mhdptx sapb registers.
>>
>>    reg-names:
>>      minItems: 1
>> -    maxItems: 2
>> +    maxItems: 3
>>      items:
>>        - const: mhdptx
>>        - const: j721e-intg
>> +      - const: mhdptx-sapb
>
>The reg & reg-names changes appear to be introducing the failure.
>
>>
>>    clocks:
>>      maxItems: 1
>> @@ -53,6 +56,11 @@ properties:
>>    power-domains:
>>      maxItems: 1
>>
>> +  hdcp-config:
>> +    maxItems: 1
>> +    description:
>> +      HDCP version supported. Bit [0]:HDCP2.2 [1]:HDCP1.4.
>> +
>>    interrupts:
>>      maxItems: 1
>>
>> @@ -98,15 +106,15 @@ allOf:
>>      then:
>>        properties:
>>          reg:
>> -          minItems: 2
>> +          minItems: 3
>>          reg-names:
>> -          minItems: 2
>> +          minItems: 3
>>      else:
>>        properties:
>>          reg:
>> -          maxItems: 1
>> +          maxItems: 2
>>          reg-names:
>> -          maxItems: 1
>> +          maxItems: 2
>>
>>  required:
>>    - compatible
>> --
>> 2.25.1
>>