| Message ID | D2DCF9E2F70EDC93+20240912025539.1928223-3-wangyuli@uniontech.com (mailing list archive) |
|---|---|
| State | Not Applicable |
| Headers | show |
| Series | [6.6,v2,1/4] riscv: dts: starfive: add assigned-clock* to limit frquency | expand |
| Context | Check | Description |
|---|---|---|
| netdev/tree_selection | success | Not a local patch |
> On 12.09.24 10:55, WangYuli wrote: > > From: Xingyu Wu <xingyu.wu@starfivetech.com> > > [ Upstream commit 92cfc35838b2a4006abb9e3bafc291b56f135d01 ] > > Add I2Srx/I2Stx0/I2Stx1 nodes and pins configuration for the StarFive JH7110 > SoC. > > Signed-off-by: Xingyu Wu <xingyu.wu@starfivetech.com> > Reviewed-by: Walker Chen <walker.chen@starfivetech.com> > Signed-off-by: Conor Dooley <conor.dooley@microchip.com> > Signed-off-by: WangYuli <wangyuli@uniontech.com> > --- > .../jh7110-starfive-visionfive-2.dtsi | 58 +++++++++++++++++ > arch/riscv/boot/dts/starfive/jh7110.dtsi | 65 +++++++++++++++++++ > 2 files changed, 123 insertions(+) > > diff --git a/arch/riscv/boot/dts/starfive/jh7110-starfive-visionfive-2.dtsi > b/arch/riscv/boot/dts/starfive/jh7110-starfive-visionfive-2.dtsi > index 4874e3bb42ab..caa59b9b2f19 100644 > --- a/arch/riscv/boot/dts/starfive/jh7110-starfive-visionfive-2.dtsi > +++ b/arch/riscv/boot/dts/starfive/jh7110-starfive-visionfive-2.dtsi > @@ -202,6 +202,24 @@ &i2c6 { > status = "okay"; > }; > > +&i2srx { > + pinctrl-names = "default"; > + pinctrl-0 = <&i2srx_pins>; > + status = "okay"; > +}; > + > +&i2stx0 { > + pinctrl-names = "default"; > + pinctrl-0 = <&mclk_ext_pins>; > + status = "okay"; > +}; > + > +&i2stx1 { > + pinctrl-names = "default"; > + pinctrl-0 = <&i2stx1_pins>; > + status = "okay"; > +}; > + > &mmc0 { > max-frequency = <100000000>; > assigned-clocks = <&syscrg JH7110_SYSCLK_SDIO0_SDCARD>; @@ - > 340,6 +358,46 @@ GPOEN_SYS_I2C6_DATA, > }; > }; > > + 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/ Best regards, Hal > diff --git a/arch/riscv/boot/dts/starfive/jh7110.dtsi > b/arch/riscv/boot/dts/starfive/jh7110.dtsi > index e85464c328d0..621b68c02ea8 100644 > --- a/arch/riscv/boot/dts/starfive/jh7110.dtsi > +++ b/arch/riscv/boot/dts/starfive/jh7110.dtsi > @@ -512,6 +512,30 @@ tdm: tdm@10090000 { > status = "disabled"; > }; > > + i2srx: i2s@100e0000 { > + compatible = "starfive,jh7110-i2srx"; > + reg = <0x0 0x100e0000 0x0 0x1000>; > + clocks = <&syscrg JH7110_SYSCLK_I2SRX_BCLK_MST>, > + <&syscrg JH7110_SYSCLK_I2SRX_APB>, > + <&syscrg JH7110_SYSCLK_MCLK>, > + <&syscrg JH7110_SYSCLK_MCLK_INNER>, > + <&mclk_ext>, > + <&syscrg JH7110_SYSCLK_I2SRX_BCLK>, > + <&syscrg JH7110_SYSCLK_I2SRX_LRCK>, > + <&i2srx_bclk_ext>, > + <&i2srx_lrck_ext>; > + clock-names = "i2sclk", "apb", "mclk", > + "mclk_inner", "mclk_ext", "bclk", > + "lrck", "bclk_ext", "lrck_ext"; > + resets = <&syscrg JH7110_SYSRST_I2SRX_APB>, > + <&syscrg JH7110_SYSRST_I2SRX_BCLK>; > + dmas = <0>, <&dma 24>; > + dma-names = "tx", "rx"; > + starfive,syscon = <&sys_syscon 0x18 0x2>; > + #sound-dai-cells = <0>; > + status = "disabled"; > + }; > + > usb0: usb@10100000 { > compatible = "starfive,jh7110-usb"; > ranges = <0x0 0x0 0x10100000 0x100000>; @@ - > 736,6 +760,47 @@ spi6: spi@120a0000 { > status = "disabled"; > }; > > + i2stx0: i2s@120b0000 { > + compatible = "starfive,jh7110-i2stx0"; > + reg = <0x0 0x120b0000 0x0 0x1000>; > + clocks = <&syscrg > JH7110_SYSCLK_I2STX0_BCLK_MST>, > + <&syscrg JH7110_SYSCLK_I2STX0_APB>, > + <&syscrg JH7110_SYSCLK_MCLK>, > + <&syscrg JH7110_SYSCLK_MCLK_INNER>, > + <&mclk_ext>; > + clock-names = "i2sclk", "apb", "mclk", > + "mclk_inner","mclk_ext"; > + resets = <&syscrg JH7110_SYSRST_I2STX0_APB>, > + <&syscrg JH7110_SYSRST_I2STX0_BCLK>; > + dmas = <&dma 47>; > + dma-names = "tx"; > + #sound-dai-cells = <0>; > + status = "disabled"; > + }; > + > + i2stx1: i2s@120c0000 { > + compatible = "starfive,jh7110-i2stx1"; > + reg = <0x0 0x120c0000 0x0 0x1000>; > + clocks = <&syscrg > JH7110_SYSCLK_I2STX1_BCLK_MST>, > + <&syscrg JH7110_SYSCLK_I2STX1_APB>, > + <&syscrg JH7110_SYSCLK_MCLK>, > + <&syscrg JH7110_SYSCLK_MCLK_INNER>, > + <&mclk_ext>, > + <&syscrg JH7110_SYSCLK_I2STX1_BCLK>, > + <&syscrg JH7110_SYSCLK_I2STX1_LRCK>, > + <&i2stx_bclk_ext>, > + <&i2stx_lrck_ext>; > + clock-names = "i2sclk", "apb", "mclk", > + "mclk_inner", "mclk_ext", "bclk", > + "lrck", "bclk_ext", "lrck_ext"; > + resets = <&syscrg JH7110_SYSRST_I2STX1_APB>, > + <&syscrg JH7110_SYSRST_I2STX1_BCLK>; > + dmas = <&dma 48>; > + dma-names = "tx"; > + #sound-dai-cells = <0>; > + status = "disabled"; > + }; > + > sfctemp: temperature-sensor@120e0000 { > compatible = "starfive,jh7110-temp"; > reg = <0x0 0x120e0000 0x0 0x10000>; > -- > 2.43.4
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.
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,
On Thu, Sep 12, 2024 at 07:19:05PM +0800, WangYuli wrote: > 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. > > > Got it. Thanks for pointing that out, and sorry for bothering you all... Patch 1 still seems like it could be backported though, even if these pwmdac patches are not suitble?
diff --git a/arch/riscv/boot/dts/starfive/jh7110-starfive-visionfive-2.dtsi b/arch/riscv/boot/dts/starfive/jh7110-starfive-visionfive-2.dtsi index 4874e3bb42ab..caa59b9b2f19 100644 --- a/arch/riscv/boot/dts/starfive/jh7110-starfive-visionfive-2.dtsi +++ b/arch/riscv/boot/dts/starfive/jh7110-starfive-visionfive-2.dtsi @@ -202,6 +202,24 @@ &i2c6 { status = "okay"; }; +&i2srx { + pinctrl-names = "default"; + pinctrl-0 = <&i2srx_pins>; + status = "okay"; +}; + +&i2stx0 { + pinctrl-names = "default"; + pinctrl-0 = <&mclk_ext_pins>; + status = "okay"; +}; + +&i2stx1 { + pinctrl-names = "default"; + pinctrl-0 = <&i2stx1_pins>; + status = "okay"; +}; + &mmc0 { max-frequency = <100000000>; assigned-clocks = <&syscrg JH7110_SYSCLK_SDIO0_SDCARD>; @@ -340,6 +358,46 @@ GPOEN_SYS_I2C6_DATA, }; }; + 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, diff --git a/arch/riscv/boot/dts/starfive/jh7110.dtsi b/arch/riscv/boot/dts/starfive/jh7110.dtsi index e85464c328d0..621b68c02ea8 100644 --- a/arch/riscv/boot/dts/starfive/jh7110.dtsi +++ b/arch/riscv/boot/dts/starfive/jh7110.dtsi @@ -512,6 +512,30 @@ tdm: tdm@10090000 { status = "disabled"; }; + i2srx: i2s@100e0000 { + compatible = "starfive,jh7110-i2srx"; + reg = <0x0 0x100e0000 0x0 0x1000>; + clocks = <&syscrg JH7110_SYSCLK_I2SRX_BCLK_MST>, + <&syscrg JH7110_SYSCLK_I2SRX_APB>, + <&syscrg JH7110_SYSCLK_MCLK>, + <&syscrg JH7110_SYSCLK_MCLK_INNER>, + <&mclk_ext>, + <&syscrg JH7110_SYSCLK_I2SRX_BCLK>, + <&syscrg JH7110_SYSCLK_I2SRX_LRCK>, + <&i2srx_bclk_ext>, + <&i2srx_lrck_ext>; + clock-names = "i2sclk", "apb", "mclk", + "mclk_inner", "mclk_ext", "bclk", + "lrck", "bclk_ext", "lrck_ext"; + resets = <&syscrg JH7110_SYSRST_I2SRX_APB>, + <&syscrg JH7110_SYSRST_I2SRX_BCLK>; + dmas = <0>, <&dma 24>; + dma-names = "tx", "rx"; + starfive,syscon = <&sys_syscon 0x18 0x2>; + #sound-dai-cells = <0>; + status = "disabled"; + }; + usb0: usb@10100000 { compatible = "starfive,jh7110-usb"; ranges = <0x0 0x0 0x10100000 0x100000>; @@ -736,6 +760,47 @@ spi6: spi@120a0000 { status = "disabled"; }; + i2stx0: i2s@120b0000 { + compatible = "starfive,jh7110-i2stx0"; + reg = <0x0 0x120b0000 0x0 0x1000>; + clocks = <&syscrg JH7110_SYSCLK_I2STX0_BCLK_MST>, + <&syscrg JH7110_SYSCLK_I2STX0_APB>, + <&syscrg JH7110_SYSCLK_MCLK>, + <&syscrg JH7110_SYSCLK_MCLK_INNER>, + <&mclk_ext>; + clock-names = "i2sclk", "apb", "mclk", + "mclk_inner","mclk_ext"; + resets = <&syscrg JH7110_SYSRST_I2STX0_APB>, + <&syscrg JH7110_SYSRST_I2STX0_BCLK>; + dmas = <&dma 47>; + dma-names = "tx"; + #sound-dai-cells = <0>; + status = "disabled"; + }; + + i2stx1: i2s@120c0000 { + compatible = "starfive,jh7110-i2stx1"; + reg = <0x0 0x120c0000 0x0 0x1000>; + clocks = <&syscrg JH7110_SYSCLK_I2STX1_BCLK_MST>, + <&syscrg JH7110_SYSCLK_I2STX1_APB>, + <&syscrg JH7110_SYSCLK_MCLK>, + <&syscrg JH7110_SYSCLK_MCLK_INNER>, + <&mclk_ext>, + <&syscrg JH7110_SYSCLK_I2STX1_BCLK>, + <&syscrg JH7110_SYSCLK_I2STX1_LRCK>, + <&i2stx_bclk_ext>, + <&i2stx_lrck_ext>; + clock-names = "i2sclk", "apb", "mclk", + "mclk_inner", "mclk_ext", "bclk", + "lrck", "bclk_ext", "lrck_ext"; + resets = <&syscrg JH7110_SYSRST_I2STX1_APB>, + <&syscrg JH7110_SYSRST_I2STX1_BCLK>; + dmas = <&dma 48>; + dma-names = "tx"; + #sound-dai-cells = <0>; + status = "disabled"; + }; + sfctemp: temperature-sensor@120e0000 { compatible = "starfive,jh7110-temp"; reg = <0x0 0x120e0000 0x0 0x10000>;