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

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20250916101438.GA3585920@google.com>
Date: Tue, 16 Sep 2025 11:14:38 +0100
From: Lee Jones <lee@...nel.org>
To: Binbin Zhou <zhoubb.aaron@...il.com>
Cc: Corey Minyard <minyard@....org>, Huacai Chen <chenhuacai@...nel.org>,
	Binbin Zhou <zhoubinbin@...ngson.cn>,
	Xuerui Wang <kernel@...0n.name>, loongarch@...ts.linux.dev,
	linux-kernel@...r.kernel.org,
	openipmi-developer@...ts.sourceforge.net, jeffbai@...c.io,
	kexybiscuit@...c.io, wangyao@...ote.com
Subject: Re: (subset) [PATCH v11 0/3] LoongArch: Add Loongson-2K BMC support

On Tue, 16 Sep 2025, Binbin Zhou wrote:

> Hi Lee:
> 
> Thanks for your reply.
> 
> On Tue, Sep 16, 2025 at 4:40 PM Lee Jones <lee@...nel.org> wrote:
> >
> > On Fri, 12 Sep 2025, Binbin Zhou wrote:
> >
> > > Hi Lee:
> > >
> > > On Thu, Sep 11, 2025 at 10:33 PM Lee Jones <lee@...nel.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.
> 
> However, there still seems to be a text dependency issue. The IPMI
> patch modifies the MAINTAINERS, which depends on the first patch.
> If the entire series of patches cannot be merged together, does this
> mean the IPMI patch can only be merged after the MFD patch has been
> merged into the mainline?

No, not at all.  So long as all patches come together during the
merge-window, there is no issue.

-- 
Lee Jones [李琼斯]