Hi, > -----Original Message----- > From: Marek Vasut <ma...@denx.de> > Sent: Thursday, December 26, 2024 2:19 AM > To: Abbarapu, Venkatesh <venkatesh.abbar...@amd.com>; u-boot@lists.denx.de; > tudor.amba...@linaro.org; j-humphr...@ti.com > Cc: Simek, Michal <michal.si...@amd.com>; ja...@amarulasolutions.com; > vigne...@ti.com; u-kum...@ti.com; tr...@konsulko.com; sean...@gmail.com; > caleb.conno...@linaro.org; s...@chromium.org; william.zh...@broadcom.com; > stefa...@posteo.net; quentin.sch...@cherry.de; takahiro.kuw...@infineon.com; > p-mant...@ti.com; git (AMD-Xilinx) <g...@amd.com> > Subject: Re: [PATCH v4] mtd: spi-nor: Fix the spi_nor_read() when config > SPI_STACKED_PARALLEL is enabled > > On 12/24/24 4:37 PM, Venkatesh Yadav Abbarapu wrote: > > [...] > > > @@ -1593,18 +1596,22 @@ static int spi_nor_read(struct mtd_info *mtd, loff_t > from, size_t len, > > } > > > > while (len) { > > - bank = (u32)from / SZ_16M; > > - if (nor->flags & SNOR_F_HAS_PARALLEL) > > - bank /= 2; > > + read_len = len; > > + offset = from; > > > > - rem_bank_len = SZ_16M * (bank + 1); > > - if (nor->flags & SNOR_F_HAS_PARALLEL) > > - rem_bank_len *= 2; > > - rem_bank_len -= from; > > + if (CONFIG_IS_ENABLED(SPI_FLASH_BAR)) { > > + bank = (u32)from / SZ_16M; > > + if (nor->flags & SNOR_F_HAS_PARALLEL) > > + bank /= 2; > Is this code which operates on (nor->flags & SNOR_F_HAS_PARALLEL) really > supposed to be enabled if (CONFIG_IS_ENABLED(SPI_FLASH_BAR)) is SET or > instead if STACKED_PARALLEL symbol is SET ?
The FLASH_BAR and STACKED_PARALLEL configs are independent. They won't depend on each other. Thanks Venkatesh