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

2012-06-14 Thread Mohammed, Afzal
Hi, On Thu, Jun 14, 2012 at 15:06:25, Tony Lindgren wrote: > * Mohammed, Afzal [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 sp

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

2012-06-14 Thread Tony Lindgren
* Mohammed, Afzal [120614 01:59]: > Hi Tony, > > On Wed, Jun 13, 2012 at 21:58:53, Hunter, Jon wrote: > > > > Presently there are no peripherals in mainline turning on writeprotect, > > > initially intention was to just disable writeprotect at the end of probe > > > and not provide platforms opp

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

2012-06-14 Thread Mohammed, Afzal
Hi Tony, On Wed, Jun 13, 2012 at 21:58:53, Hunter, Jon wrote: > > Presently there are no peripherals in mainline turning on writeprotect, > > initially intention was to just disable writeprotect at the end of probe > > and not provide platforms opportunity to enable writeprotect as none > > need

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

2012-06-13 Thread Jon Hunter
Hi Afzal, On 06/13/2012 01:10 AM, Mohammed, Afzal wrote: > Hi Jon, > > On Wed, Jun 13, 2012 at 00:12:27, Hunter, Jon wrote: > >> I am still wondering if we should warn against multiple devices using >> the wait pin. I see that if could be valid to have multiple memory >> devices of the same type

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

2012-06-12 Thread Mohammed, Afzal
Hi Jon, On Wed, Jun 13, 2012 at 00:12:27, Hunter, Jon wrote: > I am still wondering if we should warn against multiple devices using > the wait pin. I see that if could be valid to have multiple memory > devices of the same type using the WP pin spanning multiple CS. However, > in that case would

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

2012-06-12 Thread Jon Hunter
On 06/11/2012 09:27 AM, Afzal Mohammed wrote: > GPMC has a writeprotect pin that can be connected to > peripherals. If any CS wants to enable writeprotect, > writeprotect will be enabled, once CS configurations > are finished. I am still wondering if we should warn against multiple devices using