[PULL][Zesty] Low-pin-count (LPC) controller support for arm64

[Seth Forshee]

On Mon, Apr 24, 2017 at 03:39:08PM -0600, dann frazier wrote:
> On Mon, Apr 24, 2017 at 8:20 AM, Seth Forshee
> <seth.forshee at canonical.com> wrote:
> > On Thu, Apr 20, 2017 at 03:23:33PM -0600, dann frazier wrote:
> >> Here's an updated PR that increases the buffer size to match the
> >> length attribute which, as you noted off-list, follows a pattern used
> >> elsewhere in the kernel. I've also provided this feedback in a review
> >> of the current LPC patch series.
> >>
> >> The following changes since commit 815d83d7188878caeaff28a15be14a8d676dc045:
> >>
> >>   Linux 4.10.10 (2017-04-12 07:26:53 -0600)
> >>
> >> are available in the git repository at:
> >>
> >>   git://git.launchpad.net/~dannf/ubuntu/+source/linux/+git/linux lpc-20170420
> >>
> >> for you to fetch changes up to 2ff710b9447e5e50533970c5740189fbf1fff58a:
> >>
> >>   UBUNTU: SAUCE: LIBIO: Make the size of the acpi_resource buffer
> >> match it's length property (2017-04-20 15:20:19 -0600)
> >
> > Sorry for the delay in getting this review back to you.
> >
> > One thing you might want to consider. I note that in the latest upstream
> > submission this has changed from libio to some other name. Adopting what
> > will (presumably) be the name used upstream might make it less
> > cumbersome when you need to backport future changes.
> 
> The name has changed a few times already (extio->libio->logic pio),
> and I'm not confident the last one will stick either. The current name
> was not one of the ones upstream suggested, and it hasn't been ACK'd
> yet. My preference would therefore be to merge as-is, unless you feel
> strongly otherwise.

No, it's fine. I only mentioned it because I thought it might make your
future backporting work easier.

Seth