mmc: atmel: Zap global 'initialized' variable
authorMarek Vasut <marex@denx.de>
Fri, 23 Oct 2015 18:46:31 +0000 (20:46 +0200)
committerAndreas Bießmann <andreas.devel@googlemail.com>
Sun, 1 Nov 2015 21:02:15 +0000 (22:02 +0100)
Global variables are bad. Get rid of this particular one, so we can
correctly instantiate multiple atmel mci interfaces, without having
them interfere with one another.

Signed-off-by: Marek Vasut <marex@denx.de>
Reviewed-by: Andreas Bießmann <andreas.devel@googlemail.com>
drivers/mmc/gen_atmel_mci.c

index 74d53a3140bada62e8f5d2309a45924ca7b4c693..da870c646ebbac06734ca1b7a840bd7b01a23806 100644 (file)
 struct atmel_mci_priv {
        struct mmc_config       cfg;
        struct atmel_mci        *mci;
+       unsigned int            initialized:1;
 };
 
-static int initialized = 0;
-
 /* Read Atmel MCI IP version */
 static unsigned int atmel_mci_get_version(struct atmel_mci *mci)
 {
@@ -121,7 +120,7 @@ static void mci_set_mode(struct mmc *mmc, u32 hz, u32 blklen)
 
        udelay(50);
 
-       initialized = 1;
+       priv->initialized = 1;
 }
 
 /* Return the CMDR with flags for a given command and data packet */
@@ -210,7 +209,7 @@ mci_send_cmd(struct mmc *mmc, struct mmc_cmd *cmd, struct mmc_data *data)
        u32 error_flags = 0;
        u32 status;
 
-       if (!initialized) {
+       if (!priv->initialized) {
                puts ("MCI not initialized!\n");
                return COMM_ERR;
        }
@@ -415,6 +414,7 @@ int atmel_mci_init(void *regs)
        cfg->ops = &atmel_mci_ops;
 
        priv->mci = (struct atmel_mci *)regs;
+       priv->initialized = 0;
 
        /* need to be able to pass these in on a board by board basis */
        cfg->voltages = MMC_VDD_32_33 | MMC_VDD_33_34;