diff options
| author | Robby Cai <R63905@freescale.com> | 2012-01-20 17:31:08 +0800 |
|---|---|---|
| committer | Robby Cai <R63905@freescale.com> | 2012-01-20 17:35:57 +0800 |
| commit | 1a5fa49e7fc5f3516dfa667371c2be6b57a6ab32 (patch) | |
| tree | 68ad5f1c4c877892bc1590fff32b825227f75752 /drivers | |
| parent | 630bee9f3f76f5f6724d7b4ce10dd3239ee28553 (diff) | |
ENGR00172490: FIX: PMIC registers may not be accessible in u-boot via SPI
The system PMIC registers may not be accessible in u-boot via SPI if
function pmic_reg() is called in the latter part of boot up process in u-boot.
It is because the imx_spi_slave structure is allocated from malloc() in
the spi_setup_slave() function. However, this structure is not completely
initialized, which may result in using a dirty control register value
at CSPI during transfer.
memset() the imx_spi_slave structure after malloc() can resolve this problem
Please refer to CT39243849.
Signed-off-by: Robby Cai <R63905@freescale.com>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/spi/imx_cspi.c | 4 | ||||
| -rw-r--r-- | drivers/spi/imx_ecspi.c | 4 |
2 files changed, 6 insertions, 2 deletions
diff --git a/drivers/spi/imx_cspi.c b/drivers/spi/imx_cspi.c index 438ab8df5a..1f4e11752b 100644 --- a/drivers/spi/imx_cspi.c +++ b/drivers/spi/imx_cspi.c @@ -1,5 +1,5 @@ /* - * (C) Copyright 2008-2010 Freescale Semiconductor, Inc. + * (C) Copyright 2008-2012 Freescale Semiconductor, Inc. * * See file CREDITS for list of people who contributed to this * project. @@ -116,6 +116,8 @@ struct spi_slave *spi_setup_slave(unsigned int bus, unsigned int cs, if (!imx_spi_slave) return NULL; + memset(imx_spi_slave, 0, sizeof(struct imx_spi_dev_t)); + imx_spi_slave->slave.bus = bus; imx_spi_slave->slave.cs = cs; diff --git a/drivers/spi/imx_ecspi.c b/drivers/spi/imx_ecspi.c index 1e86cf1fae..2e26205d89 100644 --- a/drivers/spi/imx_ecspi.c +++ b/drivers/spi/imx_ecspi.c @@ -1,5 +1,5 @@ /* - * (C) Copyright 2008-2010 Freescale Semiconductor, Inc. + * (C) Copyright 2008-2012 Freescale Semiconductor, Inc. * * See file CREDITS for list of people who contributed to this * project. @@ -152,6 +152,8 @@ struct spi_slave *spi_setup_slave(unsigned int bus, unsigned int cs, if (!imx_spi_slave) return NULL; + memset(imx_spi_slave, 0, sizeof(struct imx_spi_dev_t)); + imx_spi_slave->slave.bus = bus; imx_spi_slave->slave.cs = cs; |