From: Nicolas F. <nic...@at...> - 2009-07-06 10:45:21
|
Hi, Nicolas Ferre : > From: Hong Xu <ho...@at...> > > Modify atmel LCD driver: atmel_lcdfb for at91sam9g10. This add a clock > management equivalent to at91sam9261. Can we consider an inclusion in linux-fbdev ? > Signed-off-by: Hong Xu <ho...@at...> > Signed-off-by: Nicolas Ferre <nic...@at...> Andrew Victor added his: Acked-by: Andrew Victor <li...@ma...> > --- > drivers/video/Kconfig | 4 ++-- > drivers/video/atmel_lcdfb.c | 6 ++++-- > drivers/video/backlight/Kconfig | 2 +- > 3 files changed, 7 insertions(+), 5 deletions(-) > > diff --git a/drivers/video/Kconfig b/drivers/video/Kconfig > index 5152168..2dbede4 100644 > --- a/drivers/video/Kconfig > +++ b/drivers/video/Kconfig > @@ -933,7 +933,7 @@ config FB_S1D13XXX > > config FB_ATMEL > tristate "AT91/AT32 LCD Controller support" > - depends on FB && (ARCH_AT91SAM9261 || ARCH_AT91SAM9263 || ARCH_AT91SAM9RL || ARCH_AT91SAM9G45 || ARCH_AT91CAP9 || AVR32) > + depends on FB && (ARCH_AT91SAM9261 || ARCH_AT91SAM9G10 || ARCH_AT91SAM9263 || ARCH_AT91SAM9RL || ARCH_AT91SAM9G45 || ARCH_AT91CAP9 || AVR32) > select FB_CFB_FILLRECT > select FB_CFB_COPYAREA > select FB_CFB_IMAGEBLIT > @@ -949,7 +949,7 @@ config FB_INTSRAM > > config FB_ATMEL_STN > bool "Use a STN display with AT91/AT32 LCD Controller" > - depends on FB_ATMEL && MACH_AT91SAM9261EK > + depends on FB_ATMEL && (MACH_AT91SAM9261EK || MACH_AT91SAM9G10EK) > default n > help > Say Y if you want to connect a STN LCD display to the AT91/AT32 LCD > diff --git a/drivers/video/atmel_lcdfb.c b/drivers/video/atmel_lcdfb.c > index 9a577a8..fc10ac5 100644 > --- a/drivers/video/atmel_lcdfb.c > +++ b/drivers/video/atmel_lcdfb.c > @@ -188,7 +188,8 @@ static unsigned long compute_hozval(unsigned long xres, unsigned long lcdcon2) > { > unsigned long value; > > - if (!(cpu_is_at91sam9261() || cpu_is_at32ap7000())) > + if (!(cpu_is_at91sam9261() || cpu_is_at91sam9g10() > + || cpu_is_at32ap7000())) > return xres; > > value = xres; > @@ -827,7 +828,8 @@ static int __init atmel_lcdfb_probe(struct platform_device *pdev) > info->fix = atmel_lcdfb_fix; > > /* Enable LCDC Clocks */ > - if (cpu_is_at91sam9261() || cpu_is_at32ap7000()) { > + if (cpu_is_at91sam9261() || cpu_is_at91sam9g10() > + || cpu_is_at32ap7000()) { > sinfo->bus_clk = clk_get(dev, "hck1"); > if (IS_ERR(sinfo->bus_clk)) { > ret = PTR_ERR(sinfo->bus_clk); > diff --git a/drivers/video/backlight/Kconfig b/drivers/video/backlight/Kconfig > index f9d19be..90861cd 100644 > --- a/drivers/video/backlight/Kconfig > +++ b/drivers/video/backlight/Kconfig > @@ -110,7 +110,7 @@ config BACKLIGHT_CLASS_DEVICE > config BACKLIGHT_ATMEL_LCDC > bool "Atmel LCDC Contrast-as-Backlight control" > depends on BACKLIGHT_CLASS_DEVICE && FB_ATMEL > - default y if MACH_SAM9261EK || MACH_SAM9263EK > + default y if MACH_SAM9261EK || MACH_SAM9G10EK || MACH_SAM9263EK > help > This provides a backlight control internal to the Atmel LCDC > driver. If the LCD "contrast control" on your board is wired Best regards, -- Nicolas Ferre |
From: Andrew M. <ak...@li...> - 2009-07-20 22:25:08
|
On Mon, 06 Jul 2009 12:32:53 +0200 Nicolas Ferre <nic...@at...> wrote: > Hi, > > Nicolas Ferre : > > From: Hong Xu <ho...@at...> > > > > Modify atmel LCD driver: atmel_lcdfb for at91sam9g10. This add a clock > > management equivalent to at91sam9261. > > Can we consider an inclusion in linux-fbdev ? The patch looks OK to me. > > Signed-off-by: Hong Xu <ho...@at...> > > Signed-off-by: Nicolas Ferre <nic...@at...> > > Andrew Victor added his: > Acked-by: Andrew Victor <li...@ma...> > > > --- > > drivers/video/Kconfig | 4 ++-- > > drivers/video/atmel_lcdfb.c | 6 ++++-- > > drivers/video/backlight/Kconfig | 2 +- > > 3 files changed, 7 insertions(+), 5 deletions(-) Please merge it via an ARM tree? |
From: Nicolas F. <nic...@at...> - 2009-07-21 10:14:10
|
Andrew Morton : > On Mon, 06 Jul 2009 12:32:53 +0200 > Nicolas Ferre <nic...@at...> wrote: > >> Hi, >> >> Nicolas Ferre : >>> From: Hong Xu <ho...@at...> >>> >>> Modify atmel LCD driver: atmel_lcdfb for at91sam9g10. This add a clock >>> management equivalent to at91sam9261. >> Can we consider an inclusion in linux-fbdev ? > > The patch looks OK to me. > >>> Signed-off-by: Hong Xu <ho...@at...> >>> Signed-off-by: Nicolas Ferre <nic...@at...> >> Andrew Victor added his: >> Acked-by: Andrew Victor <li...@ma...> >> >>> --- >>> drivers/video/Kconfig | 4 ++-- >>> drivers/video/atmel_lcdfb.c | 6 ++++-- >>> drivers/video/backlight/Kconfig | 2 +- >>> 3 files changed, 7 insertions(+), 5 deletions(-) > > Please merge it via an ARM tree? Ok, queued in arm patch tracking system with number: 5614/1 http://www.arm.linux.org.uk/developer/patches/viewpatch.php?id=5614/1 Thanks, Bye, -- Nicolas Ferre |