ti_pci1410a.c: In function 'voltage_set':
ti_pci1410a.c:421: error: label at end of compound statement
ti_pci1410a.c:435: error: label at end of compound statement
cc1: warnings being treated as errors
ti_pci1410a.c: At top level:
ti_pci1410a.c:609: warning: pointer targets in initialization differ in 
signedness
ti_pci1410a.c: In function 'identify':
ti_pci1410a.c:651: warning: pointer targets in passing argument 1 of 'puts' 
differ in signedness
ti_pci1410a.c:656: warning: pointer targets in passing argument 1 of 'strcmp' 
differ in signedness
ti_pci1410a.c:656: warning: pointer targets in passing argument 2 of 'strcmp' 
differ in signedness

remove do_pinit allready implement in cmd_pcmcia.c
Fix pcmcia_{on,off} protoptype and add it in definition to pcmcia.h

Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <[EMAIL PROTECTED]>
---
 drivers/pcmcia/ti_pci1410a.c |   61 +++++++++--------------------------------
 include/pcmcia.h             |    2 +
 2 files changed, 16 insertions(+), 47 deletions(-)

diff --git a/drivers/pcmcia/ti_pci1410a.c b/drivers/pcmcia/ti_pci1410a.c
index c876d0c..d37e5a8 100644
--- a/drivers/pcmcia/ti_pci1410a.c
+++ b/drivers/pcmcia/ti_pci1410a.c
@@ -62,13 +62,10 @@
 #include <pci.h>
 #include <asm/io.h>
 
-#include <pcmcia.h>
-
 #if defined(CONFIG_CMD_PCMCIA)
 
-int pcmcia_on(int ide_base_bus);
+#include <pcmcia.h>
 
-static int  pcmcia_off(void);
 static int  hardware_disable(int slot);
 static int  hardware_enable(int slot);
 static int  voltage_set(int slot, int vcc, int vpp);
@@ -77,42 +74,13 @@ static void print_fixed(volatile uchar *p);
 static int  identify(volatile uchar *p);
 static int  check_ide_device(int slot, int ide_base_bus);
 
-
 /* ------------------------------------------------------------------------- */
 
-
 const char *indent = "\t   ";
 
 /* ------------------------------------------------------------------------- */
 
 
-int do_pinit(cmd_tbl_t *cmdtp, int flag, int argc, char *argv[])
-{
-#ifndef CFG_FIRST_PCMCIA_BUS
-# define CFG_FIRST_PCMCIA_BUS 0
-#endif
-
-       int rcode = 0;
-
-       if (argc != 2) {
-               printf ("Usage: pinit {on | off}\n");
-               return 1;
-       }
-       if (strcmp(argv[1],"on") == 0) {
-               rcode = pcmcia_on(CFG_FIRST_PCMCIA_BUS);
-       } else if (strcmp(argv[1],"off") == 0) {
-               rcode = pcmcia_off();
-       } else {
-               printf ("Usage: pinit {on | off}\n");
-               return 1;
-       }
-
-       return rcode;
-}
-
-/* ------------------------------------------------------------------------- */
-
-
 static struct pci_device_id supported[] = {
        { PCI_VENDOR_ID_TI, 0xac50 }, /* Ti PCI1410A */
        { PCI_VENDOR_ID_TI, 0xac56 }, /* Ti PCI1510 */
@@ -123,15 +91,18 @@ static pci_dev_t devbusfn;
 static u32 socket_base;
 static u32 pcmcia_cis_ptr;
 
-int pcmcia_on(int ide_base_bus)
+int pcmcia_on(void)
 {
+#ifndef CFG_FIRST_PCMCIA_BUS
+# define CFG_FIRST_PCMCIA_BUS 0
+#endif
        u16 dev_id;
        u32 socket_status;
        int slot = 0;
        int cis_len;
        u16 io_base;
        u16 io_len;
-
+       int ide_base_bus = CFG_FIRST_PCMCIA_BUS;
        /*
         * Find the CardBus PCI device(s).
         */
@@ -244,8 +215,7 @@ int pcmcia_on(int ide_base_bus)
 
 /* ------------------------------------------------------------------------- */
 
-
-static int pcmcia_off (void)
+int pcmcia_off (void)
 {
        int slot = 0;
 
@@ -285,10 +255,8 @@ static int pcmcia_off (void)
        return 0;
 }
 
-
 /* ------------------------------------------------------------------------- */
 
-
 #define        MAX_TUPEL_SZ    512
 #define MAX_FEATURES   4
 int ide_devices_found;
@@ -392,7 +360,6 @@ static int check_ide_device(int slot, int ide_base_bus)
        return 0;
 }
 
-
 static int voltage_set(int slot, int vcc, int vpp)
 {
        u32 socket_control;
@@ -418,6 +385,7 @@ static int voltage_set(int slot, int vcc, int vpp)
                break;
        case 0:
        default:
+               break;
        }
 
        switch (vpp) {
@@ -432,6 +400,7 @@ static int voltage_set(int slot, int vcc, int vpp)
                break;
        case 0:
        default:
+               break;
        }
 
        writel(socket_control, reg);
@@ -443,7 +412,6 @@ static int voltage_set(int slot, int vcc, int vpp)
        return 0;
 }
 
-
 static int hardware_enable(int slot)
 {
        u32 socket_status;
@@ -490,7 +458,6 @@ static int hardware_enable(int slot)
        return ((readb(socket_base+0x801)&0x6c)==0x6c)?0:1;
 }
 
-
 static int hardware_disable(int slot)
 {
        voltage_set(slot, 0, 0);
@@ -605,17 +572,17 @@ static void print_fixed(volatile uchar *p)
 #define MAX_IDENT_CHARS                64
 #define        MAX_IDENT_FIELDS        4
 
-static uchar *known_cards[] = {
+static char *known_cards[] = {
        "ARGOSY PnPIDE D5",
        NULL
 };
 
 static int identify(volatile uchar *p)
 {
-       uchar id_str[MAX_IDENT_CHARS];
+       char id_str[MAX_IDENT_CHARS];
        uchar data;
-       uchar *t;
-       uchar **card;
+       char *t;
+       char **card;
        int i, done;
 
        if (p == NULL)
@@ -648,7 +615,7 @@ static int identify(volatile uchar *p)
                        break;
                }
        }
-       puts(id_str);
+       puts((char*)id_str);
        putc('\n');
 
        for (card=known_cards; *card; ++card) {
diff --git a/include/pcmcia.h b/include/pcmcia.h
index 7305805..cc4c78c 100644
--- a/include/pcmcia.h
+++ b/include/pcmcia.h
@@ -318,4 +318,6 @@ extern u_int *pcmcia_pgcrx[];
 extern int check_ide_device(int slot);
 #endif
 
+int pcmcia_off(void);
+int pcmcia_on(void);
 #endif /* _PCMCIA_H */
-- 
1.5.5.1


-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft 
Defy all challenges. Microsoft(R) Visual Studio 2008. 
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Reply via email to