linux-kernel - Re: [PATCH 2/2] mfd: simple-mfd-i2c: add a reboot cell for the SpacemiT P1 chip

Open Source and information security mailing list archives

Message-ID: <aNmiDIUzx5eUEOQT@aurel32.net>
Date: Sun, 28 Sep 2025 23:01:00 +0200
From: Aurelien Jarno <aurelien@...el32.net>
To: Troy Mitchell <troy.mitchell@...ux.spacemit.com>
Cc: linux-kernel@...r.kernel.org, Lee Jones <lee@...nel.org>,
	Sebastian Reichel <sre@...nel.org>, Yixun Lan <dlan@...too.org>,
	Paul Walmsley <paul.walmsley@...ive.com>,
	Palmer Dabbelt <palmer@...belt.com>,
	Albert Ou <aou@...s.berkeley.edu>, Alexandre Ghiti <alex@...ti.fr>,
	"open list:SYSTEM RESET/SHUTDOWN DRIVERS" <linux-pm@...r.kernel.org>,
	"open list:RISC-V SPACEMIT SoC Support" <linux-riscv@...ts.infradead.org>,
	"open list:RISC-V SPACEMIT SoC Support" <spacemit@...ts.linux.dev>
Subject: Re: [PATCH 2/2] mfd: simple-mfd-i2c: add a reboot cell for the
 SpacemiT P1 chip

On 2025-09-28 10:08, Troy Mitchell wrote:
> Hi Aurelien, Thanks for your patch!
> 
> On Sun, Sep 28, 2025 at 12:07:41AM +0200, Aurelien Jarno wrote:
> > Add a "spacemit-p1-reboot" cell for the SpacemiT P1 chip.
> > 
> > Signed-off-by: Aurelien Jarno <aurelien@...el32.net>
> > ---
> >  drivers/mfd/simple-mfd-i2c.c | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/drivers/mfd/simple-mfd-i2c.c b/drivers/mfd/simple-mfd-i2c.c
> > index 696b602051260..2e86efb0c82b8 100644
> > --- a/drivers/mfd/simple-mfd-i2c.c
> > +++ b/drivers/mfd/simple-mfd-i2c.c
> > @@ -99,6 +99,7 @@ static const struct regmap_config spacemit_p1_regmap_config = {
> >  };
> >  
> >  static const struct mfd_cell spacemit_p1_cells[] = {
> > +	{ .name = "spacemit-p1-reboot", },
> I’m not sure if this name is the best fit here.
> Since the driver also implements reboot and power-off functionality,
> would it make more sense to call it spacemit-p1-power?
> I’ll leave it up to you.

I see your point, for this driver naming everything was among the most 
complex things. I have chosen "spacemit-p1-reboot" to be consistent with  
the naming used in the driver itself. I chose them from recently added 
drivers supporting both poweroff and reset, that seems to all use the 
term reboot.

> Otherwise, LGTM.
> 
> Reviewed-by: Troy Mitchell <troy.mitchell@...ux.spacemit.com>
> >  	{ .name = "spacemit-p1-regulator", },
> >  	{ .name = "spacemit-p1-rtc", },
> >  };
> > -- 
> > 2.47.2
> > 
> > 
> > _______________________________________________
> > linux-riscv mailing list
> > linux-riscv@...ts.infradead.org
> > http://lists.infradead.org/mailman/listinfo/linux-riscv
> 

-- 
Aurelien Jarno                          GPG: 4096R/1DDD8C9B
aurelien@...el32.net                     http://aurel32.net

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives