[U-Boot] [PATCH v5 17/18] spi: Add SPI NOR protection mechanism

[Fabio Estevam]

Jagan,

On Tue, Nov 3, 2015 at 12:51 PM, Jagan Teki <jteki at openedev.com> wrote:

>> +       if (flash->is_locked(flash, offset, len) > 0) {
>> +               printf("offset 0x%x is protected and cannot be written\n", offset);
>> +               return -EINVAL;
>> +       }
>
> This flash lock check related to non-dm lock ops so it's look not good
> to me as we assigned lock ops for both dm and non-dm cases.

This is getting totally ridiculous: you have been seeing this patch
series so many times and now you say this is not OK and you don't give
me any clue as to what you would like me to change.

That's why I proposed earlier that you take the non-SPI patches first
and then we can rework the SPI flash pieces.

> I will come back for this patch changes what I thought, probably lock
> ops shouldn't be separate for dm and non-dm as we have different ops
> based on the flash itself.

I wish we can make progress on this. Not sure how to proceed though.