Re: [PATCH v3 5/7] mfd: cros_ec: Sync to the latest cros_ec_commands.h from EC sources

2014-06-17 Thread Paul Bolle
On Tue, 2014-06-17 at 10:20 -0600, Stephen Warren wrote: > On 06/17/2014 02:53 AM, Paul Bolle wrote: > > So, in summary, while we're apparently only discussing a single comment, > > I would appreciate it if it could be reworded, preferably by dropping > > that the CONFIG_ prefix. But other people

Re: [PATCH v3 5/7] mfd: cros_ec: Sync to the latest cros_ec_commands.h from EC sources

2014-06-17 Thread Stephen Warren
On 06/17/2014 02:53 AM, Paul Bolle wrote: > Doug, > > On Fri, 2014-06-13 at 08:22 -0700, Doug Anderson wrote: >> On Fri, Jun 13, 2014 at 1:08 AM, Paul Bolle wrote: >>> On Wed, 2014-06-11 at 08:11 -0700, Doug Anderson wrote: This is a config option on the ChromeOS EC

Re: [PATCH v3 5/7] mfd: cros_ec: Sync to the latest cros_ec_commands.h from EC sources

2014-06-17 Thread Paul Bolle
Doug, On Fri, 2014-06-13 at 08:22 -0700, Doug Anderson wrote: > On Fri, Jun 13, 2014 at 1:08 AM, Paul Bolle wrote: > > On Wed, 2014-06-11 at 08:11 -0700, Doug Anderson wrote: > >> This is a config option on the ChromeOS EC > >> . Doing a

Re: [PATCH v3 5/7] mfd: cros_ec: Sync to the latest cros_ec_commands.h from EC sources

2014-06-17 Thread Paul Bolle
Doug, On Fri, 2014-06-13 at 08:22 -0700, Doug Anderson wrote: On Fri, Jun 13, 2014 at 1:08 AM, Paul Bolle pebo...@tiscali.nl wrote: On Wed, 2014-06-11 at 08:11 -0700, Doug Anderson wrote: This is a config option on the ChromeOS EC https://chromium.googlesource.com/chromiumos/platform/ec.

Re: [PATCH v3 5/7] mfd: cros_ec: Sync to the latest cros_ec_commands.h from EC sources

2014-06-17 Thread Stephen Warren
On 06/17/2014 02:53 AM, Paul Bolle wrote: Doug, On Fri, 2014-06-13 at 08:22 -0700, Doug Anderson wrote: On Fri, Jun 13, 2014 at 1:08 AM, Paul Bolle pebo...@tiscali.nl wrote: On Wed, 2014-06-11 at 08:11 -0700, Doug Anderson wrote: This is a config option on the ChromeOS EC

Re: [PATCH v3 5/7] mfd: cros_ec: Sync to the latest cros_ec_commands.h from EC sources

2014-06-17 Thread Paul Bolle
On Tue, 2014-06-17 at 10:20 -0600, Stephen Warren wrote: On 06/17/2014 02:53 AM, Paul Bolle wrote: So, in summary, while we're apparently only discussing a single comment, I would appreciate it if it could be reworded, preferably by dropping that the CONFIG_ prefix. But other people might

Re: [PATCH v3 5/7] mfd: cros_ec: Sync to the latest cros_ec_commands.h from EC sources

2014-06-13 Thread Doug Anderson
Paul, On Fri, Jun 13, 2014 at 1:08 AM, Paul Bolle wrote: > Doug, > > On Wed, 2014-06-11 at 08:11 -0700, Doug Anderson wrote: >> On Wed, Jun 11, 2014 at 3:37 AM, Paul Bolle wrote: >> > On Tue, 2014-05-20 at 09:46 +0100, Lee Jones wrote: >> >> On Wed, 30 Apr 2014, Doug Anderson wrote: >> >> >

Re: [PATCH v3 5/7] mfd: cros_ec: Sync to the latest cros_ec_commands.h from EC sources

2014-06-13 Thread Paul Bolle
Doug, On Wed, 2014-06-11 at 08:11 -0700, Doug Anderson wrote: > On Wed, Jun 11, 2014 at 3:37 AM, Paul Bolle wrote: > > On Tue, 2014-05-20 at 09:46 +0100, Lee Jones wrote: > >> On Wed, 30 Apr 2014, Doug Anderson wrote: > >> > From: Bill Richardson > >> > > >> > This just updates

Re: [PATCH v3 5/7] mfd: cros_ec: Sync to the latest cros_ec_commands.h from EC sources

2014-06-13 Thread Paul Bolle
Doug, On Wed, 2014-06-11 at 08:11 -0700, Doug Anderson wrote: On Wed, Jun 11, 2014 at 3:37 AM, Paul Bolle pebo...@tiscali.nl wrote: On Tue, 2014-05-20 at 09:46 +0100, Lee Jones wrote: On Wed, 30 Apr 2014, Doug Anderson wrote: From: Bill Richardson wfric...@chromium.org This just

Re: [PATCH v3 5/7] mfd: cros_ec: Sync to the latest cros_ec_commands.h from EC sources

2014-06-13 Thread Doug Anderson
Paul, On Fri, Jun 13, 2014 at 1:08 AM, Paul Bolle pebo...@tiscali.nl wrote: Doug, On Wed, 2014-06-11 at 08:11 -0700, Doug Anderson wrote: On Wed, Jun 11, 2014 at 3:37 AM, Paul Bolle pebo...@tiscali.nl wrote: On Tue, 2014-05-20 at 09:46 +0100, Lee Jones wrote: On Wed, 30 Apr 2014, Doug

Re: [PATCH v3 5/7] mfd: cros_ec: Sync to the latest cros_ec_commands.h from EC sources

2014-06-11 Thread Doug Anderson
Paul, On Wed, Jun 11, 2014 at 3:37 AM, Paul Bolle wrote: > On Tue, 2014-05-20 at 09:46 +0100, Lee Jones wrote: >> On Wed, 30 Apr 2014, Doug Anderson wrote: >> > From: Bill Richardson >> > >> > This just updates include/linux/mfd/cros_ec_commands.h to match the >> > latest EC version (which is

Re: [PATCH v3 5/7] mfd: cros_ec: Sync to the latest cros_ec_commands.h from EC sources

2014-06-11 Thread Paul Bolle
On Tue, 2014-05-20 at 09:46 +0100, Lee Jones wrote: > On Wed, 30 Apr 2014, Doug Anderson wrote: > > From: Bill Richardson > > > > This just updates include/linux/mfd/cros_ec_commands.h to match the > > latest EC version (which is the One True Source for such things). See > >

Re: [PATCH v3 5/7] mfd: cros_ec: Sync to the latest cros_ec_commands.h from EC sources

2014-06-11 Thread Paul Bolle
On Tue, 2014-05-20 at 09:46 +0100, Lee Jones wrote: On Wed, 30 Apr 2014, Doug Anderson wrote: From: Bill Richardson wfric...@chromium.org This just updates include/linux/mfd/cros_ec_commands.h to match the latest EC version (which is the One True Source for such things). See

Re: [PATCH v3 5/7] mfd: cros_ec: Sync to the latest cros_ec_commands.h from EC sources

2014-06-11 Thread Doug Anderson
Paul, On Wed, Jun 11, 2014 at 3:37 AM, Paul Bolle pebo...@tiscali.nl wrote: On Tue, 2014-05-20 at 09:46 +0100, Lee Jones wrote: On Wed, 30 Apr 2014, Doug Anderson wrote: From: Bill Richardson wfric...@chromium.org This just updates include/linux/mfd/cros_ec_commands.h to match the

Re: [PATCH v3 5/7] mfd: cros_ec: Sync to the latest cros_ec_commands.h from EC sources

2014-05-20 Thread Lee Jones
On Wed, 30 Apr 2014, Doug Anderson wrote: > From: Bill Richardson > > This just updates include/linux/mfd/cros_ec_commands.h to match the > latest EC version (which is the One True Source for such things). See > > > [dianders: took

Re: [PATCH v3 5/7] mfd: cros_ec: Sync to the latest cros_ec_commands.h from EC sources

2014-05-20 Thread Lee Jones
On Wed, 30 Apr 2014, Doug Anderson wrote: From: Bill Richardson wfric...@chromium.org This just updates include/linux/mfd/cros_ec_commands.h to match the latest EC version (which is the One True Source for such things). See https://chromium.googlesource.com/chromiumos/platform/ec