Dear Bo Shen, [...] > +/* > + * the CLUT register map as following > + * RCLUT(24 ~ 16), GCLUT(15 ~ 8), BCLUT(7 ~ 0) > + */ > +void lcd_setcolreg(ushort regno, ushort red, ushort green, ushort blue) > +{ > + lcdc_writel((red << 16) & 0xff0000 | (green << 8) & 0xff00 | > + blue & 0xff, panel_info.mmio + ATMEL_LCDC_LUT(regno)); > +} > +
Why don't you #define these values instead? > void lcd_ctrl_init(void *lcdbase) > { > unsigned long value; Best regards, Marek Vasut _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot