[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <Yyz70LckHqyiNhGa@minyard.net>
Date: Thu, 22 Sep 2022 19:20:32 -0500
From: Corey Minyard <minyard@....org>
To: ChiaWei Wang <chiawei_wang@...eedtech.com>
Cc: "joel@....id.au" <joel@....id.au>,
"andrew@...id.au" <andrew@...id.au>,
"openipmi-developer@...ts.sourceforge.net"
<openipmi-developer@...ts.sourceforge.net>,
"linux-arm-kernel@...ts.infradead.org"
<linux-arm-kernel@...ts.infradead.org>,
"linux-aspeed@...ts.ozlabs.org" <linux-aspeed@...ts.ozlabs.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"openbmc@...ts.ozlabs.org" <openbmc@...ts.ozlabs.org>
Subject: Re: [PATCH] ipmi: kcs: aspeed: Update port address comments
On Fri, Sep 23, 2022 at 12:08:07AM +0000, ChiaWei Wang wrote:
> Hi Corey,
>
> > From: Corey Minyard <tcminyard@...il.com> On Behalf Of Corey Minyard
> > Sent: Friday, September 23, 2022 2:58 AM
> >
> > On Tue, Sep 20, 2022 at 10:03:33AM +0800, Chia-Wei Wang wrote:
> > > Remove AST_usrGuide_KCS.pdf as it is no longer maintained.
> >
> > Even if it's no longer maintained, is it useful? It seems better to leave in
> > useful documentation unless it has been replaced with something else.
>
> This document has no permeant public link to access.
> Aspeed has dropped this file but we keep receiving customer request asking for this document.
> The most important part regarding KCS port rule is still kept in the updated comment.
I mean, if you have code that is implementing what is documented, why
did you remove the document? I don't understand why you would retire
documentation that people still want to use.
I could put it on the IPMI sourceforge or github page as a historical
document.
-corey
>
> Regards,
> Chiawei
>
> >
> >
> > >
> > > Add more descriptions as the driver now supports the I/O address
> > > configurations for both the KCS Data and Cmd/Status interface
> > > registers.
> > >
> > > Signed-off-by: Chia-Wei Wang <chiawei_wang@...eedtech.com>
> > > ---
> > > drivers/char/ipmi/kcs_bmc_aspeed.c | 29 ++++++++++++++++++-----------
> > > 1 file changed, 18 insertions(+), 11 deletions(-)
> > >
> > > diff --git a/drivers/char/ipmi/kcs_bmc_aspeed.c
> > > b/drivers/char/ipmi/kcs_bmc_aspeed.c
> > > index cdc88cde1e9a..19c32bf50e0e 100644
> > > --- a/drivers/char/ipmi/kcs_bmc_aspeed.c
> > > +++ b/drivers/char/ipmi/kcs_bmc_aspeed.c
> > > @@ -207,17 +207,24 @@ static void aspeed_kcs_updateb(struct
> > > kcs_bmc_device *kcs_bmc, u32 reg, u8 mask, }
> > >
> > > /*
> > > - * AST_usrGuide_KCS.pdf
> > > - * 2. Background:
> > > - * we note D for Data, and C for Cmd/Status, default rules are
> > > - * A. KCS1 / KCS2 ( D / C:X / X+4 )
> > > - * D / C : CA0h / CA4h
> > > - * D / C : CA8h / CACh
> > > - * B. KCS3 ( D / C:XX2h / XX3h )
> > > - * D / C : CA2h / CA3h
> > > - * D / C : CB2h / CB3h
> > > - * C. KCS4
> > > - * D / C : CA4h / CA5h
> > > + * We note D for Data, and C for Cmd/Status, default rules are
> > > + *
> > > + * 1. Only the D address is given:
> > > + * A. KCS1/KCS2 (D/C: X/X+4)
> > > + * D/C: CA0h/CA4h
> > > + * D/C: CA8h/CACh
> > > + * B. KCS3 (D/C: XX2/XX3h)
> > > + * D/C: CA2h/CA3h
> > > + * C. KCS4 (D/C: X/X+1)
> > > + * D/C: CA4h/CA5h
> > > + *
> > > + * 2. Both the D/C addresses are given:
> > > + * A. KCS1/KCS2/KCS4 (D/C: X/Y)
> > > + * D/C: CA0h/CA1h
> > > + * D/C: CA8h/CA9h
> > > + * D/C: CA4h/CA5h
> > > + * B. KCS3 (D/C: XX2/XX3h)
> > > + * D/C: CA2h/CA3h
> > > */
> > > static int aspeed_kcs_set_address(struct kcs_bmc_device *kcs_bmc, u32
> > > addrs[2], int nr_addrs) {
> > > --
> > > 2.25.1
> > >
Powered by blists - more mailing lists