Hi, On Mon, Jul 23, 2012 at 13:56:42, Manjunathappa, Prakash wrote: > Hi, > > On Thu, Jul 19, 2012 at 21:07:46, Manjunathappa, Prakash wrote: > > LCD controller on am335x supports 24bpp raster configuration in addition > > to ones on da850. LCDC also supports 24bpp in unpacked format having > > ARGB:8888 32bpp format data in DDR, but it doesn't interpret alpha > > component of the data. > > > > Signed-off-by: Manjunathappa, Prakash <prakash.pm@xxxxxx> > > Cc: Anatolij Gustschin <agust@xxxxxxx> > > --- [...] > > @@ -546,6 +559,8 @@ static int lcd_cfg_frame_buffer(struct da8xx_fb_par *par, u32 width, u32 height, > > return 0; > > } > > > > + > > +#define CNVT_TOHW(val, width) ((((val)<<(width))+0x7FFF-(val))>>16) > > since multiple FB drivers have re-defined this macro, I will move this to common place(linux/fb.h) and > convert it as inline function. > This macro has lot of variations in various hardware and as I do not have bigger picture to address them all, so for now I will be dropping the attempt of moving it to common place. This will be taken up later point of time. Thanks, Prakash > > > static int fb_setcolreg(unsigned regno, unsigned red, unsigned green, > [...] > -- To unsubscribe from this list: send the line "unsubscribe linux-fbdev" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html