lkml.org 
[lkml]   [2023]   [Dec]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [PATCH v3 3/3] soc: nuvoton: add NPCM BPC driver
Hi Arnd,

Thanks for your suggestion.

Appreciate it if Joel, OpenBMC Linux kernel maintainer, could share
his thoughts about it.


On Thu, 14 Dec 2023 at 17:49, Arnd Bergmann <arnd@arndb.de> wrote:
>
> On Thu, Dec 14, 2023, at 14:09, Tomer Maimon wrote:
> > On Thu, 14 Dec 2023 at 14:44, Arnd Bergmann <arnd@arndb.de> wrote:
> >> >
> >> > +config NPCM_BP
> >> > + tristate "NPCM BIOS Post Code support"
> >> > + depends on (ARCH_NPCM || COMPILE_TEST)
> >> > + help
> >> > + Provides NPCM driver to control the BIOS Post Code
> >> > + interface which allows the BMC to monitor and save
> >> > + the data written by the host to an arbitrary I/O port,
> >> > + the BPC is connected to the host thourgh LPC or eSPI bus.
> >> > +
> >>
> >> This one in particular looks like this might be implemented
> >> by more than one BMC type, it's a fairly generic functionality.
> >>
> >> Have you talked to the other maintainers of SoCs used in
> >> OpenBMC about coming up with a common interface?
> > Yes, Both Nuvoton and Aspeed use the same user-facing code to manage
> > the host snooping.
> > https://github.com/openbmc/phosphor-host-postd
>
> Ok, that's good. I found the driver in drivers/soc/aspeed/aspeed-lpc-snoop.c
> now and see that the implementation looks very similar.
>
> I think we should do two things here:
>
> - split out the common code into a shared module that exports the
> symbols to be used by either one
>
> - find a better place for both drivers outside of drivers/soc.
> I would suggest drivers/misc/bmc/ but am open to other suggestions.
>
> Arnd

Best regards,

Tomer

\
 
 \ /
  Last update: 2023-12-14 17:51    [W:0.079 / U:0.100 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site