Re: [PATCH v3] net: ethernet: support "fixed-link" DT node on nb8800 driver

2016-02-08 Thread Sebastian Frias
On 02/05/2016 04:26 PM, Måns Rullgård wrote: Sebastian Frias writes: On 02/05/2016 04:08 PM, Måns Rullgård wrote: Sebastian Frias writes: On 02/05/2016 03:34 PM, Måns Rullgård wrote: Sebastian Frias writes: Signed-off-by: Sebastian

Re: [PATCH v3] net: ethernet: support "fixed-link" DT node on nb8800 driver

2016-02-08 Thread Mason
On 08/02/2016 14:37, Måns Rullgård wrote: > Sebastian Frias wrote: > >> By the way, I know some people like the command line, email, etc. but >> there ought to be other tools better suited for patch review... > > Some kernel subsystems use http://patchwork.ozlabs.org/ to track status > of

Re: [PATCH v3] net: ethernet: support "fixed-link" DT node on nb8800 driver

2016-02-08 Thread Måns Rullgård
Sebastian Frias writes: > On 02/08/2016 03:11 PM, Mason wrote: >> On 08/02/2016 14:37, Måns Rullgård wrote: >> >>> Sebastian Frias wrote: >>> By the way, I know some people like the command line, email, etc. but there ought to be other tools better suited for patch

Re: [PATCH v3] net: ethernet: support "fixed-link" DT node on nb8800 driver

2016-02-08 Thread Sebastian Frias
On 02/08/2016 03:11 PM, Mason wrote: On 08/02/2016 14:37, Måns Rullgård wrote: Sebastian Frias wrote: By the way, I know some people like the command line, email, etc. but there ought to be other tools better suited for patch review... Some kernel subsystems use

Re: [PATCH v3] net: ethernet: support "fixed-link" DT node on nb8800 driver

2016-02-08 Thread Måns Rullgård
Sebastian Frias writes: >>> By the way, I know some people like the command line, email, etc. but >>> there ought to be other tools better suited for patch review... >> >> Some kernel subsystems use http://patchwork.ozlabs.org/ to track status >> of various patches. >> > >

Re: [PATCH v3] net: ethernet: support "fixed-link" DT node on nb8800 driver

2016-02-08 Thread Sebastian Frias
On 02/08/2016 02:37 PM, Måns Rullgård wrote: Sebastian Frias writes: On 02/05/2016 04:26 PM, Måns Rullgård wrote: Sebastian Frias writes: On 02/05/2016 04:08 PM, Måns Rullgård wrote: Sebastian Frias writes: On 02/05/2016 03:34 PM,

Re: [PATCH v3] net: ethernet: support "fixed-link" DT node on nb8800 driver

2016-02-08 Thread Måns Rullgård
Sebastian Frias writes: > On 02/05/2016 04:26 PM, Måns Rullgård wrote: >> Sebastian Frias writes: >> >>> On 02/05/2016 04:08 PM, Måns Rullgård wrote: Sebastian Frias writes: > On 02/05/2016 03:34 PM, Måns Rullgård wrote:

Re: [PATCH v3] net: ethernet: support "fixed-link" DT node on nb8800 driver

2016-02-05 Thread Sebastian Frias
On 02/05/2016 04:08 PM, Måns Rullgård wrote: Sebastian Frias writes: On 02/05/2016 03:34 PM, Måns Rullgård wrote: Sebastian Frias writes: Signed-off-by: Sebastian Frias Please change the subject to something like "net: ethernet:

Re: [PATCH v3] net: ethernet: support "fixed-link" DT node on nb8800 driver

2016-02-05 Thread Måns Rullgård
Sebastian Frias writes: > On 02/05/2016 04:08 PM, Måns Rullgård wrote: >> Sebastian Frias writes: >> >>> On 02/05/2016 03:34 PM, Måns Rullgård wrote: Sebastian Frias writes: > Signed-off-by: Sebastian Frias

Re: [PATCH v3] net: ethernet: support "fixed-link" DT node on nb8800 driver

2016-02-05 Thread Måns Rullgård
Sebastian Frias writes: > On 02/05/2016 03:34 PM, Måns Rullgård wrote: >> Sebastian Frias writes: >> >>> Signed-off-by: Sebastian Frias >> >> Please change the subject to something like "net: ethernet: nb8800: >> support fixed-link DT node"

Re: [PATCH v3] net: ethernet: support "fixed-link" DT node on nb8800 driver

2016-02-05 Thread Sebastian Frias
On 02/05/2016 03:34 PM, Måns Rullgård wrote: Sebastian Frias writes: Signed-off-by: Sebastian Frias Please change the subject to something like "net: ethernet: nb8800: support fixed-link DT node" and add a comment body. The subject is pretty explicit