From: Krzysztof Kozlowski <krzk@kernel.org>
To: Vishnu Reddy <busanna.reddy@oss.qualcomm.com>
Cc: Vikash Garodia <vikash.garodia@oss.qualcomm.com>,
Dikshita Agarwal <dikshita.agarwal@oss.qualcomm.com>,
Abhinav Kumar <abhinav.kumar@linux.dev>,
Bryan O'Donoghue <bod@kernel.org>,
Mauro Carvalho Chehab <mchehab@kernel.org>,
Joerg Roedel <joro@8bytes.org>, Will Deacon <will@kernel.org>,
Robin Murphy <robin.murphy@arm.com>,
Hans Verkuil <hverkuil@kernel.org>,
Stefan Schmidt <stefan.schmidt@linaro.org>,
Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
Stanimir Varbanov <stanimir.k.varbanov@gmail.com>,
Bjorn Andersson <andersson@kernel.org>,
Konrad Dybcio <konradybcio@kernel.org>,
linux-kernel@vger.kernel.org, linux-media@vger.kernel.org,
linux-arm-msm@vger.kernel.org, iommu@lists.linux.dev,
devicetree@vger.kernel.org
Subject: Re: [PATCH v6 04/14] dt-bindings: media: qcom,venus-common: Raise maxItems for clocks and power-domains
Date: Sat, 16 May 2026 11:31:19 +0200 [thread overview]
Message-ID: <20260516-bald-versatile-wallaby-bcac6c@quoll> (raw)
In-Reply-To: <20260515-glymur-v6-4-f6a99cb43a24@oss.qualcomm.com>
On Fri, May 15, 2026 at 04:51:19PM +0530, Vishnu Reddy wrote:
> The current maxItems constraints for clocks and power-domains in the
> common venus schema were sized for platforms available at the time of
> authoring. The glymur platform introduces a dual core architecture
> that requires more clocks and power domains, exceeding these limits.
>
> Raise maxItems for clocks, clock-names, power-domains and
> power-domain-names to accommodate the glymur platform.
>
> The glymur platform-specific schema have fixed constraints for these
> properties, so the common schema only acts as an upper bound.
>
> Signed-off-by: Vishnu Reddy <busanna.reddy@oss.qualcomm.com>
> ---
> Documentation/devicetree/bindings/media/qcom,venus-common.yaml | 8 ++++----
This commit alone makes no sense. Do not introduce commits which have no
meaning. Why it has no meaning? Because imagine we stop the patchset
here and we do not apply the rest.
Squash the patches.
Best regards,
Krzysztof
next prev parent reply other threads:[~2026-05-16 9:31 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-05-15 11:21 [PATCH v6 00/14] media: iris: Add support for glymur platform Vishnu Reddy
2026-05-15 11:21 ` [PATCH v6 01/14] media: iris: Add iris vpu bus support Vishnu Reddy
2026-05-15 11:21 ` [PATCH v6 02/14] iommu: Add iris-vpu-bus to iommu_buses Vishnu Reddy
2026-05-15 11:21 ` [PATCH v6 03/14] media: iris: Fix VM count passed to firmware Vishnu Reddy
2026-05-15 11:21 ` [PATCH v6 04/14] dt-bindings: media: qcom,venus-common: Raise maxItems for clocks and power-domains Vishnu Reddy
2026-05-16 9:31 ` Krzysztof Kozlowski [this message]
2026-05-15 11:21 ` [PATCH v6 05/14] dt-bindings: media: qcom,glymur-iris: Add glymur video codec Vishnu Reddy
2026-05-15 11:21 ` [PATCH v6 06/14] media: iris: Add context bank hooks for platform specific initialization Vishnu Reddy
2026-05-15 11:21 ` [PATCH v6 07/14] media: iris: Enable Secure PAS support with IOMMU managed by Linux Vishnu Reddy
2026-05-15 11:21 ` [PATCH v6 08/14] media: iris: Rename clock and power domain macros to use vcodec suffix Vishnu Reddy
2026-05-15 11:21 ` [PATCH v6 09/14] media: iris: Use power domain type to look up pd_devs index Vishnu Reddy
2026-05-15 11:21 ` [PATCH v6 10/14] media: iris: Add power sequence for Glymur Vishnu Reddy
2026-05-15 11:21 ` [PATCH v6 11/14] media: iris: Add support to select core for dual core platforms Vishnu Reddy
2026-05-15 11:21 ` [PATCH v6 12/14] media: iris: Add platform data for glymur Vishnu Reddy
2026-05-15 11:21 ` [PATCH v6 13/14] arm64: dts: qcom: glymur: Add iris video node Vishnu Reddy
2026-05-15 11:21 ` [PATCH v6 14/14] arm64: dts: qcom: glymur-crd: Enable iris video codec node Vishnu Reddy
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=20260516-bald-versatile-wallaby-bcac6c@quoll \
--to=krzk@kernel.org \
--cc=abhinav.kumar@linux.dev \
--cc=andersson@kernel.org \
--cc=bod@kernel.org \
--cc=busanna.reddy@oss.qualcomm.com \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=dikshita.agarwal@oss.qualcomm.com \
--cc=hverkuil@kernel.org \
--cc=iommu@lists.linux.dev \
--cc=joro@8bytes.org \
--cc=konradybcio@kernel.org \
--cc=krzk+dt@kernel.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=mchehab@kernel.org \
--cc=robh@kernel.org \
--cc=robin.murphy@arm.com \
--cc=stanimir.k.varbanov@gmail.com \
--cc=stefan.schmidt@linaro.org \
--cc=vikash.garodia@oss.qualcomm.com \
--cc=will@kernel.org \
/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