From: Krzysztof Kozlowski <krzk@kernel.org>
To: Paul Elder <paul.elder@ideasonboard.com>,
Paul Elder <paul.elder+renesas@ideasonboard.com>
Cc: Geert Uytterhoeven <geert+renesas@glider.be>,
Stephen Boyd <sboyd@kernel.org>,
Brian Masney <bmasney+clk@redhat.com>,
Jerome Brunet <jbrunet+clk@baylibre.com>,
Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
Magnus Damm <magnus.damm@gmail.com>,
Ulf Hansson <ulfh@kernel.org>,
Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
Mauro Carvalho Chehab <mchehab@kernel.org>,
Philipp Zabel <p.zabel@pengutronix.de>,
Kieran Bingham <kieran.bingham+renesas@ideasonboard.com>,
linux-renesas-soc@vger.kernel.org, linux-clk@vger.kernel.org,
linux-kernel@vger.kernel.org, devicetree@vger.kernel.org,
linux-pm@vger.kernel.org, linux-media@vger.kernel.org
Subject: Re: [PATCH v4 01/13] dt-bindings: clock: renesas,r8a78000-cpg: Add CPG_SGDn_VIO_DP_{TX,OTHER}
Date: Wed, 11 Nov 2026 10:16:38 +0100 [thread overview]
Message-ID: <e179b228-3f18-4b99-9c54-93699d13e63b@kernel.org> (raw)
In-Reply-To: <179438844820.272237.18262716067322018404@neptunite.rasen.tech>
On 11/11/2026 10:14, Paul Elder wrote:
> Quoting Krzysztof Kozlowski (2026-11-10 18:30:50)
>> On Mon, Nov 09, 2026 at 10:34:22PM +0900, Paul Elder wrote:
>>> Add the CPG_SGDn_VIO_DP_TX and CPG_SGDn_VIO_DP_OTHER clocks that are
>>> used by FCPV, VSP, VCON, and DPTX cores.
>>>
>>> It should be noted that there is likely a discrepancy regarding the
>>> clocks. In practice it ought not to be an issue as all the clocks derive
>>> from the same SGD1_VIO_MAIN clock and none of them are programmable or
>>> gateable.
>>>
>>> According to the datasheet, the FCPV[BID] and VSPD cores use
>>> SGD*_VIO_DP_TXφ while the VSP[BI] cores use SGD*_VIO_DP_OTHERφ.
>>> However, educated guessing suggests that IP cores in the same pipeline
>>> should be using the same clocks:
>>> - FCPV[BI] -> VSP[BI]: SGD*_VIO_DP_OTHERφ
>>> - FCPVD -> VSPD -> VCON -> DPTX: SGD*_VIO_DP_TXφ
>>>
>>> Signed-off-by: Paul Elder <paul.elder+renesas@ideasonboard.com>
>>>
>>> ---
>>> Changes in v4:
>>> - Add all the clocks
>>> - remove DNI
>>
>> What is DNI?
>
> Do not integrate. Most of the patches here were DNI in previous versions
> because they depended on a lot of downstream things, but now those dependencies
> are in the process of upstreaming so these can graduate from DNI status.
Upstream clock indices ABI does not depend on downstream at all, so
seeing this unmergeable is just confusing.
Best regards,
Krzysztof
next prev parent reply other threads:[~2026-11-11 9:16 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-11-09 13:34 [PATCH v4 00/13] media: rcar-fcp, vsp1: Enable on R-Car X5H Paul Elder
2026-11-09 13:34 ` [PATCH v4 01/13] dt-bindings: clock: renesas,r8a78000-cpg: Add CPG_SGDn_VIO_DP_{TX,OTHER} Paul Elder
2026-11-10 9:30 ` Krzysztof Kozlowski
2026-11-11 9:14 ` Paul Elder
2026-11-11 9:16 ` Krzysztof Kozlowski [this message]
2026-11-11 9:16 ` Krzysztof Kozlowski
2026-11-09 13:34 ` [PATCH v4 02/13] pmdomain: renesas: r8a78000: Add SCMI maps for VIO Paul Elder
2026-11-09 13:34 ` [PATCH v4 03/13] arm64: renesas: r8a78000: Add FCPV[BI] nodes Paul Elder
2026-11-09 13:34 ` [PATCH v4 04/13] arm64: renesas: r8a78000: Add VSP[BI] nodes Paul Elder
2026-11-09 13:34 ` [PATCH v4 05/13] arm64: renesas: r8a78000: Add FCPVD and VSPD nodes Paul Elder
2026-11-09 13:34 ` [PATCH v4 06/13] media: vsp1: Support full VSPB on R-Car M3-W, M3-N and E3 Paul Elder
2026-11-09 13:34 ` [PATCH v4 07/13] media: vsp1: Add device info entries for VSPD on X5H Paul Elder
2026-11-09 13:34 ` [PATCH v4 08/13] media: rcar-fcp: Handle resets Paul Elder
2026-11-09 13:34 ` [PATCH v4 09/13] media: vsp1: Fix access to reserved registers Paul Elder
2026-11-09 13:34 ` [PATCH v4 10/13] media: vsp1: Fix du_complete race condition issue Paul Elder
2026-11-09 13:34 ` [PATCH v4 11/13] dt-bindings: Increase Renesas X5H MDLC PD specifier bit width Paul Elder
2026-11-09 13:34 ` [PATCH v4 12/13] media: vsp1: Complete buffers as queued in start_streaming error path Paul Elder
2026-11-09 13:34 ` [PATCH DNI v4 13/13] media: rcar-fcp: Check device revision at probe time Paul Elder
2026-11-09 14:10 ` Geert Uytterhoeven
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=e179b228-3f18-4b99-9c54-93699d13e63b@kernel.org \
--to=krzk@kernel.org \
--cc=bmasney+clk@redhat.com \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=geert+renesas@glider.be \
--cc=jbrunet+clk@baylibre.com \
--cc=kieran.bingham+renesas@ideasonboard.com \
--cc=krzk+dt@kernel.org \
--cc=laurent.pinchart@ideasonboard.com \
--cc=linux-clk@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=linux-renesas-soc@vger.kernel.org \
--cc=magnus.damm@gmail.com \
--cc=mchehab@kernel.org \
--cc=p.zabel@pengutronix.de \
--cc=paul.elder+renesas@ideasonboard.com \
--cc=paul.elder@ideasonboard.com \
--cc=robh@kernel.org \
--cc=sboyd@kernel.org \
--cc=ulfh@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
all inboxes | Powered by JetHome®