Messages in this thread | | | Subject | Re: [RFC PATCH] mfd: dt: Add Aspeed LPC binding | From | Andrew Jeffery <> | Date | Fri, 18 Nov 2016 10:11:41 +1030 |
| |
On Thu, 2016-11-17 at 10:30 +0100, Linus Walleij wrote: > > On Thu, Nov 17, 2016 at 7:06 AM, Andrew Jeffery <andrew@aj.id.au> wrote: > > > +* Device tree bindings for the Aspeed LPC Controller > > We are going overboard with the lingo sometimes, to the point that we do not > understand how terse things become.
Sorry for that, it is a bit terse. Sometimes I struggle to gauge how much context I should provide.
Joel's reply covers the details of your queries below, but I'll make sure to add the information to the bindings as well.
> > LPC = Low Pin Count, right? > Explain that right here: it is a slow external bus, right?
> > +The Aspeed LPC controller contains registers for a variety of functions. Not > > +all registers for a function are contiguous, and some registers are referenced > > +by functions outside the LPC controller. > > + > > +Note that this is separate from the H8S/2168 compatible register set occupying > > +the start of the LPC controller address space. > > + > > +Some significant functions in the LPC controller: > > + > > +* LPC Host Controller > > +* Host Interface Controller > > Host interface to what? > > > +* iBT Controller > > What is iBT? > > > +* SuperIO Scratch registers > > Again more context please. > > With standards documents, either explain everything or provide > pointers for the information.
ACK!
Andrew[unhandled content-type:application/pgp-signature] | |