2021-02-26 16:20:33

by Parshuram Raju Thombare

[permalink] [raw]
Subject: [PATCH 0/2] enable HDCP in Cadence MHDP bridge driver

This patch series enables HDCP in Cadence MHDP DPI/DP bridge driver.

Parshuram Thombare (2):
dt-bindings: drm/bridge: MHDP8546 bridge binding changes for HDCP
drm: bridge: cdns-mhdp8546: Enable HDCP

.../display/bridge/cdns,mhdp8546.yaml | 20 +-
drivers/gpu/drm/bridge/cadence/Makefile | 2 +-
.../drm/bridge/cadence/cdns-mhdp8546-core.c | 105 +++-
.../drm/bridge/cadence/cdns-mhdp8546-core.h | 21 +
.../drm/bridge/cadence/cdns-mhdp8546-hdcp.c | 578 ++++++++++++++++++
.../drm/bridge/cadence/cdns-mhdp8546-hdcp.h | 89 +++
6 files changed, 796 insertions(+), 19 deletions(-)
create mode 100644 drivers/gpu/drm/bridge/cadence/cdns-mhdp8546-hdcp.c
create mode 100644 drivers/gpu/drm/bridge/cadence/cdns-mhdp8546-hdcp.h

--
2.25.1


2021-02-26 16:20:46

by Parshuram Raju Thombare

[permalink] [raw]
Subject: [PATCH 1/2] dt-bindings: drm/bridge: MHDP8546 bridge binding changes for HDCP

Add binding changes for HDCP in the MHDP8546 DPI/DP bridge binding.

Signed-off-by: Parshuram Thombare <[email protected]>
---
.../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

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

2021-02-26 16:38:41

by Robert Foss

[permalink] [raw]
Subject: Re: [PATCH 1/2] dt-bindings: drm/bridge: MHDP8546 bridge binding changes for HDCP

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,mhdp8546.yaml


On Fri, 26 Feb 2021 at 17:18, Parshuram Thombare <[email protected]> wrote:
>
> Add binding changes for HDCP in the MHDP8546 DPI/DP bridge binding.
>
> Signed-off-by: Parshuram Thombare <[email protected]>
> ---
> .../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
>

2021-03-01 08:57:45

by Parshuram Raju Thombare

[permalink] [raw]
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 <[email protected]>
>Sent: Friday, February 26, 2021 10:06 PM
>To: Parshuram Raju Thombare <[email protected]>
>Cc: Rob Herring <[email protected]>; Laurent Pinchart
><[email protected]>; David Airlie <[email protected]>; Daniel
>Vetter <[email protected]>; dri-devel <[email protected]>; open
>list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS
><[email protected]>; linux-kernel <[email protected]>;
>Andrzej Hajda <[email protected]>; Neil Armstrong
><[email protected]>; [email protected]; [email protected]; Swapnil
>Kashinath Jakhade <[email protected]>; Milind Parab
><[email protected]>
>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 <[email protected]>
>wrote:
>>
>> Add binding changes for HDCP in the MHDP8546 DPI/DP bridge binding.
>>
>> Signed-off-by: Parshuram Thombare <[email protected]>
>> ---
>> .../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
>>