* Re: [RESEND][PATCH v5] mfd: simple-mfd-i2c: add a reboot cell for the SpacemiT P1 chip [not found] ` <adPUTRAlzGBs2wQG@aurel32.net> @ 2026-04-06 21:55 ` Vincent Legoll 2026-04-08 2:48 ` Troy Mitchell 0 siblings, 1 reply; 2+ messages in thread From: Vincent Legoll @ 2026-04-06 21:55 UTC (permalink / raw) To: linux-kernel, Lee Jones, Yixun Lan, linux-riscv, spacemit Cc: Troy Mitchell, Andi Shyti, Yixun Lan, linux-i2c, linux-kernel, linux-riscv, spacemit, Troy Mitchell, Aurelien Jarno Hi Aurelien, On 4/6/26 17:42, Aurelien Jarno wrote: > On 2026-04-06 11:46, Vincent Legoll wrote: >> Tell me if there's a new iteration to test, and/or if you want me testing it >> in a different way. > Yes, as written in the original mail, this patchset depends on PIO > support in the spacemit I2C driver: > https://lore.kernel.org/lkml/20260207-b4-k3-i2c-pio-v7-2-626942d94d91@linux.spacemit.com/ > > Without it, you might get this warning message and reboot is not 100% > reliable. I still believe it's better than no reboot support at all. OK, I tested a new stack: next-20260406, PIO patchset, reset patchset & USB patchset: $ git log --oneline 5030369b32c4 (HEAD -> opirv2-pio-reset-usb) riscv: dts: spacemit: Enable USB3.0/PCIe on OrangePi RV2 73274ce37441 riscv: dts: spacemit: Define the P1 PMIC regulators for OrangePi RV2 cda12b8bdbfe riscv: dts: spacemit: Enable i2c8 adapter for OrangePi RV2 68ce409351c6 (opirv2-pio-reset) mfd: simple-mfd-i2c: add a reboot cell for the SpacemiT P1 chip 245e0b8e8a91 i2c: spacemit: introduce pio for k1 a588d756621d i2c: spacemit: move i2c_xfer_msg() 816f193dd0d9 (tag: next-20260406, next/master, next/HEAD) Add linux-next specific files for 20260406 This boots and reset properly without the previous WARN. So if any of those those series need added T-Bs, then here's mine: Tested-by: Vincent Legoll<legoll@online.fr> # OrangePi-RV2 I also believe this is better than nothing: reboot + (harmless) WARN instead of hang. PS: I added recipients for the PIO patchset as CC here, for the T-B, hope this is not bothering. Thanks ^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [RESEND][PATCH v5] mfd: simple-mfd-i2c: add a reboot cell for the SpacemiT P1 chip 2026-04-06 21:55 ` [RESEND][PATCH v5] mfd: simple-mfd-i2c: add a reboot cell for the SpacemiT P1 chip Vincent Legoll @ 2026-04-08 2:48 ` Troy Mitchell 0 siblings, 0 replies; 2+ messages in thread From: Troy Mitchell @ 2026-04-08 2:48 UTC (permalink / raw) To: Vincent Legoll, linux-kernel, Lee Jones, Yixun Lan, linux-riscv, spacemit Cc: Troy Mitchell, Andi Shyti, Yixun Lan, linux-i2c, Aurelien Jarno On Tue Apr 7, 2026 at 5:55 AM CST, Vincent Legoll wrote: > Hi Aurelien, > > On 4/6/26 17:42, Aurelien Jarno wrote: >> On 2026-04-06 11:46, Vincent Legoll wrote: >>> Tell me if there's a new iteration to test, and/or if you want me testing it >>> in a different way. >> Yes, as written in the original mail, this patchset depends on PIO >> support in the spacemit I2C driver: >> https://lore.kernel.org/lkml/20260207-b4-k3-i2c-pio-v7-2-626942d94d91@linux.spacemit.com/ >> >> Without it, you might get this warning message and reboot is not 100% >> reliable. I still believe it's better than no reboot support at all. > > > OK, I tested a new stack: next-20260406, PIO patchset, reset patchset & > USB patchset: > > > $ git log --oneline > 5030369b32c4 (HEAD -> opirv2-pio-reset-usb) riscv: dts: spacemit: Enable > USB3.0/PCIe on OrangePi RV2 > 73274ce37441 riscv: dts: spacemit: Define the P1 PMIC regulators for > OrangePi RV2 > cda12b8bdbfe riscv: dts: spacemit: Enable i2c8 adapter for OrangePi RV2 > 68ce409351c6 (opirv2-pio-reset) mfd: simple-mfd-i2c: add a reboot cell > for the SpacemiT P1 chip > 245e0b8e8a91 i2c: spacemit: introduce pio for k1 > a588d756621d i2c: spacemit: move i2c_xfer_msg() > 816f193dd0d9 (tag: next-20260406, next/master, next/HEAD) Add linux-next > specific files for 20260406 > > This boots and reset properly without the previous WARN. > > So if any of those those series need added T-Bs, then here's mine: > > Tested-by: Vincent Legoll<legoll@online.fr> # OrangePi-RV2 I'll take this tag to my PIO patchset. Thanks! - Troy ^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2026-04-08 2:49 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20260401165437.1048786-3-aurelien@aurel32.net>
[not found] ` <b7e182f9-8bd8-40e6-acaa-dc6e556f2a53@online.fr>
[not found] ` <adPUTRAlzGBs2wQG@aurel32.net>
2026-04-06 21:55 ` [RESEND][PATCH v5] mfd: simple-mfd-i2c: add a reboot cell for the SpacemiT P1 chip Vincent Legoll
2026-04-08 2:48 ` Troy Mitchell
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox