[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <57B002D3A0A289D5+d784ecd6-ed84-4aa1-ae58-0c67d2de1989@uniontech.com>
Date: Thu, 12 Sep 2024 19:19:05 +0800
From: WangYuli <wangyuli@...ontech.com>
To: Conor Dooley <conor.dooley@...rochip.com>,
Hal Feng <hal.feng@...rfivetech.com>
Cc: "stable@...r.kernel.org" <stable@...r.kernel.org>,
"gregkh@...uxfoundation.org" <gregkh@...uxfoundation.org>,
"sashal@...nel.org" <sashal@...nel.org>,
William Qiu <william.qiu@...rfivetech.com>,
"emil.renner.berthing@...onical.com" <emil.renner.berthing@...onical.com>,
Xingyu Wu <xingyu.wu@...rfivetech.com>,
Walker Chen <walker.chen@...rfivetech.com>, "robh@...nel.org"
<robh@...nel.org>, "kernel@...il.dk" <kernel@...il.dk>,
"robh+dt@...nel.org" <robh+dt@...nel.org>,
"krzysztof.kozlowski+dt@...aro.org" <krzysztof.kozlowski+dt@...aro.org>,
"conor+dt@...nel.org" <conor+dt@...nel.org>,
"paul.walmsley@...ive.com" <paul.walmsley@...ive.com>,
"palmer@...belt.com" <palmer@...belt.com>,
"aou@...s.berkeley.edu" <aou@...s.berkeley.edu>,
"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
"linux-riscv@...ts.infradead.org" <linux-riscv@...ts.infradead.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"richardcochran@...il.com" <richardcochran@...il.com>,
"netdev@...r.kernel.org" <netdev@...r.kernel.org>
Subject: Re: [PATCH 6.6 v2 3/4] riscv: dts: starfive: Add the nodes and pins
of I2Srx/I2Stx0/I2Stx1
On 2024/9/12 18:40, Conor Dooley wrote:
> On Thu, Sep 12, 2024 at 10:23:09AM +0000, Hal Feng wrote:
>>> On 12.09.24 10:55, WangYuli wrote:
>>> + i2srx_pins: i2srx-0 {
>>> + clk-sd-pins {
>>> + pinmux = <GPIOMUX(38, GPOUT_LOW,
>>> + GPOEN_DISABLE,
>>> + GPI_SYS_I2SRX_BCLK)>,
>>> + <GPIOMUX(63, GPOUT_LOW,
>>> + GPOEN_DISABLE,
>>> + GPI_SYS_I2SRX_LRCK)>,
>>> + <GPIOMUX(38, GPOUT_LOW,
>>> + GPOEN_DISABLE,
>>> + GPI_SYS_I2STX1_BCLK)>,
>>> + <GPIOMUX(63, GPOUT_LOW,
>>> + GPOEN_DISABLE,
>>> + GPI_SYS_I2STX1_LRCK)>,
>>> + <GPIOMUX(61, GPOUT_LOW,
>>> + GPOEN_DISABLE,
>>> + GPI_SYS_I2SRX_SDIN0)>;
>>> + input-enable;
>>> + };
>>> + };
>>> +
>>> + i2stx1_pins: i2stx1-0 {
>>> + sd-pins {
>>> + pinmux = <GPIOMUX(44, GPOUT_SYS_I2STX1_SDO0,
>>> + GPOEN_ENABLE,
>>> + GPI_NONE)>;
>>> + bias-disable;
>>> + input-disable;
>>> + };
>>> + };
>>> +
>>> + mclk_ext_pins: mclk-ext-0 {
>>> + mclk-ext-pins {
>>> + pinmux = <GPIOMUX(4, GPOUT_LOW,
>>> + GPOEN_DISABLE,
>>> + GPI_SYS_MCLK_EXT)>;
>>> + input-enable;
>>> + };
>>> + };
>>> +
>>> mmc0_pins: mmc0-0 {
>>> rst-pins {
>>> pinmux = <GPIOMUX(62, GPOUT_SYS_SDIO0_RST,
>> The above changes had been reverted in commit e0503d47e93d in the mainline.
>> Is it appropriate to merge this patch into the stable branch?
>>
>> https://lore.kernel.org/all/20240415125033.86909-1-hannah.peuckmann@canonical.com/
> Hah, I had gone looking this morning because I had a hunch that there
> was some missing fix this series didn't, but couldn't remember what it
> was. I completely forgot that some of this was non-present overlay
> related stuff that had had to be reverted.
>
> So yes, if it had to be reverted in mainline, it shouldn't get
> backported. Thanks for spotting that Hal.
>
> Cheers,
> Conor.
Got it. Thanks for pointing that out, and sorry for bothering you all...
And thank you for your patience...
Thanks,
--
WangYuli
Download attachment "OpenPGP_0xC5DA1F3046F40BEE.asc" of type "application/pgp-keys" (633 bytes)
Download attachment "OpenPGP_signature.asc" of type "application/pgp-signature" (237 bytes)
Powered by blists - more mailing lists