Skip to contentSkip to conversation

pxa/colibri270: Fix typo in colibri_pxa270_flash

PATCH New Marek Vasut 6 participants 12 messages Jun 4, 2010 – Jun 12, 2010

Subsystems (3)
lore
Share & tools
lore
#0 lore raw

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
index 96ce4ae..98673ac 100644
@@ -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
#1 lore raw

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
index d60db87..dbfe4ba 100644
@@ -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
index 1963819..83a1a55 100644
@@ -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
index 5305a39..0b36d7d 100644
@@ -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
index 3d284ff..93c11a0 100644
@@ -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
index d8b4469..4eb0258 100644
@@ -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
index ecc1a40..a5429cd 100644
@@ -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
index 3a7925c..b78c6b4 100644
@@ -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
maintainer
#2 lore raw

PATCH 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.h and pass platform data properly.

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

#3 lore raw

PATCH 2/3 pxa: Fix all devices using wm97xx_batt.h

In reply to Marek Vasut

Dne 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.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/palmtreo.c line 28

Quoted 12 lines
arch/arm/mach-pxa/palmtreo.c +0 -1
index d8b4469..4eb0258 100644
@@ -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

#6 lore raw

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
deleted file mode 100644
index a1d6419..0000000
@@ -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
#8 lore raw

PATCH 3/3 wm97xx_batt: Remove redundant header file

In reply to Marek Vasut
Quoted 18 lines · 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 -6
deleted file mode 100644
index a1d6419..0000000
@@ -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.

#9 lore raw

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
---
 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 -13
deleted file mode 100644
index a1d6419..0000000
@@ -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.h shows:

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?

#10 lore raw

PATCH 1/3 pxa/colibri270: Fix typo in colibri_pxa270_flash

In reply to Marek Vasut
Quoted 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
---
 arch/arm/mach-pxa/colibri-pxa270.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
arch/arm/mach-pxa/colibri-pxa270.c +1 -1
index 96ce4ae..98673ac 100644
@@ -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
#11 lore raw

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

---

 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.c index 96ce4ae..98673ac 100644

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) {}