Re: [Resending - PATCH 1/3] Triton BCI driver board/device setup for OMAP3430

2008-08-05 Thread Tony Lindgren
* Felipe Balbi <[EMAIL PROTECTED]> [080805 17:06]:
> On Tue, Aug 05, 2008 at 04:35:51PM +0300, Tony Lindgren wrote:
> > * Felipe Balbi <[EMAIL PROTECTED]> [080723 10:32]:
> > > 
> > > 
> > > On Wed, 23 Jul 2008 10:16:27 +0530, "Madhusudhan Chikkature"
> > > <[EMAIL PROTECTED]> wrote:
> > > > Hi Felipe,
> > > > 
> > > >> Tony, weird that we still have these prototypes in these headers.
> > > >> Could be some merging conflict ?
> > > > Yes. I see that these prototypes are still present in the board3430 and
> > > > board2430 header files in the omap tree.
> > > > 
> > > >> Anyways, please apply the attached patch. We're using
> > > >> usb_musb_init() and usb_ehci_init() now.
> > > > You mean I should apply the attached patch you sent for local use, 
> > > > right?
> > > > And I guess I dont need to resend this perticular BCI patch, am I
> > > correct?
> > > 
> > > No :-)
> > > 
> > > that was to Tony. Just replied to your mail so it's easy to see why we 
> > > need
> > > that patch :-)
> > 
> > Pushing these today. Do you have a patch for moving the extern
> > prototypes into the twl header?
> 
> I was talking about these:
> 
> From: Felipe Balbi <[EMAIL PROTECTED]>
> 
> arch: omap: get rid of usb_init() protytpes
> 
> we now use usb_musb_init() and usb_ehci_init() functions

OK, thanks. Will push today.

Tony

> Signed-off-by: Felipe Balbi <[EMAIL PROTECTED]>
> ---
> 
> diff --git a/include/asm-arm/arch-omap/board-2430sdp.h 
> b/include/asm-arm/arch-omap/board-2430sdp.h
> index fde6915..83d0eec 100644
> --- a/include/asm-arm/arch-omap/board-2430sdp.h
> +++ b/include/asm-arm/arch-omap/board-2430sdp.h
> @@ -36,6 +36,5 @@
>  
>  /* Function prototypes */
>  extern void sdp2430_flash_init(void);
> -extern void sdp2430_usb_init(void);
>  
>  #endif /* __ASM_ARCH_OMAP_2430SDP_H */
> diff --git a/include/asm-arm/arch-omap/board-3430sdp.h 
> b/include/asm-arm/arch-omap/board-3430sdp.h
> index d583008..85f769e 100644
> --- a/include/asm-arm/arch-omap/board-3430sdp.h
> +++ b/include/asm-arm/arch-omap/board-3430sdp.h
> @@ -29,7 +29,6 @@
>  #ifndef __ASM_ARCH_OMAP_3430SDP_H
>  #define __ASM_ARCH_OMAP_3430SDP_H
>  
> -extern void sdp3430_usb_init(void);
>  extern void sdp3430_flash_init(void);
>  
>  #define DEBUG_BASE 0x0800  /* debug board */
> 
> -- 
> balbi
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [Resending - PATCH 1/3] Triton BCI driver board/device setup for OMAP3430

2008-08-05 Thread Felipe Balbi
On Tue, Aug 05, 2008 at 04:35:51PM +0300, Tony Lindgren wrote:
> * Felipe Balbi <[EMAIL PROTECTED]> [080723 10:32]:
> > 
> > 
> > On Wed, 23 Jul 2008 10:16:27 +0530, "Madhusudhan Chikkature"
> > <[EMAIL PROTECTED]> wrote:
> > > Hi Felipe,
> > > 
> > >> Tony, weird that we still have these prototypes in these headers.
> > >> Could be some merging conflict ?
> > > Yes. I see that these prototypes are still present in the board3430 and
> > > board2430 header files in the omap tree.
> > > 
> > >> Anyways, please apply the attached patch. We're using
> > >> usb_musb_init() and usb_ehci_init() now.
> > > You mean I should apply the attached patch you sent for local use, right?
> > > And I guess I dont need to resend this perticular BCI patch, am I
> > correct?
> > 
> > No :-)
> > 
> > that was to Tony. Just replied to your mail so it's easy to see why we need
> > that patch :-)
> 
> Pushing these today. Do you have a patch for moving the extern
> prototypes into the twl header?

I was talking about these:

From: Felipe Balbi <[EMAIL PROTECTED]>

arch: omap: get rid of usb_init() protytpes

we now use usb_musb_init() and usb_ehci_init() functions

Signed-off-by: Felipe Balbi <[EMAIL PROTECTED]>
---

diff --git a/include/asm-arm/arch-omap/board-2430sdp.h 
b/include/asm-arm/arch-omap/board-2430sdp.h
index fde6915..83d0eec 100644
--- a/include/asm-arm/arch-omap/board-2430sdp.h
+++ b/include/asm-arm/arch-omap/board-2430sdp.h
@@ -36,6 +36,5 @@
 
 /* Function prototypes */
 extern void sdp2430_flash_init(void);
-extern void sdp2430_usb_init(void);
 
 #endif /* __ASM_ARCH_OMAP_2430SDP_H */
diff --git a/include/asm-arm/arch-omap/board-3430sdp.h 
b/include/asm-arm/arch-omap/board-3430sdp.h
index d583008..85f769e 100644
--- a/include/asm-arm/arch-omap/board-3430sdp.h
+++ b/include/asm-arm/arch-omap/board-3430sdp.h
@@ -29,7 +29,6 @@
 #ifndef __ASM_ARCH_OMAP_3430SDP_H
 #define __ASM_ARCH_OMAP_3430SDP_H
 
-extern void sdp3430_usb_init(void);
 extern void sdp3430_flash_init(void);
 
 #define DEBUG_BASE 0x0800  /* debug board */

-- 
balbi
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [Resending - PATCH 1/3] Triton BCI driver board/device setup for OMAP3430

2008-08-05 Thread Tony Lindgren
* Felipe Balbi <[EMAIL PROTECTED]> [080723 10:32]:
> 
> 
> On Wed, 23 Jul 2008 10:16:27 +0530, "Madhusudhan Chikkature"
> <[EMAIL PROTECTED]> wrote:
> > Hi Felipe,
> > 
> >> Tony, weird that we still have these prototypes in these headers.
> >> Could be some merging conflict ?
> > Yes. I see that these prototypes are still present in the board3430 and
> > board2430 header files in the omap tree.
> > 
> >> Anyways, please apply the attached patch. We're using
> >> usb_musb_init() and usb_ehci_init() now.
> > You mean I should apply the attached patch you sent for local use, right?
> > And I guess I dont need to resend this perticular BCI patch, am I
> correct?
> 
> No :-)
> 
> that was to Tony. Just replied to your mail so it's easy to see why we need
> that patch :-)

Pushing these today. Do you have a patch for moving the extern
prototypes into the twl header?

Tony
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [Resending - PATCH 1/3] Triton BCI driver board/device setup for OMAP3430

2008-07-23 Thread Felipe Balbi


On Wed, 23 Jul 2008 10:16:27 +0530, "Madhusudhan Chikkature"
<[EMAIL PROTECTED]> wrote:
> Hi Felipe,
> 
>> Tony, weird that we still have these prototypes in these headers.
>> Could be some merging conflict ?
> Yes. I see that these prototypes are still present in the board3430 and
> board2430 header files in the omap tree.
> 
>> Anyways, please apply the attached patch. We're using
>> usb_musb_init() and usb_ehci_init() now.
> You mean I should apply the attached patch you sent for local use, right?
> And I guess I dont need to resend this perticular BCI patch, am I
correct?

No :-)

that was to Tony. Just replied to your mail so it's easy to see why we need
that patch :-)

-- 
Best Regards,

Felipe Balbi
http://blog.felipebalbi.com
[EMAIL PROTECTED]

--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [Resending - PATCH 1/3] Triton BCI driver board/device setup for OMAP3430

2008-07-22 Thread Madhusudhan Chikkature
Hi Felipe,

> Tony, weird that we still have these prototypes in these headers.
> Could be some merging conflict ?
Yes. I see that these prototypes are still present in the board3430 and 
board2430 header files in the omap tree.

> Anyways, please apply the attached patch. We're using
> usb_musb_init() and usb_ehci_init() now.
You mean I should apply the attached patch you sent for local use, right? And I 
guess I dont need to resend this perticular BCI patch, am I correct?

Regards,
Madhu

- Original Message - 
From: "Felipe Balbi" <[EMAIL PROTECTED]>
To: "Tony Lindgren" <[EMAIL PROTECTED]>
Cc: "Madhusudhan Chikkature" <[EMAIL PROTECTED]>; ; 
<[EMAIL PROTECTED]>
Sent: Tuesday, July 22, 2008 6:03 PM
Subject: Re: [Resending - PATCH 1/3] Triton BCI driver board/device setup for 
OMAP3430


> Hi,
> 
> 
> On Tue, 22 Jul 2008 17:25:33 +0530 (IST), "Madhusudhan Chikkature"
> <[EMAIL PROTECTED]> wrote:
>>  extern void sdp3430_usb_init(void);
> 
> Tony, weird that we still have these prototypes in these headers.
> Could be some merging conflict ?
> 
> Anyways, please apply the attached patch. We're using
> usb_musb_init() and usb_ehci_init() now.
> 
> 
> 
> -- 
> Best Regards,
> 
> Felipe Balbi
> http://blog.felipebalbi.com
> [EMAIL PROTECTED]
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [Resending - PATCH 1/3] Triton BCI driver board/device setup for OMAP3430

2008-07-22 Thread Felipe Balbi
Hi,


On Tue, 22 Jul 2008 17:25:33 +0530 (IST), "Madhusudhan Chikkature"
<[EMAIL PROTECTED]> wrote:
>  extern void sdp3430_usb_init(void);

Tony, weird that we still have these prototypes in these headers.
Could be some merging conflict ?

Anyways, please apply the attached patch. We're using
usb_musb_init() and usb_ehci_init() now.



-- 
Best Regards,

Felipe Balbi
http://blog.felipebalbi.com
[EMAIL PROTECTED]From cf96bd037227979a448e03779ec44d0c31c1fa29 Mon Sep 17 00:00:00 2001
From: Felipe Balbi <[EMAIL PROTECTED]>
Date: Tue, 22 Jul 2008 15:07:08 +0300
Subject: [PATCH] arch: omap: remove unused prototypes

We don't have those functions anymore. Probably some
merge conflict put the prototypes back.

Removing them.

Signed-off-by: Felipe Balbi <[EMAIL PROTECTED]>
---
 include/asm-arm/arch-omap/board-2430sdp.h |1 -
 include/asm-arm/arch-omap/board-3430sdp.h |1 -
 2 files changed, 0 insertions(+), 2 deletions(-)

diff --git a/include/asm-arm/arch-omap/board-2430sdp.h b/include/asm-arm/arch-omap/board-2430sdp.h
index fde6915..83d0eec 100644
--- a/include/asm-arm/arch-omap/board-2430sdp.h
+++ b/include/asm-arm/arch-omap/board-2430sdp.h
@@ -36,6 +36,5 @@
 
 /* Function prototypes */
 extern void sdp2430_flash_init(void);
-extern void sdp2430_usb_init(void);
 
 #endif /* __ASM_ARCH_OMAP_2430SDP_H */
diff --git a/include/asm-arm/arch-omap/board-3430sdp.h b/include/asm-arm/arch-omap/board-3430sdp.h
index d583008..85f769e 100644
--- a/include/asm-arm/arch-omap/board-3430sdp.h
+++ b/include/asm-arm/arch-omap/board-3430sdp.h
@@ -29,7 +29,6 @@
 #ifndef __ASM_ARCH_OMAP_3430SDP_H
 #define __ASM_ARCH_OMAP_3430SDP_H
 
-extern void sdp3430_usb_init(void);
 extern void sdp3430_flash_init(void);
 
 #define DEBUG_BASE			0x0800  /* debug board */
-- 
1.5.6.1.156.ge903b



[Resending - PATCH 1/3] Triton BCI driver board/device setup for OMAP3430

2008-07-22 Thread Madhusudhan Chikkature
From: Madhusudhan Chikkature<[EMAIL PROTECTED]>

Device setup/board level initialization for the Triton battery charger
interface driver for the OMAP3430 SDP.

Signed-off-by: Madhusudhan Chikkature<[EMAIL PROTECTED]>
---
 arch/arm/mach-omap2/Makefile  |3 +
 arch/arm/mach-omap2/bci.c |   57 ++
 arch/arm/mach-omap2/board-3430sdp.c   |1
 include/asm-arm/arch-omap/bci.h   |   17 
 include/asm-arm/arch-omap/board-3430sdp.h |1
 5 files changed, 78 insertions(+), 1 deletion(-)

Index: linux-omap-2.6/arch/arm/mach-omap2/Makefile
===
--- linux-omap-2.6.orig/arch/arm/mach-omap2/Makefile2008-07-22
11:58:54.0 +0530
+++ linux-omap-2.6/arch/arm/mach-omap2/Makefile 2008-07-22 12:01:10.0
+0530
@@ -47,7 +47,8 @@ obj-$(CONFIG_MACH_OMAP_3430SDP)   += boar
   hsmmc.o \
   usb-musb.o \
   usb-ehci.o \
-  board-3430sdp-flash.o
+  board-3430sdp-flash.o \
+  bci.o
 obj-$(CONFIG_MACH_OMAP3EVM)+= board-omap3evm.o \
   hsmmc.o \
   usb-musb.o usb-ehci.o \
Index: linux-omap-2.6/arch/arm/mach-omap2/board-3430sdp.c
===
--- linux-omap-2.6.orig/arch/arm/mach-omap2/board-3430sdp.c 2008-07-22
11:58:54.0 +0530
+++ linux-omap-2.6/arch/arm/mach-omap2/board-3430sdp.c  2008-07-22
12:01:10.0 +0530
@@ -299,6 +299,7 @@ static void __init omap_3430sdp_init(voi
ARRAY_SIZE(sdp3430_spi_board_info));
ads7846_dev_init();
sdp3430_flash_init();
+   twl4030_bci_battery_init();
omap_serial_init();
usb_musb_init();
usb_ehci_init();
Index: linux-omap-2.6/include/asm-arm/arch-omap/board-3430sdp.h
===
--- linux-omap-2.6.orig/include/asm-arm/arch-omap/board-3430sdp.h   
2008-07-22
11:58:54.0 +0530
+++ linux-omap-2.6/include/asm-arm/arch-omap/board-3430sdp.h2008-07-22
12:01:10.0 +0530
@@ -31,6 +31,7 @@

 extern void sdp3430_usb_init(void);
 extern void sdp3430_flash_init(void);
+extern void twl4030_bci_battery_init(void);

 #define DEBUG_BASE 0x0800  /* debug board */

Index: linux-omap-2.6/arch/arm/mach-omap2/bci.c
===
--- /dev/null   1970-01-01 00:00:00.0 +
+++ linux-omap-2.6/arch/arm/mach-omap2/bci.c2008-07-22 12:02:39.0 
+0530
@@ -0,0 +1,57 @@
+/*
+ * linux/arch/arm/mach-omap2/bci.c
+ *
+ * TWL4030 BCI platform device setup/initialization
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ */
+
+#include 
+#include 
+#include 
+#include 
+#include 
+
+#if defined(CONFIG_TWL4030_BCI_BATTERY) || \
+   defined(CONFIG_TWL4030_BCI_BATTERY_MODULE)
+/*
+ * Thermistor Calibration for Current Source and MADC
+ * Tolerance (for THS05-3H103F)
+ */
+static int sdp3430_batt_table[] = {
+/* 0 C*/
+30800, 29500, 28300, 27100,
+26000, 24900, 23900, 22900, 22000, 21100, 20300, 19400, 18700, 17900,
+17200, 16500, 15900, 15300, 14700, 14100, 13600, 13100, 12600, 12100,
+11600, 11200, 10800, 10400, 1, 9630,   9280,   8950,   8620,   8310,
+8020,   7730,   7460,   7200,   6950,   6710,   6470,   6250,   6040,   5830,
+5640,   5450,   5260,   5090,   4920,   4760,   4600,   4450,   4310,   4170,
+4040,   3910,   3790,   3670,   3550
+};
+
+static struct twl4030_bci_platform_data sdp3430_bci_data = {
+  .battery_tmp_tbl = sdp3430_batt_table,
+  .tblsize = ARRAY_SIZE(sdp3430_batt_table),
+};
+
+static struct platform_device twl4030_bci_battery_device = {
+   .name   = "twl4030-bci-battery",
+   .id = -1,
+   .dev= {
+   .platform_data  = &sdp3430_bci_data,
+   },
+   .num_resources  = 0,
+};
+
+void __init twl4030_bci_battery_init(void)
+{
+   (void) platform_device_register(&twl4030_bci_battery_device);
+}
+#else
+void __init twl4030_bci_battery_init(void)
+{
+}
+#endif
Index: linux-omap-2.6/include/asm-arm/arch-omap/bci.h
===
--- /dev/null   1970-01-01 00:00:00.0 +
+++ linux-omap-2.6/include/asm-arm/arch-omap/bci.h  2008-07-22
12:01:10.0 +0530
@@ -0,0 +1,17 @@
+/*
+ *  linux/include/asm-arm/arch-omap/bci.h
+ *
+ *  Copyright (C) 2008 Texas Instrum