Hi Bing,
On Mon, Aug 7, 2017 at 1:09 PM, Bin Meng <bmeng...@gmail.com> wrote:
> On Fri, Aug 4, 2017 at 12:21 PM, Bin Meng <bmeng...@gmail.com> wrote:
>> On Wed, Aug 2, 2017 at 6:26 AM, Bin Meng <bmeng...@gmail.com> wrote:
>>> Hi Jagan,
>>>
>>> On Wed, Aug 2, 2017 at 12:01 AM, Jagan Teki <jagannadh.t...@gmail.com>
>>> wrote:
>>>> On Sun, Jul 23, 2017 at 8:14 PM, Bin Meng <bmeng...@gmail.com> wrote:
>>>>> On some flash (like Macronix), QE (quad enable) bit is in the same
>>>>> status register as BP# bits, and we need preserve its original value
>>>>> during a reboot cycle as this is required by some platforms (like
>>>>> Intel ICH SPI controller working under descriptor mode).
>>>>>
>>>>> Signed-off-by: Bin Meng <bmeng...@gmail.com>
>>>>> ---
>>>>>
>>>>> drivers/mtd/spi/spi_flash.c | 17 +++++++++++++++--
>>>>> 1 file changed, 15 insertions(+), 2 deletions(-)
>>>>>
>>>>> diff --git a/drivers/mtd/spi/spi_flash.c b/drivers/mtd/spi/spi_flash.c
>>>>> index 0034a28..7d8c660 100644
>>>>> --- a/drivers/mtd/spi/spi_flash.c
>>>>> +++ b/drivers/mtd/spi/spi_flash.c
>>>>> @@ -947,11 +947,24 @@ int spi_flash_scan(struct spi_flash *flash)
>>>>> if (IS_ERR_OR_NULL(info))
>>>>> return -ENOENT;
>>>>>
>>>>> - /* Flash powers up read-only, so clear BP# bits */
>>>>> + /*
>>>>> + * Flash powers up read-only, so clear BP# bits.
>>>>> + *
>>>>> + * Note on some flash (like Macronix), QE (quad enable) bit is in
>>>>> the
>>>>> + * same status register as BP# bits, and we need preserve its
>>>>> original
>>>>> + * value during a reboot cycle as this is required by some
>>>>> platforms
>>>>> + * (like Intel ICH SPI controller working under descriptor mode).
>>>>> + */
>>>>> if (JEDEC_MFR(info) == SPI_FLASH_CFI_MFR_ATMEL ||
>>>>> - JEDEC_MFR(info) == SPI_FLASH_CFI_MFR_MACRONIX ||
>>>>> JEDEC_MFR(info) == SPI_FLASH_CFI_MFR_SST)
>>>>> write_sr(flash, 0);
>>>>> + if (JEDEC_MFR(info) == SPI_FLASH_CFI_MFR_MACRONIX) {
>>>>> + u8 sr = 0;
>>>>> +
>>>>> + read_sr(flash, &sr);
>>>>> + sr &= STATUS_QEB_MXIC;
>>>>> + write_sr(flash, sr);
>>>>> + }
>>>>
>>>> It doesn't make sense to have one(specific) controller fix to be
>>>> generic to all macronix chips, think about alternative.
>>>>
>>>
>>> This is no way to fix at the controller level. Actually this is
>>> nothing related to controller level. It's just the bootstrap settings
>>> (QE bit in this case) that cannot be overwritten by someone else
>>> (although it's seen on Intel, it might happen on some other
>>> architecture). The logic in the codes are having issues. Its comment
>>> says: clear BP# bits, but it clears QE bit for Macronix flash as well,
>>> which is wrong. The update was just to make sure the codes do as what
>>> its comment says.
>>>
>>> If you have any other alternative, please suggest.
>>>
>>
>> Ping again..
Wait for sometime, I've queue that I need to review-it and respond
accordingly patches with latest may take some time. And commenting
yes will respond soon.
thanks!
--
Jagan Teki
Free Software Engineer | www.openedev.com
U-Boot, Linux | Upstream Maintainer
Hyderabad, India.
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot
