From: Yixun Lan <dlan@kernel.org>
To: Andre Heider <a.heider@gmail.com>
Cc: Paul Walmsley <pjw@kernel.org>,
Palmer Dabbelt <palmer@dabbelt.com>,
Albert Ou <aou@eecs.berkeley.edu>,
Alexandre Ghiti <alex@ghiti.fr>, Yixun Lan <dlan@gentoo.org>,
linux-riscv@lists.infradead.org, spacemit@lists.linux.dev
Subject: Re: [PATCH v3 0/6] spacemit: k1-musepi-pro: add missing devices
Date: Wed, 13 May 2026 07:46:44 +0000 [thread overview]
Message-ID: <20260513074644-GKA3624092@kernel.org> (raw)
In-Reply-To: <20260513071958.29574-1-a.heider@gmail.com>
Andre,
Thanks for pushing this new version, overall, this series looks good to me,
Although there are few quite trivial issues, which I could help to fix it while
applying, let's give community some time to test and review..
P.S I haven't run it on musepi board, but trust you tested all functions..
On 09:19 Wed 13 May , Andre Heider wrote:
> Add the missing devices with their required regulators. The regulators are named after
> their label in the vendor schematics[0].
>
> v3:
> - drop reg_usb_vbus, non-controllable and served only as vin-supply
> - fixed regulator-vcc-4v0 typo
> - fold addition of pmic regulator labels into the pmic patch
> - eeprom is a GT24C02B, fixed commit message
> - added reg_qspi_vcc1v833
> - fixed uboot nor name offset
> - add missing "reg_" prefix for reg_pcie_vcc_3v3
> - drop pcie{1,2}'s vpcie3v3-supply, it's already specified on the root ports
>
> v2:
> - drop reg_vcc_5v0_sys as per "non-controllable" review comments on
> similar patches for other boards (kept the root reg_usb_vbus for though)
> - add reg_vcc_4v0 instead as p1 vin?-supply, misread the schematics
> - drop mpcie_vcc_3v3 and use pcie_vcc_3v3 for pcie2* too
> ("non-controllable" again)
> - all of the above makes it consistent with other k1 boards
>
> v1: https://lore.kernel.org/spacemit/20260505160623.1369347-1-a.heider@gmail.com/T/#t
>
> $ cat /sys/kernel/debug/regulator/regulator_summary
> regulator use open bypass opmode voltage current min max
> ---------------------------------------------------------------------------------------
> regulator-dummy 1 0 0 unknown 0mV 0mA 0mV 0mV
> PCIE_VCC3V3 5 4 0 unknown 3300mV 0mA 3300mV 3300mV
> ca400000.pcie:pcie@0-vpcie3v3 1 0mA 0mV 0mV
> ca800000.pcie:pcie@0-vpcie3v3 1 0mA 0mV 0mV
> ca800000.pcie-vpcie3v3 1 0mA 0mV 0mV
> ca400000.pcie-vpcie3v3 1 0mA 0mV 0mV
> VCC4V0 8 10 0 unknown 4000mV 0mA 4000mV 4000mV
> buck1 1 0 0 unknown 1050mV 0mA 500mV 3450mV
> buck2 1 0 0 unknown 900mV 0mA 500mV 3450mV
> buck3 1 1 0 unknown 1800mV 0mA 500mV 1800mV
> 2-0050-vcc 0 0mA 0mV 0mV
> buck4 2 2 0 unknown 3300mV 0mA 500mV 3300mV
> d4280000.mmc-vmmc 0 0mA 3300mV 3300mV
> QSPI_VCC1833 2 1 0 unknown 1833mV 0mA 1833mV 1833mV
> spi0.0-vcc 1 0mA 0mV 0mV
> buck5 3 7 0 unknown 2100mV 0mA 500mV 3450mV
> dldo1 0 0 0 unknown 1200mV 0mA 500mV 3400mV
> dldo2 0 0 0 unknown 500mV 0mA 500mV 3400mV
> dldo3 0 0 0 unknown 500mV 0mA 500mV 3400mV
> dldo4 1 0 0 unknown 1800mV 0mA 500mV 3400mV
> dldo5 0 0 0 unknown 500mV 0mA 500mV 3400mV
> dldo6 1 0 0 unknown 1800mV 0mA 500mV 3400mV
> dldo7 0 0 0 unknown 500mV 0mA 500mV 3400mV
> buck6 1 0 0 unknown 1100mV 0mA 500mV 3450mV
> aldo1 1 1 0 unknown 3300mV 0mA 500mV 3400mV
> d4280000.mmc-vqmmc 1 0mA 0mV 0mV
> aldo2 0 0 0 unknown 500mV 0mA 500mV 3400mV
> aldo3 0 0 0 unknown 500mV 0mA 500mV 3400mV
> aldo4 0 0 0 unknown 500mV 0mA 500mV 3400mV
> 5V_VBUS 2 1 0 unknown 5000mV 0mA 5000mV 5000mV
> c0a00000.usb-vbus 1 0mA 0mV 0mV
> VCC5V0_HUB 2 1 0 unknown 5000mV 0mA 5000mV 5000mV
> c0a00000.usb:hub@1-vdd 1 0mA 0mV 0mV
>
> [0] https://github.com/spacemit-com/docs-product/blob/main/en/k1_muse_pi_pro/pi_pro_hw.md
>
> Andre Heider (6):
> riscv: dts: spacemit: k1-musepi-pro: add PMIC and power infrastructure
> riscv: dts: spacemit: k1-musepi-pro: add 24c02 eeprom
> riscv: dts: spacemit: k1-musepi-pro: enable QSPI and add SPI NOR
> riscv: dts: spacemit: k1-musepi-pro: enable USB 3 ports
> riscv: dts: spacemit: k1-musepi-pro: enable PCIe ports
> riscv: dts: spacemit: k1-musepi-pro: set default console baud rate
>
> .../riscv/boot/dts/spacemit/k1-musepi-pro.dts | 309 +++++++++++++++++-
> 1 file changed, 308 insertions(+), 1 deletion(-)
>
> --
> 2.53.0
>
--
Yixun Lan (dlan)
_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv
WARNING: multiple messages have this Message-ID (diff)
From: Yixun Lan <dlan@kernel.org>
To: Andre Heider <a.heider@gmail.com>
Cc: Paul Walmsley <pjw@kernel.org>,
Palmer Dabbelt <palmer@dabbelt.com>,
Albert Ou <aou@eecs.berkeley.edu>,
Alexandre Ghiti <alex@ghiti.fr>, Yixun Lan <dlan@gentoo.org>,
linux-riscv@lists.infradead.org, spacemit@lists.linux.dev
Subject: Re: [PATCH v3 0/6] spacemit: k1-musepi-pro: add missing devices
Date: Wed, 13 May 2026 07:46:44 +0000 [thread overview]
Message-ID: <20260513074644-GKA3624092@kernel.org> (raw)
In-Reply-To: <20260513071958.29574-1-a.heider@gmail.com>
Andre,
Thanks for pushing this new version, overall, this series looks good to me,
Although there are few quite trivial issues, which I could help to fix it while
applying, let's give community some time to test and review..
P.S I haven't run it on musepi board, but trust you tested all functions..
On 09:19 Wed 13 May , Andre Heider wrote:
> Add the missing devices with their required regulators. The regulators are named after
> their label in the vendor schematics[0].
>
> v3:
> - drop reg_usb_vbus, non-controllable and served only as vin-supply
> - fixed regulator-vcc-4v0 typo
> - fold addition of pmic regulator labels into the pmic patch
> - eeprom is a GT24C02B, fixed commit message
> - added reg_qspi_vcc1v833
> - fixed uboot nor name offset
> - add missing "reg_" prefix for reg_pcie_vcc_3v3
> - drop pcie{1,2}'s vpcie3v3-supply, it's already specified on the root ports
>
> v2:
> - drop reg_vcc_5v0_sys as per "non-controllable" review comments on
> similar patches for other boards (kept the root reg_usb_vbus for though)
> - add reg_vcc_4v0 instead as p1 vin?-supply, misread the schematics
> - drop mpcie_vcc_3v3 and use pcie_vcc_3v3 for pcie2* too
> ("non-controllable" again)
> - all of the above makes it consistent with other k1 boards
>
> v1: https://lore.kernel.org/spacemit/20260505160623.1369347-1-a.heider@gmail.com/T/#t
>
> $ cat /sys/kernel/debug/regulator/regulator_summary
> regulator use open bypass opmode voltage current min max
> ---------------------------------------------------------------------------------------
> regulator-dummy 1 0 0 unknown 0mV 0mA 0mV 0mV
> PCIE_VCC3V3 5 4 0 unknown 3300mV 0mA 3300mV 3300mV
> ca400000.pcie:pcie@0-vpcie3v3 1 0mA 0mV 0mV
> ca800000.pcie:pcie@0-vpcie3v3 1 0mA 0mV 0mV
> ca800000.pcie-vpcie3v3 1 0mA 0mV 0mV
> ca400000.pcie-vpcie3v3 1 0mA 0mV 0mV
> VCC4V0 8 10 0 unknown 4000mV 0mA 4000mV 4000mV
> buck1 1 0 0 unknown 1050mV 0mA 500mV 3450mV
> buck2 1 0 0 unknown 900mV 0mA 500mV 3450mV
> buck3 1 1 0 unknown 1800mV 0mA 500mV 1800mV
> 2-0050-vcc 0 0mA 0mV 0mV
> buck4 2 2 0 unknown 3300mV 0mA 500mV 3300mV
> d4280000.mmc-vmmc 0 0mA 3300mV 3300mV
> QSPI_VCC1833 2 1 0 unknown 1833mV 0mA 1833mV 1833mV
> spi0.0-vcc 1 0mA 0mV 0mV
> buck5 3 7 0 unknown 2100mV 0mA 500mV 3450mV
> dldo1 0 0 0 unknown 1200mV 0mA 500mV 3400mV
> dldo2 0 0 0 unknown 500mV 0mA 500mV 3400mV
> dldo3 0 0 0 unknown 500mV 0mA 500mV 3400mV
> dldo4 1 0 0 unknown 1800mV 0mA 500mV 3400mV
> dldo5 0 0 0 unknown 500mV 0mA 500mV 3400mV
> dldo6 1 0 0 unknown 1800mV 0mA 500mV 3400mV
> dldo7 0 0 0 unknown 500mV 0mA 500mV 3400mV
> buck6 1 0 0 unknown 1100mV 0mA 500mV 3450mV
> aldo1 1 1 0 unknown 3300mV 0mA 500mV 3400mV
> d4280000.mmc-vqmmc 1 0mA 0mV 0mV
> aldo2 0 0 0 unknown 500mV 0mA 500mV 3400mV
> aldo3 0 0 0 unknown 500mV 0mA 500mV 3400mV
> aldo4 0 0 0 unknown 500mV 0mA 500mV 3400mV
> 5V_VBUS 2 1 0 unknown 5000mV 0mA 5000mV 5000mV
> c0a00000.usb-vbus 1 0mA 0mV 0mV
> VCC5V0_HUB 2 1 0 unknown 5000mV 0mA 5000mV 5000mV
> c0a00000.usb:hub@1-vdd 1 0mA 0mV 0mV
>
> [0] https://github.com/spacemit-com/docs-product/blob/main/en/k1_muse_pi_pro/pi_pro_hw.md
>
> Andre Heider (6):
> riscv: dts: spacemit: k1-musepi-pro: add PMIC and power infrastructure
> riscv: dts: spacemit: k1-musepi-pro: add 24c02 eeprom
> riscv: dts: spacemit: k1-musepi-pro: enable QSPI and add SPI NOR
> riscv: dts: spacemit: k1-musepi-pro: enable USB 3 ports
> riscv: dts: spacemit: k1-musepi-pro: enable PCIe ports
> riscv: dts: spacemit: k1-musepi-pro: set default console baud rate
>
> .../riscv/boot/dts/spacemit/k1-musepi-pro.dts | 309 +++++++++++++++++-
> 1 file changed, 308 insertions(+), 1 deletion(-)
>
> --
> 2.53.0
>
--
Yixun Lan (dlan)
next prev parent reply other threads:[~2026-05-13 7:47 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-05-13 7:19 [PATCH v3 0/6] spacemit: k1-musepi-pro: add missing devices Andre Heider
2026-05-13 7:19 ` Andre Heider
2026-05-13 7:19 ` [PATCH v3 1/6] riscv: dts: spacemit: k1-musepi-pro: add PMIC and power infrastructure Andre Heider
2026-05-13 7:19 ` Andre Heider
2026-05-13 7:48 ` Yixun Lan
2026-05-13 7:48 ` Yixun Lan
2026-05-13 8:02 ` Yixun Lan
2026-05-13 8:02 ` Yixun Lan
2026-05-14 2:28 ` sashiko-bot
2026-05-13 7:19 ` [PATCH v3 2/6] riscv: dts: spacemit: k1-musepi-pro: add 24c02 eeprom Andre Heider
2026-05-13 7:19 ` Andre Heider
2026-05-13 7:19 ` [PATCH v3 3/6] riscv: dts: spacemit: k1-musepi-pro: enable QSPI and add SPI NOR Andre Heider
2026-05-13 7:19 ` Andre Heider
2026-05-14 2:53 ` sashiko-bot
2026-05-13 7:19 ` [PATCH v3 4/6] riscv: dts: spacemit: k1-musepi-pro: enable USB 3 ports Andre Heider
2026-05-13 7:19 ` Andre Heider
2026-05-13 7:19 ` [PATCH v3 5/6] riscv: dts: spacemit: k1-musepi-pro: enable PCIe ports Andre Heider
2026-05-13 7:19 ` Andre Heider
2026-05-13 7:19 ` [PATCH v3 6/6] riscv: dts: spacemit: k1-musepi-pro: set default console baud rate Andre Heider
2026-05-13 7:19 ` Andre Heider
2026-05-13 7:46 ` Yixun Lan [this message]
2026-05-13 7:46 ` [PATCH v3 0/6] spacemit: k1-musepi-pro: add missing devices Yixun Lan
2026-05-13 7:56 ` Andre Heider
2026-05-13 7:56 ` Andre Heider
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=20260513074644-GKA3624092@kernel.org \
--to=dlan@kernel.org \
--cc=a.heider@gmail.com \
--cc=alex@ghiti.fr \
--cc=aou@eecs.berkeley.edu \
--cc=dlan@gentoo.org \
--cc=linux-riscv@lists.infradead.org \
--cc=palmer@dabbelt.com \
--cc=pjw@kernel.org \
--cc=spacemit@lists.linux.dev \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.