On Mon, Aug 03, 2020 at 11:18:12AM +0100, Richard Hughes wrote: > On Mon, 3 Aug 2020 at 10:57, Mika Westerberg > <mika.westerb...@linux.intel.com> wrote: > > I think instead of this we should simply make it so that the driver > > never tries to make the chip writable. > > I think this is a good idea, but I wasn't sure if it was an acceptable > behaviour change. Should the driver still try to set BCR_WPD when > writing an image (i.e. defer the setting of write enable until later), > or just not set the BCR register at all? I think your last comment was > the latter, but wanted to check.
I would say not set it at all. I think it was (my) mistake to set it in the first place.
