Re: (subset) [PATCH v11 0/3] LoongArch: Add Loongson-2K BMC support

[Lee Jones <lee@kernel.org>]

On Fri, 12 Sep 2025, Binbin Zhou wrote:

> Hi Lee:
> 
> On Thu, Sep 11, 2025 at 10:33 PM Lee Jones <lee@kernel.org> wrote:
> >
> > On Thu, 04 Sep 2025 20:35:04 +0800, Binbin Zhou wrote:
> > > This patchset introduces the Loongson-2K BMC.
> > >
> > > It is a PCIe device present on servers similar to the Loongson-3 CPUs.
> > > And it is a multifunctional device (MFD), such as display as a sub-function
> > > of it.
> > >
> > > For IPMI, according to the existing design, we use software simulation to
> > > implement the KCS interface registers: Stauts/Command/Data_Out/Data_In.
> > >
> > > [...]
> >
> > Applied, thanks!
> >
> > [1/3] mfd: ls2kbmc: Introduce Loongson-2K BMC core driver
> >       commit: 67c2639e1fc1a07b45d216af659c0dd92a370c68
> > [2/3] mfd: ls2kbmc: Add Loongson-2K BMC reset function support
> >       commit: 2364ccc827e44064e9763f2ae2d1dcc5f945fdf3
> 
> Thanks for acknowledging my patchset.
> 
> I can't confirm why you didn't apply the IPMI patch, but this appears
> to break the patchset's integrity, potentially causing missing Kconfig
> dependencies (IPMI_LS2K select MFD_LS2K_BMC_CORE).

Pretty sure this doesn't break anything.

What build errors do you see as a result?

> Additionally, as Corey previously explained[1], this patch can be
> applied through your side.
> 
> [1]: https://lore.kernel.org/all/aFVtNAY4u2gDiLDS@mail.minyard.net/

We only apply cross-subsystem patch-sets to a single tree if there are
good reasons to do so.  In this instance, I can't see any reason why the
IPMI driver cannot go in via it's own repo.

-- 
Lee Jones [李琼斯]