On Mon, 10 Jan 2011 20:00:27 +0100 Alexander Stein wrote: > features is a bit array, not byte array > > Signed-off-by: Alexander Stein <alexander.stein@xxxxxxxxxxxxxxxxxxxxxxxxx> > Acked-by: Anssi Hannula <anssi.hannula@xxxxxx> > --- > Changes in v2: > * Added proposed space char > > Documentation/input/ff.txt | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/Documentation/input/ff.txt b/Documentation/input/ff.txt > index ded4d5f..d3e046b 100644 > --- a/Documentation/input/ff.txt > +++ b/Documentation/input/ff.txt > @@ -49,7 +49,7 @@ This information is subject to change. > #include <linux/input.h> > #include <sys/ioctl.h> > > -unsigned long features[1 + FF_MAX/sizeof(unsigned long)]; > +unsigned long features[1 + FF_MAX / (8 * sizeof(unsigned long))]; How about: unsigned long features[FF_CNT / (8 * sizeof(unsigned long))]; Dmitry, want to merge some version of this patch?? > int ioctl(int file_descriptor, int request, unsigned long *features); > > "request" must be EVIOCGBIT(EV_FF, size of features array in bytes ) > -- --- ~Randy *** Remember to use Documentation/SubmitChecklist when testing your code *** -- To unsubscribe from this list: send the line "unsubscribe linux-doc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html