[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <6567146d2dfb0287e482ec1c441b31d9@manjaro.org>
Date: Sun, 20 Oct 2024 20:04:09 +0200
From: Dragan Simic <dsimic@...jaro.org>
To: Diederik de Haas <didi.debian@...ow.org>
Cc: linux-rockchip@...ts.infradead.org, heiko@...ech.de,
linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org,
devicetree@...r.kernel.org, robh@...nel.org, krzk+dt@...nel.org,
conor+dt@...nel.org
Subject: Re: [PATCH 2/3] arm64: dts: rockchip: Prepare RK356x SoC dtsi files
for per-variant OPPs
Hello Diederik,
On 2024-10-19 20:09, Diederik de Haas wrote:
> On Sat Oct 12, 2024 at 9:41 PM CEST, Diederik de Haas wrote:
>> On Sat Oct 12, 2024 at 7:04 PM CEST, Dragan Simic wrote:
>> >
>> > -&pipegrf {
>> > - compatible = "rockchip,rk3566-pipe-grf", "syscon";
>>
>> This seems unrelated?
>>
>> > +&cpu0 {
>> > + operating-points-v2 = <&cpu0_opp_table>;
>> > };
>> >
>> > -&power {
>> > - power-domain@...568_PD_PIPE {
>> > - reg = <RK3568_PD_PIPE>;
>> > - clocks = <&cru PCLK_PIPE>;
>> > - pm_qos = <&qos_pcie2x1>,
>> > - <&qos_sata1>,
>> > - <&qos_sata2>,
>> > - <&qos_usb3_0>,
>> > - <&qos_usb3_1>;
>> > - #power-domain-cells = <0>;
>> > - };
>>
>> This seems unrelated to me and possibly a functional change?
>> If this was intended, then a description in the commit message would
>> be
>> nice why this is appropriate and possibly moved to a separate patch?
>>
>> > +&cpu1 {
>> > + operating-points-v2 = <&cpu0_opp_table>;
>> > +};
>> > +
>> > +&cpu2 {
>> > + operating-points-v2 = <&cpu0_opp_table>;
>> > };
>> >
>> > -&usb_host0_xhci {
>> > - phys = <&usb2phy0_otg>;
>> > - phy-names = "usb2-phy";
>> > - extcon = <&usb2phy0>;
>> > - maximum-speed = "high-speed";
>>
>> This also looks unrelated and a functional change?
>>
>> > +&cpu3 {
>> > + operating-points-v2 = <&cpu0_opp_table>;
>> > };
>> >
>> > -&vop {
>> > - compatible = "rockchip,rk3566-vop";
>>
>> This also looks unrelated?
>
> It turns out I was wrong.
> The elements I thought were removed, aren't removed.
>
> Sorry for the noise.
No worries. I tried to tune and adjust the patch generation
parameters as best as possible, but some parts of the produced
patches still remained slightly confusing.
By the way, a few months ago there was a discussion on the Git
mailing list about making Git perform such parameter adjustment
magic itself, which back then seemed like a good idea to me,
but after dealing with a few rather complex patches myself, I
no longer think that's actually possible.
Powered by blists - more mailing lists