lkml.org 
[lkml]   [2016]   [Mar]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 2/2] misc: nirtfeatures: physical interface elements
On Tue, Mar 15, 2016 at 09:53:27AM -0500, Josh Cartwright wrote:
> Hey Kyle-
>
> On Mon, Mar 14, 2016 at 04:55:08PM -0500, Kyle Roeschley wrote:
> > From: Gratian Crisan <gratian.crisan@ni.com>
>
> From what I understand, this was mostly Aaron's work, so he should get
> authorship. I could be wrong, though, but you'll want to check.
>

Correct, that was just 'git format-patch' grabbing the name of the final
committer on our internal repo. I'll clear that up.

> > These changes add support for PIEs (physical interface elements), which
> > are defined as physical elements fixed to a controller/chassis with
> > which a user can interact (e.g. LEDs and switches) and whose meaning
> > is user-defined and implementation-specific.
> [..]
> > ---
> > drivers/misc/nirtfeatures.c | 753 ++++++++++++++++++++++++++++++++++++++++----
> > 1 file changed, 694 insertions(+), 59 deletions(-)
>
> This patchset is awkwardly split up, especially because you are removing
> what you are adding in the first patch. I would suggest coming up with
> a better patch breakdown that doesn't do this, to make it easier on
> reviewers.
>
> Perhaps, breaking it up into a patchset where each patch implements a
> different class of functionality (leds, input).

Seems reasonable, I'll try to clean this up some.

>
> Josh

--
Kyle Roeschley
Software Engineer
National Instruments

\
 
 \ /
  Last update: 2016-03-15 17:41    [W:0.040 / U:0.072 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site