On Fri, Jan 31, 2014 at 12:32 PM, Frank Praznik <frank.praz...@oh.rr.com> 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.praz...@oh.rr.com> > --- > 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 majord...@vger.kernel.org > 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 majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html