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
next prev parent reply other threads:[~2026-05-13 7:47 UTC|newest]
Thread overview: 11+ 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 ` [PATCH v3 1/6] riscv: dts: spacemit: k1-musepi-pro: add PMIC and power infrastructure Andre Heider
2026-05-13 7:48 ` Yixun Lan
2026-05-13 8:02 ` Yixun Lan
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 ` [PATCH v3 3/6] riscv: dts: spacemit: k1-musepi-pro: enable QSPI and add SPI NOR Andre Heider
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 ` [PATCH v3 5/6] riscv: dts: spacemit: k1-musepi-pro: enable PCIe ports 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:46 ` Yixun Lan [this message]
2026-05-13 7:56 ` [PATCH v3 0/6] spacemit: k1-musepi-pro: add missing devices 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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox