Re: [PATCH v2 2/2] eeprom: Add IDT 89HPESx driver bindings file

2016-12-13 Thread Serge Semin
On Mon, Dec 12, 2016 at 05:04:31PM -0600, Rob Herring wrote: > On Mon, Dec 5, 2016 at 1:04 PM, Serge Semin wrote: > > On Mon, Dec 05, 2016 at 11:27:07AM -0600, Rob Herring > > wrote: > >> On Mon, Dec 5, 2016 at 9:25 AM, Serge Semin

Re: [PATCH v2 2/2] eeprom: Add IDT 89HPESx driver bindings file

2016-12-13 Thread Serge Semin
On Mon, Dec 12, 2016 at 05:04:31PM -0600, Rob Herring wrote: > On Mon, Dec 5, 2016 at 1:04 PM, Serge Semin wrote: > > On Mon, Dec 05, 2016 at 11:27:07AM -0600, Rob Herring > > wrote: > >> On Mon, Dec 5, 2016 at 9:25 AM, Serge Semin > >> wrote: > >> > On Mon, Dec 05, 2016 at 08:46:21AM -0600,

Re: [PATCH v2 2/2] eeprom: Add IDT 89HPESx driver bindings file

2016-12-12 Thread Rob Herring
On Mon, Dec 5, 2016 at 1:04 PM, Serge Semin wrote: > On Mon, Dec 05, 2016 at 11:27:07AM -0600, Rob Herring wrote: >> On Mon, Dec 5, 2016 at 9:25 AM, Serge Semin wrote: >> > On Mon, Dec 05, 2016 at 08:46:21AM -0600, Rob Herring

Re: [PATCH v2 2/2] eeprom: Add IDT 89HPESx driver bindings file

2016-12-12 Thread Rob Herring
On Mon, Dec 5, 2016 at 1:04 PM, Serge Semin wrote: > On Mon, Dec 05, 2016 at 11:27:07AM -0600, Rob Herring wrote: >> On Mon, Dec 5, 2016 at 9:25 AM, Serge Semin wrote: >> > On Mon, Dec 05, 2016 at 08:46:21AM -0600, Rob Herring >> > wrote: >> >> On Tue, Nov 29, 2016 at 01:38:21AM +0300, Serge

Re: [PATCH v2 2/2] eeprom: Add IDT 89HPESx driver bindings file

2016-12-08 Thread Serge Semin
Rob, Could you please respond on these comments? I've got some free time, so I wanna rewrite the code until I've not got busy again. Regards, -Sergey On Mon, Dec 05, 2016 at 10:04:56PM +0300, Serge Semin wrote: > On Mon, Dec 05, 2016 at 11:27:07AM -0600, Rob Herring

Re: [PATCH v2 2/2] eeprom: Add IDT 89HPESx driver bindings file

2016-12-08 Thread Serge Semin
Rob, Could you please respond on these comments? I've got some free time, so I wanna rewrite the code until I've not got busy again. Regards, -Sergey On Mon, Dec 05, 2016 at 10:04:56PM +0300, Serge Semin wrote: > On Mon, Dec 05, 2016 at 11:27:07AM -0600, Rob Herring wrote: > > On Mon, Dec 5,

Re: [PATCH v2 2/2] eeprom: Add IDT 89HPESx driver bindings file

2016-12-05 Thread Serge Semin
On Mon, Dec 05, 2016 at 11:27:07AM -0600, Rob Herring wrote: > On Mon, Dec 5, 2016 at 9:25 AM, Serge Semin wrote: > > On Mon, Dec 05, 2016 at 08:46:21AM -0600, Rob Herring > > wrote: > >> On Tue, Nov 29, 2016 at 01:38:21AM +0300, Serge

Re: [PATCH v2 2/2] eeprom: Add IDT 89HPESx driver bindings file

2016-12-05 Thread Serge Semin
On Mon, Dec 05, 2016 at 11:27:07AM -0600, Rob Herring wrote: > On Mon, Dec 5, 2016 at 9:25 AM, Serge Semin wrote: > > On Mon, Dec 05, 2016 at 08:46:21AM -0600, Rob Herring > > wrote: > >> On Tue, Nov 29, 2016 at 01:38:21AM +0300, Serge Semin wrote: > >> > See cover-letter for changelog > >> >

Re: [PATCH v2 2/2] eeprom: Add IDT 89HPESx driver bindings file

2016-12-05 Thread Rob Herring
On Mon, Dec 5, 2016 at 9:25 AM, Serge Semin wrote: > On Mon, Dec 05, 2016 at 08:46:21AM -0600, Rob Herring wrote: >> On Tue, Nov 29, 2016 at 01:38:21AM +0300, Serge Semin wrote: >> > See cover-letter for changelog >> > >> > Signed-off-by: Serge Semin

Re: [PATCH v2 2/2] eeprom: Add IDT 89HPESx driver bindings file

2016-12-05 Thread Rob Herring
On Mon, Dec 5, 2016 at 9:25 AM, Serge Semin wrote: > On Mon, Dec 05, 2016 at 08:46:21AM -0600, Rob Herring wrote: >> On Tue, Nov 29, 2016 at 01:38:21AM +0300, Serge Semin wrote: >> > See cover-letter for changelog >> > >> > Signed-off-by: Serge Semin >> > >> > --- >> >

Re: [PATCH v2 2/2] eeprom: Add IDT 89HPESx driver bindings file

2016-12-05 Thread Serge Semin
On Mon, Dec 05, 2016 at 08:46:21AM -0600, Rob Herring wrote: > On Tue, Nov 29, 2016 at 01:38:21AM +0300, Serge Semin wrote: > > See cover-letter for changelog > > > > Signed-off-by: Serge Semin > > > > --- > >

Re: [PATCH v2 2/2] eeprom: Add IDT 89HPESx driver bindings file

2016-12-05 Thread Serge Semin
On Mon, Dec 05, 2016 at 08:46:21AM -0600, Rob Herring wrote: > On Tue, Nov 29, 2016 at 01:38:21AM +0300, Serge Semin wrote: > > See cover-letter for changelog > > > > Signed-off-by: Serge Semin > > > > --- > > .../devicetree/bindings/misc/idt_89hpesx.txt | 41 > > ++

Re: [PATCH v2 2/2] eeprom: Add IDT 89HPESx driver bindings file

2016-12-05 Thread Rob Herring
On Tue, Nov 29, 2016 at 01:38:21AM +0300, Serge Semin wrote: > See cover-letter for changelog > > Signed-off-by: Serge Semin > > --- > .../devicetree/bindings/misc/idt_89hpesx.txt | 41 > ++ There's not a better location for this? I can't

Re: [PATCH v2 2/2] eeprom: Add IDT 89HPESx driver bindings file

2016-12-05 Thread Rob Herring
On Tue, Nov 29, 2016 at 01:38:21AM +0300, Serge Semin wrote: > See cover-letter for changelog > > Signed-off-by: Serge Semin > > --- > .../devicetree/bindings/misc/idt_89hpesx.txt | 41 > ++ There's not a better location for this? I can't tell because you don't

Re: [PATCH v2 2/2] eeprom: Add IDT 89HPESx driver bindings file

2016-11-29 Thread Serge Semin
On Tue, Nov 29, 2016 at 08:34:36PM +0100, Greg KH wrote: > On Tue, Nov 29, 2016 at 01:38:21AM +0300, Serge Semin wrote: > > See cover-letter for changelog > > There is no cover letter in an individual patch when it gets committed > to the tree... > > So please fix,

Re: [PATCH v2 2/2] eeprom: Add IDT 89HPESx driver bindings file

2016-11-29 Thread Serge Semin
On Tue, Nov 29, 2016 at 08:34:36PM +0100, Greg KH wrote: > On Tue, Nov 29, 2016 at 01:38:21AM +0300, Serge Semin wrote: > > See cover-letter for changelog > > There is no cover letter in an individual patch when it gets committed > to the tree... > > So please fix, personally, I never read

Re: [PATCH v2 2/2] eeprom: Add IDT 89HPESx driver bindings file

2016-11-29 Thread Greg KH
On Tue, Nov 29, 2016 at 01:38:21AM +0300, Serge Semin wrote: > See cover-letter for changelog There is no cover letter in an individual patch when it gets committed to the tree... So please fix, personally, I never read cover letters, each patch should be "obvious" on it's own :) thanks, greg

Re: [PATCH v2 2/2] eeprom: Add IDT 89HPESx driver bindings file

2016-11-29 Thread Greg KH
On Tue, Nov 29, 2016 at 01:38:21AM +0300, Serge Semin wrote: > See cover-letter for changelog There is no cover letter in an individual patch when it gets committed to the tree... So please fix, personally, I never read cover letters, each patch should be "obvious" on it's own :) thanks, greg

[PATCH v2 2/2] eeprom: Add IDT 89HPESx driver bindings file

2016-11-28 Thread Serge Semin
See cover-letter for changelog Signed-off-by: Serge Semin --- .../devicetree/bindings/misc/idt_89hpesx.txt | 41 ++ 1 file changed, 41 insertions(+) create mode 100644 Documentation/devicetree/bindings/misc/idt_89hpesx.txt diff --git

[PATCH v2 2/2] eeprom: Add IDT 89HPESx driver bindings file

2016-11-28 Thread Serge Semin
See cover-letter for changelog Signed-off-by: Serge Semin --- .../devicetree/bindings/misc/idt_89hpesx.txt | 41 ++ 1 file changed, 41 insertions(+) create mode 100644 Documentation/devicetree/bindings/misc/idt_89hpesx.txt diff --git