Re: [PATCH v3 12/13] gpiolib: add new ioctl() for monitoring changes in line info
From: Bartosz Golaszewski
Date: Tue Dec 24 2019 - 07:09:53 EST
pt., 20 gru 2019 o 18:29 Bartosz Golaszewski
<bgolaszewski@xxxxxxxxxxxx> napisaÅ(a):
>
> > > struct gpioline_info alone is 32-bit aligned but its size is 72 bytes
> > > which works for 64-bit alignment. This new structure's biggest element
> > > in 64-bit, so it's 64-bit aligned on 64-bit arch. We have 72 bytes of
> > > gpioline_info, 8 bytes of timestamp, 32 bytes of event type and 5 * 32
> > > bytes of padding. Should be fine, but I'll add comments to the header.
> >
> > Yes, what I meant is to add comment at least to struct gpioline_info definition
> > that if somebody would like to change it there (which also might be a
> > problematic here, if there is no versioning scheme / length member).
> >
> > > > > + __u64 timestamp;
> > > > > + __u32 event_type;
> > > > > + __u32 padding[5]; /* for future use */
> > > > > +};
> >
> > Offtopic a bit, had you had a chance to look at Buildroot and our scripts
> > I shared?
> >
>
> Not yet, I'll do that over the weekend.
Now tested using the buildroot image you posted. Thanks for that, I'll
make sure to use it to test the v2 API for lineevents.
Bart