Subject: [PATCH 6/9] ARM: OMAP1: move omap1_bl pdata out of arch/arm/*

From: [email-masked] (Igor Grinberg)
Date: Thu, 30 Aug 2012 00:42:40 +0300

On 08/29/12 18:04, Tomi Valkeinen wrote:
> On Wed, 2012-08-29 at 02:18 +0300, Igor Grinberg wrote:
>> omap1 backlight platform data resides inside plat/board.h while it
>> should be inside include/linux/...
>> Move the omap1 backlight platform data to
>> include/linux/platform_data/.
>>
>> Cc: Richard Purdie <[email-masked]>
>> Cc: Florian Tobias Schandinat <[email-masked]>
>> Cc: linux-fbdev at vger.kernel.org
>> Signed-off-by: Igor Grinberg <[email-masked]>
>> ---
>>  arch/arm/mach-omap1/board-osk.c         |    1 +
>>  arch/arm/mach-omap1/board-palmte.c      |    1 +
>>  arch/arm/mach-omap1/board-palmtt.c      |    1 +
>>  arch/arm/mach-omap1/board-palmz71.c     |    1 +
>>  arch/arm/plat-omap/include/plat/board.h |    7 -------
>>  drivers/video/backlight/omap1_bl.c      |    2 +-
>>  include/linux/platform_data/omap1_bl.h  |   11 +++++++++++
>>  7 files changed, 16 insertions(+), 8 deletions(-)
>>  create mode 100644 include/linux/platform_data/omap1_bl.h

[...]

>> diff --git a/include/linux/platform_data/omap1_bl.h b/include/linux/platform_data/omap1_bl.h
>> new file mode 100644
>> index 0000000..881a8e9
>> --- /dev/null
>> +++ b/include/linux/platform_data/omap1_bl.h
>> @@ -0,0 +1,11 @@
>> +#ifndef __OMAP1_BL_H__
>> +#define __OMAP1_BL_H__
>> +
>> +#include <linux/device.h>
> 
> You could use struct device; here.

Yes I know, but I don't think it is any better.

> 
>> +
>> +struct omap_backlight_config {
>> +	int default_intensity;
>> +	int (*set_power)(struct device *dev, int state);
>> +};
>> +
>> +#endif
> 
> Acked-by: Tomi Valkeinen <[email-masked]>

Thanks!

- -- 
Regards,
Igor.