Re: [PATCH 06/15] Platform: OLPC: Add XO-1.75 EC driver

2018-11-19 Thread Lubomir Rintel
Hi Pavel, I've addressed some of Andy's concerns you're replying to below in a follow-up version of the patch. To many points I'm generally indifferent and prefer to lean towards making things easy for whoever may be likely to deal with the code in past. It's not clear to me who that might be, or

Re: [PATCH 06/15] Platform: OLPC: Add XO-1.75 EC driver

2018-11-19 Thread Lubomir Rintel
Hi Pavel, I've addressed some of Andy's concerns you're replying to below in a follow-up version of the patch. To many points I'm generally indifferent and prefer to lean towards making things easy for whoever may be likely to deal with the code in past. It's not clear to me who that might be, or

Re: [PATCH 06/15] Platform: OLPC: Add XO-1.75 EC driver

2018-11-19 Thread Pavel Machek
Hi! > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > > Easy to maintain when it's sorted. No / it

Re: [PATCH 06/15] Platform: OLPC: Add XO-1.75 EC driver

2018-11-19 Thread Pavel Machek
Hi! > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > > Easy to maintain when it's sorted. No / it

Re: [PATCH 06/15] Platform: OLPC: Add XO-1.75 EC driver

2018-11-13 Thread James Cameron
On Tue, Nov 13, 2018 at 06:26:09PM +0100, Lubomir Rintel wrote: > Hi, > > first of all -- thanks for such a careful review. It is very helpful. > > Wherever I don't respond to you, I'm just following what you wrote. It > would perhaps be tiresome to respond to "Yes, will fix in next version" >

Re: [PATCH 06/15] Platform: OLPC: Add XO-1.75 EC driver

2018-11-13 Thread James Cameron
On Tue, Nov 13, 2018 at 06:26:09PM +0100, Lubomir Rintel wrote: > Hi, > > first of all -- thanks for such a careful review. It is very helpful. > > Wherever I don't respond to you, I'm just following what you wrote. It > would perhaps be tiresome to respond to "Yes, will fix in next version" >

Re: [PATCH 06/15] Platform: OLPC: Add XO-1.75 EC driver

2018-11-13 Thread Lubomir Rintel
Hi, first of all -- thanks for such a careful review. It is very helpful. Wherever I don't respond to you, I'm just following what you wrote. It would perhaps be tiresome to respond to "Yes, will fix in next version" to every single point. I'll be following up with a new version in a few days;

Re: [PATCH 06/15] Platform: OLPC: Add XO-1.75 EC driver

2018-11-13 Thread Lubomir Rintel
Hi, first of all -- thanks for such a careful review. It is very helpful. Wherever I don't respond to you, I'm just following what you wrote. It would perhaps be tiresome to respond to "Yes, will fix in next version" to every single point. I'll be following up with a new version in a few days;

Re: [PATCH 06/15] Platform: OLPC: Add XO-1.75 EC driver

2018-10-19 Thread Andy Shevchenko
On Wed, Oct 10, 2018 at 8:24 PM Lubomir Rintel wrote: > > It's based off the driver from the OLPC kernel sources. Somewhat > modernized and cleaned up, for better or worse. > > Modified to plug into the olpc-ec driver infrastructure (so that battery > interface and debugfs could be reused) and

Re: [PATCH 06/15] Platform: OLPC: Add XO-1.75 EC driver

2018-10-19 Thread Andy Shevchenko
On Wed, Oct 10, 2018 at 8:24 PM Lubomir Rintel wrote: > > It's based off the driver from the OLPC kernel sources. Somewhat > modernized and cleaned up, for better or worse. > > Modified to plug into the olpc-ec driver infrastructure (so that battery > interface and debugfs could be reused) and

[PATCH 06/15] Platform: OLPC: Add XO-1.75 EC driver

2018-10-10 Thread Lubomir Rintel
It's based off the driver from the OLPC kernel sources. Somewhat modernized and cleaned up, for better or worse. Modified to plug into the olpc-ec driver infrastructure (so that battery interface and debugfs could be reused) and the SPI slave framework. Signed-off-by: Lubomir Rintel ---

[PATCH 06/15] Platform: OLPC: Add XO-1.75 EC driver

2018-10-10 Thread Lubomir Rintel
It's based off the driver from the OLPC kernel sources. Somewhat modernized and cleaned up, for better or worse. Modified to plug into the olpc-ec driver infrastructure (so that battery interface and debugfs could be reused) and the SPI slave framework. Signed-off-by: Lubomir Rintel ---