<div dir="ltr"><div>Adding Patrick and Joel to this discussion,</div><div><br></div><div>Thanks for your prompt reply!</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, 11 Jun 2019 at 16:12, Arnd Bergmann <<a href="mailto:arnd@arndb.de">arnd@arndb.de</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Mon, Jun 10, 2019 at 4:19 PM Tomer Maimon <<a href="mailto:tmaimon77@gmail.com" target="_blank">tmaimon77@gmail.com</a>> wrote:<br>
><br>
> Add Nuvoton BMC NPCM BIOS post code (BPC) driver.<br>
><br>
> The NPCM BPC monitoring two I/O address written by<br>
> the host on the Low Pin Count (LPC) bus, the capure<br>
> data stored in 128-word FIFO.<br>
><br>
> Signed-off-by: Tomer Maimon <<a href="mailto:tmaimon77@gmail.com" target="_blank">tmaimon77@gmail.com</a>><br>
<br>
We've run into this situation before, but don't have a good solution yet:<br>
<br>
The driver seems useful and well implemented, but I keep having a bad<br>
feeling about adding a chardev driver into drivers/soc for something that<br>
is clearly specific to a particular implementation on the hardware side<br>
but generic on the user interface. The same user interface might be<br>
used on an Aspeed BMC or any other one, so please coordinate at<br>
least between Novoton and Aspeed developers on creating a common<br>
user interface, and review each other's patches.<br></blockquote><div> </div><div>Probably the only vendors that will use the snoop will be Nuvoton and Aspeed.</div><div>is it worth to create new snoop common user interface for it, </div><div>if we will develop a new snoop user interface who will be the maintainer?</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Maybe we can introduce a drivers/bmc/ (or even drivers/openbmc)<br></blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">that collects all those user interfaces with a thin abstraction layer<br>
and one or two hardware specific back-ends?<br></blockquote><div> </div><div>Sounds good, Maybe we can move the KCS BMC from driver/char/ipmi to the drivers/bmc/?</div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
        Arnd<br></blockquote><div><br></div>appreciate your opinions, Joel, Andrew and Patrick?</div><div class="gmail_quote"><br></div><div class="gmail_quote">Thanks,</div><div class="gmail_quote"><br></div><div class="gmail_quote">Tomer<br class="gmail-Apple-interchange-newline"><div> </div></div></div>