On Fri, Jan 31, 2014 at 12:32 PM, Frank Praznik <frank.praznik@xxxxxxxxx> wrote: > Add the client_addr member to the hid_device struct for storing the client > address of a connected device. > > Signed-off-by: Frank Praznik <frank.praznik@xxxxxxxxx> > --- > include/linux/hid.h | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/include/linux/hid.h b/include/linux/hid.h > index 31b9d29..283fc6e 100644 > --- a/include/linux/hid.h > +++ b/include/linux/hid.h > @@ -495,6 +495,7 @@ struct hid_device { /* device report descriptor */ > char name[128]; /* Device name */ > char phys[64]; /* Device physical location */ > char uniq[64]; /* Device unique identifier (serial #) */ > + __u8 client_addr[6]; /* Device client address (if wireless) */ It is good to make small patches, but this one should definitively be squashed with 2/6. You should be able to say why you add a field in a struct by looking at the code in the patch. Cheers, Benjamin > > void *driver_data; > > -- > 1.8.5.3 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-input" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html