Hi, On 17/09/14 22:00, Fabian Frederick wrote: > Use container_of instead of casting first structure member. > > Signed-off-by: Fabian Frederick <fabf@xxxxxxxxx> > --- > PARISC cross-compiled but untested. > > drivers/video/fbdev/stifb.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/video/fbdev/stifb.c b/drivers/video/fbdev/stifb.c > index cfe8a2f..86621fa 100644 > --- a/drivers/video/fbdev/stifb.c > +++ b/drivers/video/fbdev/stifb.c > @@ -918,7 +918,7 @@ static int > stifb_setcolreg(u_int regno, u_int red, u_int green, > u_int blue, u_int transp, struct fb_info *info) > { > - struct stifb_info *fb = (struct stifb_info *) info; > + struct stifb_info *fb = container_of(info, struct stifb_info, info); > u32 color; > > if (regno >= NR_PALETTE) > @@ -978,7 +978,7 @@ stifb_setcolreg(u_int regno, u_int red, u_int green, > static int > stifb_blank(int blank_mode, struct fb_info *info) > { > - struct stifb_info *fb = (struct stifb_info *) info; > + struct stifb_info *fb = container_of(info, struct stifb_info, info); > int enable = (blank_mode == 0) ? ENABLE : DISABLE; > > switch (fb->id) { > Thanks, queuing the series for 3.18. Tomi
Attachment:
signature.asc
Description: OpenPGP digital signature