Re: [PATCH v3 3/5] char: ti-usim: Add driver for USIM module on AM43xx

From: Greg Kroah-Hartman
Date: Thu May 29 2014 - 11:50:20 EST


On Thu, May 29, 2014 at 03:35:37PM +0530, Satish Patel wrote:
> >>+enum usim_card_mode {
> >>+ USIM_CARD_MODE_ASYNC = 0, /* asynchronous mode */
> >>+ USIM_CARD_MODE_SYNC_TYPE1, /* synchronous mode: Type 1 */
> >>+ USIM_CARD_MODE_SYNC_TYPE2, /* synchronous mode: Type 2 */
> >>+ USIM_CARD_MODE_SYNC_OTHER, /* Any other synchronous type */
> >>+};
> >>+struct usim_data {
> >>+ int slot;
> >>+ int rxexplen;
> >>+ int txlen;
> >>+ unsigned char apdu[256];
> >>+};

You need to use the proper variable types for a structure that is going
to cross the user/kernel boundry in an ioctl :(

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/