[PATCH v5 14/14] ARM: OMAP2+: gpmc: writeprotect helper

[Mohammed, Afzal]

Hi,

On Thu, Jun 14, 2012 at 15:06:25, Tony Lindgren wrote:
> * Mohammed, Afzal <afzal at ti.com> [120614 01:59]:
> > On Wed, Jun 13, 2012 at 21:58:53, Hunter, Jon wrote:

> > > devices should indicate if they use the write-protect pin and we should
> > > either have this "enable on boot" as a global setting not specific to a
> > > child device or ensure that multiple devices using the wp have the same
> > > configuration of the wp on boot. In other words, if one device says
> > > enable on boot and the other does not, then warn.
> > 
> > Which one of Jon's above suggestions would you prefer,
> > 
> > i.e.
> > writeprotect as a global setting for all devices
> > Or
> > Ensure that multiple devices using wp have same configuration
> > 
> > Or anything else you have in mind ?
> 
> If it's shared it can cause undesired effects on the mounted rootfs
> if one device changes it. So some kind of checking for a shared case
> should be available before a device is allowed to change it.

Ok, I will add a check for shared case.

Regards
Afzal