Re: [PATCH v2 02/11] mmc: deprecate redundant cd-inverted and wp-inverted DT properties

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Monday 28 January 2013, Chris Ball wrote:
> On Wed, Jan 23 2013, Guennadi Liakhovetski wrote:
> > +cd-inverted and wp-inverted properties are deprecated ans shouldn't be used,
> > +instead pleaseuse the OF_GPIO_ACTIVE_LOW flag in respective GPIO bindings. Note,
> > +that the default (as defined by the SDHCI standard) CD and WP polarity is
> > +active-low, so, OF_GPIO_ACTIVE_LOW should normally be set, and only be left
> > +clear, if the polarity is inverted.
> 
> Please use this text for your next version, fixing typos and a newline:
> 
> cd-inverted and wp-inverted properties are deprecated and shouldn't be used,
> instead please use the OF_GPIO_ACTIVE_LOW flag in respective GPIO bindings.
> Note that the default (as defined by the SDHCI standard) CD and WP polarity
> is active-low, so OF_GPIO_ACTIVE_LOW should normally be set, and only be
> left clear if the polarity is inverted.

Hmm, I wonder if this is possible in general. A lot of the GPIO bindings
allow passing flags, but I think that some of them do not, for historic
reasons. If we want to deprecate the behavior in eMMC, we should also
ensure that all gpio drivers are extended to support gpio specifiers
with flags. It should be possible to extend all drivers in a compatible
way, but someone has to do that.

When we introduced the MMC binding, the situation was already like this,
and it seemed easier to leave the {wp,cd}-inverted properties as optional.

	Arnd
--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux