On Sun, Dec 04, 2011 at 12:17:33PM +0100, Valentin Rothberg wrote: > As Dan mentioned, dcon_write() will only write u16 values. The > appropriate parts have been changed. As a result of module_param() > not accepting u16 as a valid data type, unsigned short is used. > > Signed-off-by: Valentin Rothberg <valentinrothberg@xxxxxxxxxxxxxx> > --- > drivers/staging/olpc_dcon/olpc_dcon.c | 8 ++++---- > 1 files changed, 4 insertions(+), 4 deletions(-) > > diff --git a/drivers/staging/olpc_dcon/olpc_dcon.c b/drivers/staging/olpc_dcon/olpc_dcon.c > index 1e64992..1b3847b 100644 > --- a/drivers/staging/olpc_dcon/olpc_dcon.c > +++ b/drivers/staging/olpc_dcon/olpc_dcon.c > @@ -34,8 +34,8 @@ > > /* Module definitions */ > > -static int resumeline = 898; > -module_param(resumeline, int, 0444); > +static unsigned short resumeline = 898; > +module_param(resumeline, unsigned short, 0444); This breaks the compile. You have to use ushort like I explained before. regards, dan carpenter
Attachment:
signature.asc
Description: Digital signature
_______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/devel
