Re: [PATCH] mmc: core: Quieten "could not set regulator OCR" error.

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

 



Hi,

On Thu, Aug 30 2012, Mark Brown wrote:
>> Looks like regulator_desc->name should be "dummy", but I think it would
>> be an API violation for us to start caring about that.  Mark, would you
>> be interested in offering an "regulator_is_dummy(supply)" function so
>> that we can limit our error messages to non-dummy supplies?
>
> No, that's a terrible idea as previously discussed.
>
> The *sole* purpose of the dummy regulator driver is to provide a crutch
> to keep systems with broken configuraitons running, it's not indended to
> be used on production systems and is there solely as an aid for
> development.  This is why it can only be enabled via Kconfig.  If there
> is no software controllable regulator there the board should be
> providing fixed regulator.

It's being used on production systems, else I wouldn't be responding
to bug reports from distro users of x86 laptops.  Turns out that distro
kernel maintainers often like to answer "y" to new Kconfig options
without reading/understanding them fully.

> Conversely there is nothing stopping other regulator drivers having
> similar properties to the dummy driver, or constraints stopping them
> implementing things.  If the MMC framework wants to silently accept
> regulators that don't allow the voltage to be set then the MMC framework
> should enumerate the range of voltages which can be set using the
> regulator it has and then take appropriate action.

I didn't do this because I don't want to silently accept real regulators
with insufficient voltage ranges, because that would make it harder to
debug real MMC failures.

Sounds like my response to the bug reports should be "persuade your
distro kernel maintainers to turn off CONFIG_REGULATOR" -- I'll try
doing that.

Thanks,

- Chris.
-- 
Chris Ball   <cjb@xxxxxxxxxx>   <http://printf.net/>
One Laptop Per Child
--
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