(subset) [PATCH v2 1/4] mfd: simple-mfd-i2c: remove select I2C_K1
Lee Jones
lee at kernel.org
Mon Nov 10 05:45:50 PST 2025
On Fri, 07 Nov 2025, Troy Mitchell wrote:
> On Thu, Nov 06, 2025 at 04:04:21PM +0000, Lee Jones wrote:
> > On Mon, 27 Oct 2025 13:48:05 +0800, Troy Mitchell wrote:
> > > select will force a symbol to a specific value without considering
> > > its dependencies. As a result, the i2c-k1 driver will fail to build
> > > when OF or COMMON_CLK are disabled.
> > >
> > > The reason for removing I2C_K1 instead of adding a depends on condition
> > > is to keep the possibility for other SoCs to use this PMIC.
> > >
> > > [...]
> >
> > Applied, thanks!
> >
> > [1/4] mfd: simple-mfd-i2c: remove select I2C_K1
> > commit: ecf6bc474ae97c404e2125b413eb0ef3627b03c5
> Hi Lee,
>
> I think you didn't notice this reply [1]
> (Maybe because he was replying to the cover letter).
>
> As Aurelien mentioned, the current shutdown/reboot (and possibly the regulator
> as well) intends to use the `default MFD_SPACEMIT_P1`.
> So if there’s no `default m if ARCH_SPACEMIT`,
> the default value in subdevices may not make much sense.
>
> But don’t worry — to make things easier for you, I’ll send an additional
> patch based on your branch (in this series).
>
> How does that sound?
Go for it.
--
Lee Jones [李琼斯]
More information about the linux-riscv
mailing list