lupyuen/local-nuttx-update
1
0
Fork 0
forked from nuttx/nuttx-update
Code Pull requests Activity

drivers/mtd/gd25.c: spi_devid argument was not used in initialization. Commit 4fc41b55b to GD25 driver was always using priv->spi_devid initialized to zero by kmm_zalloc(). This commit initializes it from the supplied function argument.

Browse source
This commit is contained in:
Juha Niskanen 2019-10-31 07:43:21 -06:00 committed by Gregory Nutt
parent 29612c40c5
commit dc3edca5a7
2 changed files with 4 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
drivers/mtd/gd25.c
View file

@ -153,9 +153,9 @@ struct gd25_dev_s
{
struct mtd_dev_s mtd; /* MTD interface */
FAR struct spi_dev_s *spi; /* Saved SPI interface instance */
uint32_t spi_devid; /* Chip select inputs */
uint16_t nsectors; /* Number of erase sectors */
uint8_t prev_instr; /* Previous instruction given to GD25 device */
uint32_t spi_devid; /* Chip select inputs */
bool addr_4byte; /* True: Use Four-byte address */
};
@ -975,6 +975,7 @@ FAR struct mtd_dev_s *gd25_initialize(FAR struct spi_dev_s *spi,
#endif
priv->mtd.name = "gd25";
priv->spi = spi;
priv->spi_devid = spi_devid;
/* Deselect the FLASH */

4
drivers/mtd/gd5f.c
View file

@ -165,11 +165,11 @@ struct gd5f_dev_s
{
struct mtd_dev_s mtd; /* MTD interface */
FAR struct spi_dev_s *dev; /* Saved SPI interface instance */
uint8_t sectorshift; /* 17 */
uint32_t spi_devid; /* Chip select inputs */
uint16_t nsectors; /* 1024 or 2048 */
uint8_t sectorshift; /* 17 */
uint8_t pageshift; /* 11 */
uint8_t eccstatus; /* Internal ECC status */
uint32_t spi_devid; /* Chip select inputs */
};
/************************************************************************************