pxa/colibri270: Fix typo in colibri_pxa270_flash
Subsystems (3)
PATCH 1/3 pxa/colibri270: Fix typo in colibri_pxa270_flash
Append colibri_pxa270_flash(_device). This was missing.
--- arch/arm/mach-pxa/colibri-pxa270.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)
arch/arm/mach-pxa/colibri-pxa270.c +1 -1
@@ -99,7 +99,7 @@ static struct platform_device colibri_pxa270_flash_device = {static void __init colibri_pxa270_nor_init (void ) { -platform_device_register (&colibri_pxa270_flash); +platform_device_register (&colibri_pxa270_flash_device); }#else static inline void colibri_pxa270_nor_init (void ) {}
Signature · 1 line
1.7.1
PATCH 2/3 pxa: Fix all devices using wm97xx_batt.h
This file is scheduled for removal, therefore fix all devices that use this file to use the suggested wm97xx.h and pass platform data properly.
--- arch/arm/mach-pxa/mioa701.c | 10 +++++++--- arch/arm/mach-pxa/palmld.c | 19 ++++++++++--------- arch/arm/mach-pxa/palmt5.c | 19 ++++++++++--------- arch/arm/mach-pxa/palmte2.c | 20 ++++++++++++-------- arch/arm/mach-pxa/palmtreo.c | 1 - arch/arm/mach-pxa/palmtx.c | 19 ++++++++++--------- arch/arm/mach-pxa/palmz72.c | 20 ++++++++++++-------- 7 files changed, 61 insertions(+), 47 deletions(-)
arch/arm/mach-pxa/mioa701.c +7 -3
@@ -34,7 +34,7 @@#include <linux/irq.h> #include <linux/pda_power.h> #include <linux/power_supply.h> -#include <linux/wm97xx_batt.h> +#include <linux/wm97xx.h> #include <linux/mtd/physmap.h> #include <linux/usb/gpio_vbus.h> #include <linux/regulator/max1586.h>
@@ -636,7 +636,7 @@ static struct platform_device power_dev = { }, }; -static struct wm97xx_batt_info mioa701_battery_data = { +static struct wm97xx_batt_pdata mioa701_battery_data = { .batt_aux =WM97XX_AUX_ID1 , .temp_aux =-1 , .charge_gpio =-1 ,
@@ -648,6 +648,10 @@ static struct wm97xx_batt_info mioa701_battery_data = { .batt_name = "mioa701_battery", }; +static struct wm97xx_pdata mioa701_wm97xx_pdata = { + .batt_pdata = &mioa701_battery_data, +}; +/* * Voltage regulation */
@@ -716,6 +720,7 @@ struct i2c_pxa_platform_data i2c_pdata = {static pxa2xx_audio_ops_t mioa701_ac97_info = { .reset_gpio =95 , + .codec_pdata = { &mioa701_wm97xx_pdata, }, };/*
@@ -794,7 +799,6 @@ static void __init mioa701_machine_init(void)set_pxa_fb_info (&mioa701_pxafb_info);pxa_set_mci_info (&mioa701_mci_info);pxa_set_keypad_info (&mioa701_keypad_info); -wm97xx_bat_set_pdata (&mioa701_battery_data);pxa_set_udc_info (&mioa701_udc_info);pxa_set_ac97_info (&mioa701_ac97_info); pm_power_off = mioa701_poweroff;
arch/arm/mach-pxa/palmld.c +10 -9
@@ -22,7 +22,7 @@#include <linux/pda_power.h> #include <linux/pwm_backlight.h> #include <linux/gpio.h> -#include <linux/wm97xx_batt.h> +#include <linux/wm97xx.h> #include <linux/power_supply.h> #include <linux/sysdev.h> #include <linux/mtd/mtd.h>
@@ -387,9 +387,9 @@ static struct platform_device power_supply = { };/****************************************************************************** -* WM97xx battery +* WM97xx audio, battery ******************************************************************************/ -static struct wm97xx_batt_info wm97xx_batt_pdata = { +static struct wm97xx_batt_pdata palmld_batt_pdata = { .batt_aux =WM97XX_AUX_ID3 , .temp_aux =WM97XX_AUX_ID2 , .charge_gpio =-1 ,
@@ -403,15 +403,17 @@ static struct wm97xx_batt_info wm97xx_batt_pdata = { .batt_name = "main-batt", }; -/****************************************************************************** -* aSoC audio -******************************************************************************/ -static struct palm27x_asoc_info palmld_asoc_pdata = { - .jack_gpio =GPIO_NR_PALMLD_EARPHONE_DETECT , +static struct wm97xx_pdata palmld_wm97xx_pdata = { + .batt_pdata = &palmld_batt_pdata, };static pxa2xx_audio_ops_t palmld_ac97_pdata = { .reset_gpio =95 , + .codec_pdata = { &palmld_wm97xx_pdata, }, +}; + +static struct palm27x_asoc_info palmld_asoc_pdata = { + .jack_gpio =GPIO_NR_PALMLD_EARPHONE_DETECT , };static struct platform_device palmld_asoc = {
@@ -521,7 +523,6 @@ static void __init palmld_init(void)pxa_set_ac97_info (&palmld_ac97_pdata);pxa_set_ficp_info (&palmld_ficp_platform_data);pxa_set_keypad_info (&palmld_keypad_platform_data); -wm97xx_bat_set_pdata (&wm97xx_batt_pdata);platform_add_devices (devices,ARRAY_SIZE (devices)); }
arch/arm/mach-pxa/palmt5.c +10 -9
@@ -24,7 +24,7 @@#include <linux/pda_power.h> #include <linux/pwm_backlight.h> #include <linux/gpio.h> -#include <linux/wm97xx_batt.h> +#include <linux/wm97xx.h> #include <linux/power_supply.h> #include <linux/usb/gpio_vbus.h>
@@ -298,9 +298,9 @@ static struct platform_device power_supply = { };/****************************************************************************** -* WM97xx battery +* WM97xx audio, battery ******************************************************************************/ -static struct wm97xx_batt_info wm97xx_batt_pdata = { +static struct wm97xx_batt_pdata palmt5_batt_pdata = { .batt_aux =WM97XX_AUX_ID3 , .temp_aux =WM97XX_AUX_ID2 , .charge_gpio =-1 ,
@@ -314,15 +314,17 @@ static struct wm97xx_batt_info wm97xx_batt_pdata = { .batt_name = "main-batt", }; -/****************************************************************************** -* aSoC audio -******************************************************************************/ -static struct palm27x_asoc_info palmt5_asoc_pdata = { - .jack_gpio =GPIO_NR_PALMT5_EARPHONE_DETECT , +static struct wm97xx_pdata palmt5_wm97xx_pdata = { + .batt_pdata = &palmt5_batt_pdata, };static pxa2xx_audio_ops_t palmt5_ac97_pdata = { .reset_gpio =95 , + .codec_pdata = { &palmt5_wm97xx_pdata, }, +}; + +static struct palm27x_asoc_info palmt5_asoc_pdata = { + .jack_gpio =GPIO_NR_PALMT5_EARPHONE_DETECT , };static struct platform_device palmt5_asoc = {
@@ -411,7 +413,6 @@ static void __init palmt5_init(void)pxa_set_ac97_info (&palmt5_ac97_pdata);pxa_set_ficp_info (&palmt5_ficp_platform_data);pxa_set_keypad_info (&palmt5_keypad_platform_data); -wm97xx_bat_set_pdata (&wm97xx_batt_pdata);platform_add_devices (devices,ARRAY_SIZE (devices)); }
arch/arm/mach-pxa/palmte2.c +12 -8
@@ -23,7 +23,7 @@#include <linux/pda_power.h> #include <linux/pwm_backlight.h> #include <linux/gpio.h> -#include <linux/wm97xx_batt.h> +#include <linux/wm97xx.h> #include <linux/power_supply.h> #include <linux/usb/gpio_vbus.h>
@@ -271,9 +271,9 @@ static struct platform_device power_supply = { };/****************************************************************************** -* WM97xx battery +* WM97xx audio, battery ******************************************************************************/ -static struct wm97xx_batt_info wm97xx_batt_pdata = { +static struct wm97xx_batt_pdata palmte2_batt_pdata = { .batt_aux =WM97XX_AUX_ID3 , .temp_aux =WM97XX_AUX_ID2 , .charge_gpio =-1 ,
@@ -287,9 +287,14 @@ static struct wm97xx_batt_info wm97xx_batt_pdata = { .batt_name = "main-batt", }; -/****************************************************************************** -* aSoC audio -******************************************************************************/ +static struct wm97xx_pdata palmte2_wm97xx_pdata = { + .batt_pdata = &palmte2_batt_pdata, +}; + +static pxa2xx_audio_ops_t palmte2_ac97_pdata = { + .codec_pdata = { &palmte2_wm97xx_pdata, }, +}; +static struct palm27x_asoc_info palmte2_asoc_pdata = { .jack_gpio =GPIO_NR_PALMTE2_EARPHONE_DETECT , };
@@ -361,9 +366,8 @@ static void __init palmte2_init(void)set_pxa_fb_info (&palmte2_lcd_screen);pxa_set_mci_info (&palmte2_mci_platform_data);palmte2_udc_init (); -pxa_set_ac97_info (NULL ); +pxa_set_ac97_info (&palmte2_ac97_pdata);pxa_set_ficp_info (&palmte2_ficp_platform_data); -wm97xx_bat_set_pdata (&wm97xx_batt_pdata);platform_add_devices (devices,ARRAY_SIZE (devices)); }
arch/arm/mach-pxa/palmtreo.c +0 -1
@@ -23,7 +23,6 @@#include <linux/pda_power.h> #include <linux/pwm_backlight.h> #include <linux/gpio.h> -#include <linux/wm97xx_batt.h> #include <linux/power_supply.h> #include <linux/sysdev.h> #include <linux/w1-gpio.h>
arch/arm/mach-pxa/palmtx.c +10 -9
@@ -25,7 +25,7 @@#include <linux/pda_power.h> #include <linux/pwm_backlight.h> #include <linux/gpio.h> -#include <linux/wm97xx_batt.h> +#include <linux/wm97xx.h> #include <linux/power_supply.h> #include <linux/usb/gpio_vbus.h> #include <linux/mtd/nand.h>
@@ -359,9 +359,9 @@ static struct platform_device power_supply = { };/****************************************************************************** -* WM97xx battery +* WM97xx audio, battery ******************************************************************************/ -static struct wm97xx_batt_info wm97xx_batt_pdata = { +static struct wm97xx_batt_pdata palmtx_batt_pdata = { .batt_aux =WM97XX_AUX_ID3 , .temp_aux =WM97XX_AUX_ID2 , .charge_gpio =-1 ,
@@ -375,15 +375,17 @@ static struct wm97xx_batt_info wm97xx_batt_pdata = { .batt_name = "main-batt", }; -/****************************************************************************** -* aSoC audio -******************************************************************************/ -static struct palm27x_asoc_info palmtx_asoc_pdata = { - .jack_gpio =GPIO_NR_PALMTX_EARPHONE_DETECT , +static struct wm97xx_pdata palmtx_wm97xx_pdata = { + .batt_pdata = &palmtx_batt_pdata, };static pxa2xx_audio_ops_t palmtx_ac97_pdata = { .reset_gpio =95 , + .codec_pdata = { &palmtx_wm97xx_pdata, }, +}; + +static struct palm27x_asoc_info palmtx_asoc_pdata = { + .jack_gpio =GPIO_NR_PALMTX_EARPHONE_DETECT , };static struct platform_device palmtx_asoc = {
@@ -562,7 +564,6 @@ static void __init palmtx_init(void)pxa_set_ac97_info (&palmtx_ac97_pdata);pxa_set_ficp_info (&palmtx_ficp_platform_data);pxa_set_keypad_info (&palmtx_keypad_platform_data); -wm97xx_bat_set_pdata (&wm97xx_batt_pdata);platform_add_devices (devices,ARRAY_SIZE (devices)); }
arch/arm/mach-pxa/palmz72.c +12 -8
@@ -27,7 +27,7 @@#include <linux/pda_power.h> #include <linux/pwm_backlight.h> #include <linux/gpio.h> -#include <linux/wm97xx_batt.h> +#include <linux/wm97xx.h> #include <linux/power_supply.h> #include <linux/usb/gpio_vbus.h>
@@ -325,9 +325,9 @@ static struct platform_device power_supply = { };/****************************************************************************** -* WM97xx battery +* WM97xx audio, battery ******************************************************************************/ -static struct wm97xx_batt_info wm97xx_batt_pdata = { +static struct wm97xx_batt_pdata palmz72_batt_pdata = { .batt_aux =WM97XX_AUX_ID3 , .temp_aux =WM97XX_AUX_ID2 , .charge_gpio =-1 ,
@@ -341,9 +341,14 @@ static struct wm97xx_batt_info wm97xx_batt_pdata = { .batt_name = "main-batt", }; -/****************************************************************************** -* aSoC audio -******************************************************************************/ +static struct wm97xx_pdata palmz72_wm97xx_pdata = { + .batt_pdata = &palmz72_batt_pdata, +}; + +static pxa2xx_audio_ops_t palmz72_ac97_pdata = { + .codec_pdata = { &palmz72_wm97xx_pdata, }, +}; +static struct platform_device palmz72_asoc = { .name ="palm27x-asoc" , .id =-1 ,
@@ -480,10 +485,9 @@ static void __init palmz72_init(void)set_pxa_fb_info (&palmz72_lcd_screen);pxa_set_mci_info (&palmz72_mci_platform_data);palmz72_udc_init (); -pxa_set_ac97_info (NULL ); +pxa_set_ac97_info (&palmz72_ac97_pdata);pxa_set_ficp_info (&palmz72_ficp_platform_data);pxa_set_keypad_info (&palmz72_keypad_platform_data); -wm97xx_bat_set_pdata (&wm97xx_batt_pdata);platform_add_devices (devices,ARRAY_SIZE (devices)); }
Signature · 1 line
1.7.1
Robert Jarzmik
maintainerPATCH 2/3 pxa: Fix all devices using wm97xx_batt.h
Quoted 4 lines · Marek Vasut
This file is scheduled for removal, therefore fix all devices that use this file to use the suggested
wm97xx.hand pass platform data properly.Signed-off-by Marek Vasut <[email protected]>
Quoted 9 lines
--- arch/arm/mach-pxa/mioa701.c | 10 +++++++--- arch/arm/mach-pxa/palmld.c | 19 ++++++++++--------- arch/arm/mach-pxa/palmt5.c | 19 ++++++++++--------- arch/arm/mach-pxa/palmte2.c | 20 ++++++++++++-------- arch/arm/mach-pxa/palmtreo.c | 1 - arch/arm/mach-pxa/palmtx.c | 19 ++++++++++--------- arch/arm/mach-pxa/palmz72.c | 20 ++++++++++++-------- 7 files changed, 61 insertions(+), 47 deletions(-)
For mioa701:
Cheers.
--
Robert
PATCH 2/3 pxa: Fix all devices using wm97xx_batt.h
In reply to Marek VasutDne P? 4. ?ervna 2010 03:07:33 Marek Vasut napsal(a):
Quoted 14 lines
This file is scheduled for removal, therefore fix all devices that use this file to use the suggested
wm97xx.hand pass platform data properly.Signed-off-by Marek Vasut <[email protected]>--- arch/arm/mach-pxa/mioa701.c | 10 +++++++--- arch/arm/mach-pxa/palmld.c | 19 ++++++++++--------- arch/arm/mach-pxa/palmt5.c | 19 ++++++++++--------- arch/arm/mach-pxa/palmte2.c | 20 ++++++++++++-------- arch/arm/mach-pxa/palmtreo.c | 1 - arch/arm/mach-pxa/palmtx.c | 19 ++++++++++--------- arch/arm/mach-pxa/palmz72.c | 20 ++++++++++++-------- 7 files changed, 61 insertions(+), 47 deletions(-)
...
arch/arm/mach-pxa/palmtreo.c line 28
Quoted 12 lines
arch/arm/mach-pxa/palmtreo.c +0
-1@@ -23,7 +23,6 @@#include <linux/pda_power.h> #include <linux/pwm_backlight.h> #include <linux/gpio.h> -#include <linux/wm97xx_batt.h> #include <linux/power_supply.h> #include <linux/sysdev.h> #include <linux/w1-gpio.h>
Thanks for this one.
For palmtreo.c:
Best regards,
Tomas Cech
PATCH 2/3 pxa: Fix all devices using wm97xx_batt.h
In reply to Marek VasutQuoted 4 lines · Marek Vasut
This file is scheduled for removal, therefore fix all devices that use this file to use the suggested
wm97xx.hand pass platform data properly.Signed-off-by Marek Vasut <[email protected]>
Eric Miao
AppliedPATCH 2/3 pxa: Fix all devices using wm97xx_batt.h
Quoted 8 lines · Mark Brown
Quoted 4 lines · Marek Vasut
This file is scheduled for removal, therefore fix all devices that use this file to use the suggested
wm97xx.hand pass platform data properly.Signed-off-by Marek Vasut <[email protected]>Acked-by Mark Brown <[email protected]>
Applied.
PATCH 3/3 wm97xx_batt: Remove redundant header file
In reply to Marek Vasut--- include/linux/wm97xx_batt.h | 16 ---------------- 1 files changed, 0 insertions(+), 16 deletions(-) delete mode 100644 include/linux/wm97xx_batt.h
include/linux/wm97xx_batt.h deleted +0 -16
@@ -1,16 +0,0 @@ -#ifndef _LINUX_WM97XX_BAT_H -#define _LINUX_WM97XX_BAT_H - -#include <linux/wm97xx.h> - -#warning This file will be removed soon, use wm97xx.h instead! - -#define wm97xx_batt_info wm97xx_batt_pdata - -#ifdef CONFIG_BATTERY_WM97XX -void wm97xx_bat_set_pdata (struct wm97xx_batt_info *data); -#else -static inline void wm97xx_bat_set_pdata (struct wm97xx_batt_info *data) {} -#endif - -#endif
Signature · 1 line
1.7.1
PATCH 3/3 wm97xx_batt: Remove redundant header file
Quoted 1 line · Marek Vasut
Signed-off-by Marek Vasut <[email protected]>
PATCH 3/3 wm97xx_batt: Remove redundant header file
In reply to Marek VasutQuoted 18 lines · Marek Vasut
Signed-off-by Marek Vasut <[email protected]>---
?include/linux/wm97xx_batt.h | ? 16 ---------------- ?1 files changed, 0 insertions(+), 16 deletions(-) ?delete mode 100644include/linux/wm97xx_batt.hinclude/linux/wm97xx_batt.h deleted +0
-6@@ -1,16 +0,0 @@ -#ifndef _LINUX_WM97XX_BAT_H -#define _LINUX_WM97XX_BAT_H - -#include <linux/wm97xx.h> - -#warning This file will be removed soon, use wm97xx.h instead!
include/linux/wm97xx_batt.h line 16
Quoted 10 lines
include/linux/wm97xx_batt.h deleted +0
-10- -#define wm97xx_batt_info wm97xx_batt_pdata - -#ifdef CONFIG_BATTERY_WM97XX -void wm97xx_bat_set_pdata (struct wm97xx_batt_info *data); -#else -static inline void wm97xx_bat_set_pdata (struct wm97xx_batt_info *data) {} -#endif - -#endif
A git grep of wm97xx_batt.h shows:
drivers/power/wm97xx_battery.c:/* The interface is deprecated, as well as linux/wm97xx_batt.h */
So you may want to remove that part as well. Otherwise looks good to me.
PATCH 3/3 wm97xx_batt: Remove redundant header file
Dne So 12. ?ervna 2010 11:16:58 Eric Miao napsal(a):
Quoted 32 lines · Marek Vasut
Quoted 26 lines
Signed-off-by Marek Vasut <[email protected]>--- include/linux/wm97xx_batt.h | 16 ---------------- 1 files changed, 0 insertions(+), 16 deletions(-) delete mode 100644 include/linux/wm97xx_batt.hinclude/linux/wm97xx_batt.h deleted +0
-13@@ -1,16 +0,0 @@ -#ifndef _LINUX_WM97XX_BAT_H -#define _LINUX_WM97XX_BAT_H - -#include <linux/wm97xx.h> - -#warning This file will be removed soon, use wm97xx.h instead! - -#define wm97xx_batt_info wm97xx_batt_pdata - -#ifdef CONFIG_BATTERY_WM97XX -void wm97xx_bat_set_pdata (struct wm97xx_batt_info *data); -#else -static inline void wm97xx_bat_set_pdata (struct wm97xx_batt_info *data){} -#endif
include/linux/wm97xx_batt.h
Quoted 2 lines
include/linux/wm97xx_batt.h deleted +0
-2- -#endif A git grep of
wm97xx_batt.hshows:
Quoted 4 lines
drivers/power/wm97xx_battery.c:/* The interface is deprecated, as well as
linux/wm97xx_batt.h*/So you may want to remove that part as well. Otherwise looks good to me.
Thanks, good catch. But let's push this through a separate patch, shall we?
PATCH 1/3 pxa/colibri270: Fix typo in colibri_pxa270_flash
In reply to Marek VasutQuoted 1 line · Marek Vasut
Append
colibri_pxa270_flash(_device). This was missing.
Hmm, Eric fixed this typo already when commiting (y)our last patch set. Did you pull his tree?
Daniel
Quoted 27 lines
Signed-off-by Marek Vasut <[email protected]>--- arch/arm/mach-pxa/colibri-pxa270.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)arch/arm/mach-pxa/colibri-pxa270.c +1
-1@@ -99,7 +99,7 @@ static struct platform_device colibri_pxa270_flash_device = {static void __init colibri_pxa270_nor_init (void ) { -platform_device_register (&colibri_pxa270_flash); +platform_device_register (&colibri_pxa270_flash_device); }#else static inline void colibri_pxa270_nor_init (void ) {}Signature · 7 lines
1.7.1 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel at lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
PATCH 1/3 pxa/colibri270: Fix typo in colibri_pxa270_flash
Dne P? 4. ?ervna 2010 08:27:02 Daniel Mack napsal(a):
Quoted 5 lines · Marek Vasut
Quoted 1 line
Append
colibri_pxa270_flash(_device). This was missing.Hmm, Eric fixed this typo already when commiting (y)our last patch set. Did you pull his tree?
I think so ... maybe I forget to pull the recent set of changes, then ignore this patch.
Cheers
Quoted 25 lines
Daniel
Quoted 22 lines
Signed-off-by Marek Vasut <[email protected]>---
arch/arm/mach-pxa/colibri-pxa270.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)(quoted diff) +0
-0
b/arch/arm/mach-pxa/colibri-pxa270.cindex 96ce4ae..98673ac 100644arch/arm/mach-pxa/colibri-pxa270.c +1
-1@@ -99,7 +99,7 @@ static struct platform_device colibri_pxa270_flash_device = {static void __init colibri_pxa270_nor_init (void ) { -platform_device_register (&colibri_pxa270_flash); +platform_device_register (&colibri_pxa270_flash_device);}
#else
static inline voidcolibri_pxa270_nor_init(void){}