From: Heiko Stuebner <heiko@sntech.de>
To: Ulf Hansson <ulf.hansson@linaro.org>,
Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
Shawn Lin <shawn.lin@rock-chips.com>
Cc: linux-pm@vger.kernel.org, linux-rockchip@lists.infradead.org,
devicetree@vger.kernel.org, Shawn Lin <shawn.lin@rock-chips.com>
Subject: Re: [PATCH 0/3] Add shaping support for Rockchip platforms
Date: Fri, 06 Mar 2026 11:43:13 +0100 [thread overview]
Message-ID: <4490342.ejJDZkT8p0@phil> (raw)
In-Reply-To: <1772782832-216867-1-git-send-email-shawn.lin@rock-chips.com>
Hi Shawn,
Am Freitag, 6. März 2026, 08:40:29 Mitteleuropäische Normalzeit schrieb Shawn Lin:
> pm_shaping is introduced for some platforms to throttle the outstanding
> burst requests of each IP to prevent bus monopolization. It should be
> saved and restored along with the on/off of pmdomain, the same as pm_qos.
> This patchset adds support for it.
Can we get some more descriptive naming for the feature please?
In the commit messages, dt-binding and also node-names in the soc dtsi.
Because anyone glancing at either the binding or dtsi will not understand
what this is, and would even need to consult the git history for any
explanation.
The full interconnect is not part of the TRM I have, but the bits and
pieces in other parts of the TRM speak of traffic-shaping.
So for the node I'd go for:
shaping_vepu1: traffic-shaping@27f06088
and in the binding something like:
rockchip,pm-traffic-shaping = <....>
I think the whole pm_qos thing is an oversight of the past, because:
- properties should use dashes "-" not underscores
- the property name is Rockchip specific, so should likely get a prefix
Heiko
next prev parent reply other threads:[~2026-03-06 10:43 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-03-06 7:40 [PATCH 0/3] Add shaping support for Rockchip platforms Shawn Lin
2026-03-06 7:40 ` [PATCH 1/3] dt-bindings: power: rockchip: Add pm_shaping support Shawn Lin
2026-03-07 15:17 ` Krzysztof Kozlowski
2026-03-06 7:40 ` [PATCH 2/3] pmdomain: " Shawn Lin
2026-03-06 7:40 ` [PATCH 3/3] arm64: dts: rockchip: Add shaping nodes for rk3576 Shawn Lin
2026-03-07 15:18 ` Krzysztof Kozlowski
2026-03-06 10:43 ` Heiko Stuebner [this message]
2026-03-06 12:10 ` [PATCH 0/3] Add shaping support for Rockchip platforms Shawn Lin
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=4490342.ejJDZkT8p0@phil \
--to=heiko@sntech.de \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=krzk+dt@kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=linux-rockchip@lists.infradead.org \
--cc=robh@kernel.org \
--cc=shawn.lin@rock-chips.com \
--cc=ulf.hansson@linaro.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