diff options
author | Ryan Mallon <rmallon@gmail.com> | 2012-01-22 04:31:32 -0500 |
---|---|---|
committer | Ryan Mallon <rmallon@gmail.com> | 2012-03-13 20:41:10 -0400 |
commit | 0fd1958050e92c859152e775e548284582335d25 (patch) | |
tree | dd457bc43e459ff65aa0dba5034ec52de5d43364 /drivers/video/backlight | |
parent | 2ae18b471d91f7622e54f18ed3a4b5b20e9bf871 (diff) |
ep93xx: Use ioremap for backlight driver
The ep93xx backlight driver uses a single register within the
framebuffer's register space. Currently the backlight driver uses a
static IO mapping for the register since the memory cannot be
requested by both drivers.
Convert the static mapping to use ioremap so that we can remove the
dependency on mach/hardware.h. To do so, we need remove the
request_mem_region from both the backlight and framebuffer drivers,
since whichever driver is loaded second will fail with -EBUSY
otherwise.
A proper fix is still required, and a FIXME comment has been added to
both drivers.
Signed-off-by: Ryan Mallon <rmallon@gmail.com>
Suggested-by: Arnd Bergmann <arnd@arndb.de>
Cc: Mika Westerberg <mika.westerberg@iki.fi>
Cc: Richard Purdie <rpurdie@rpsys.net>
Cc: Florian Tobias Schandinat <FlorianSchandinat@gmx.de>
Acked-by: Arnd Bergmann <arnd@arndb.de>
Acked-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Diffstat (limited to 'drivers/video/backlight')
-rw-r--r-- | drivers/video/backlight/ep93xx_bl.c | 25 |
1 files changed, 14 insertions, 11 deletions
diff --git a/drivers/video/backlight/ep93xx_bl.c b/drivers/video/backlight/ep93xx_bl.c index b62b8b9063b5..08214e1f0958 100644 --- a/drivers/video/backlight/ep93xx_bl.c +++ b/drivers/video/backlight/ep93xx_bl.c | |||
@@ -17,11 +17,6 @@ | |||
17 | #include <linux/fb.h> | 17 | #include <linux/fb.h> |
18 | #include <linux/backlight.h> | 18 | #include <linux/backlight.h> |
19 | 19 | ||
20 | #include <mach/hardware.h> | ||
21 | |||
22 | #define EP93XX_RASTER_REG(x) (EP93XX_RASTER_BASE + (x)) | ||
23 | #define EP93XX_RASTER_BRIGHTNESS EP93XX_RASTER_REG(0x20) | ||
24 | |||
25 | #define EP93XX_MAX_COUNT 255 | 20 | #define EP93XX_MAX_COUNT 255 |
26 | #define EP93XX_MAX_BRIGHT 255 | 21 | #define EP93XX_MAX_BRIGHT 255 |
27 | #define EP93XX_DEF_BRIGHT 128 | 22 | #define EP93XX_DEF_BRIGHT 128 |
@@ -35,7 +30,7 @@ static int ep93xxbl_set(struct backlight_device *bl, int brightness) | |||
35 | { | 30 | { |
36 | struct ep93xxbl *ep93xxbl = bl_get_data(bl); | 31 | struct ep93xxbl *ep93xxbl = bl_get_data(bl); |
37 | 32 | ||
38 | __raw_writel((brightness << 8) | EP93XX_MAX_COUNT, ep93xxbl->mmio); | 33 | writel((brightness << 8) | EP93XX_MAX_COUNT, ep93xxbl->mmio); |
39 | 34 | ||
40 | ep93xxbl->brightness = brightness; | 35 | ep93xxbl->brightness = brightness; |
41 | 36 | ||
@@ -70,21 +65,29 @@ static int __init ep93xxbl_probe(struct platform_device *dev) | |||
70 | struct ep93xxbl *ep93xxbl; | 65 | struct ep93xxbl *ep93xxbl; |
71 | struct backlight_device *bl; | 66 | struct backlight_device *bl; |
72 | struct backlight_properties props; | 67 | struct backlight_properties props; |
68 | struct resource *res; | ||
73 | 69 | ||
74 | ep93xxbl = devm_kzalloc(&dev->dev, sizeof(*ep93xxbl), GFP_KERNEL); | 70 | ep93xxbl = devm_kzalloc(&dev->dev, sizeof(*ep93xxbl), GFP_KERNEL); |
75 | if (!ep93xxbl) | 71 | if (!ep93xxbl) |
76 | return -ENOMEM; | 72 | return -ENOMEM; |
77 | 73 | ||
74 | res = platform_get_resource(dev, IORESOURCE_MEM, 0); | ||
75 | if (!res) | ||
76 | return -ENXIO; | ||
77 | |||
78 | /* | 78 | /* |
79 | * This register is located in the range already ioremap'ed by | 79 | * FIXME - We don't do a request_mem_region here because we are |
80 | * the framebuffer driver. A MFD driver seems a bit of overkill | 80 | * sharing the register space with the framebuffer driver (see |
81 | * to handle this so use the static I/O mapping; this address | 81 | * drivers/video/ep93xx-fb.c) and doing so will cause the second |
82 | * is already virtual. | 82 | * loaded driver to return -EBUSY. |
83 | * | 83 | * |
84 | * NOTE: No locking is required; the framebuffer does not touch | 84 | * NOTE: No locking is required; the framebuffer does not touch |
85 | * this register. | 85 | * this register. |
86 | */ | 86 | */ |
87 | ep93xxbl->mmio = EP93XX_RASTER_BRIGHTNESS; | 87 | ep93xxbl->mmio = devm_ioremap(&dev->dev, res->start, |
88 | resource_size(res)); | ||
89 | if (!ep93xxbl->mmio) | ||
90 | return -ENXIO; | ||
88 | 91 | ||
89 | memset(&props, 0, sizeof(struct backlight_properties)); | 92 | memset(&props, 0, sizeof(struct backlight_properties)); |
90 | props.type = BACKLIGHT_RAW; | 93 | props.type = BACKLIGHT_RAW; |