3 Nov
2015
3 Nov
'15
3:59 p.m.
Jagan,
On Tue, Nov 3, 2015 at 12:51 PM, Jagan Teki jteki@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.