From: Parshuram Raju Thombare <pthombar@cadence.com>
To: Rob Herring <robh@kernel.org>
Cc: "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
"narmstrong@baylibre.com" <narmstrong@baylibre.com>,
"airlied@linux.ie" <airlied@linux.ie>,
"robert.foss@linaro.org" <robert.foss@linaro.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"dri-devel@lists.freedesktop.org"
<dri-devel@lists.freedesktop.org>,
"kishon@ti.com" <kishon@ti.com>,
"a.hajda@samsung.com" <a.hajda@samsung.com>,
"laurent.pinchart@ideasonboard.com"
<laurent.pinchart@ideasonboard.com>,
Swapnil Kashinath Jakhade <sjakhade@cadence.com>,
"nikhil.nd@ti.com" <nikhil.nd@ti.com>,
Milind Parab <mparab@cadence.com>
Subject: RE: [PATCH v5 1/2] dt-bindings: drm/bridge: MHDP8546 bridge binding changes for HDCP
Date: Wed, 7 Apr 2021 06:03:29 +0000 [thread overview]
Message-ID: <MWHPR07MB276881500489CBFE1F297C1FC1759@MWHPR07MB2768.namprd07.prod.outlook.com> (raw)
In-Reply-To: <20210323224546.GA1485576@robh.at.kernel.org>
Hi Rob,
Thanks for your comment and sorry for late response.
>> - minItems: 1
>> - maxItems: 2
>> + minItems: 2
>> + maxItems: 3
>
>1 entry was valid and now it is not? That's not a compatible change and
>needs an explanation at a minimum.
Yes, as the driver returns error in the absence of SAPB port address range, this
may break the driver on platform without SAPB port. This IP needs SAPB port
to support HDCP feature using embedded HDCP crypto module.
So, to maintain backward compatibility I will modify the driver and allow it
to proceed without HDCP feature, instead of returning an error, in case of no SAPB port.
Regards,
Parshuram Thombare
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
next prev parent reply other threads:[~2021-04-07 6:05 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-03-19 9:37 [PATCH v5 0/2] enable HDCP in Cadence MHDP bridge driver Parshuram Thombare
2021-03-19 9:38 ` [PATCH v5 1/2] dt-bindings: drm/bridge: MHDP8546 bridge binding changes for HDCP Parshuram Thombare
2021-03-23 22:45 ` Rob Herring
2021-04-07 6:03 ` Parshuram Raju Thombare [this message]
2021-03-19 9:38 ` [PATCH v5 2/2] drm: bridge: cdns-mhdp8546: Enable HDCP Parshuram Thombare
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=MWHPR07MB276881500489CBFE1F297C1FC1759@MWHPR07MB2768.namprd07.prod.outlook.com \
--to=pthombar@cadence.com \
--cc=a.hajda@samsung.com \
--cc=airlied@linux.ie \
--cc=devicetree@vger.kernel.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=kishon@ti.com \
--cc=laurent.pinchart@ideasonboard.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mparab@cadence.com \
--cc=narmstrong@baylibre.com \
--cc=nikhil.nd@ti.com \
--cc=robert.foss@linaro.org \
--cc=robh@kernel.org \
--cc=sjakhade@cadence.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).