Re: [PATCH 3/8] PM / OPP: Manage supply's voltage/current in a separate structure

From	Viresh Kumar <viresh.kumar@linaro.org>
Newsgroups	linux.kernel
Subject	Re: [PATCH 3/8] PM / OPP: Manage supply's voltage/current in a separate structure
Date	2016-10-13 07:50 +0200
Message-ID	<srH3X-7Gj-5@gated-at.bofh.it> (permalink)
References	<sowy5-2bk-13@gated-at.bofh.it> <sowy7-2bk-73@gated-at.bofh.it> <srzpL-1GM-3@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On 12-10-16, 16:15, Dave Gerlach wrote:
> >  /**
> >+ * struct dev_pm_opp_supply - Power supply voltage/current values
> >+ * @u_volt:	Target voltage in microvolts corresponding to this OPP
> >+ * @u_volt_min:	Minimum voltage in microvolts corresponding to thisq OPP
> >+ * @u_volt_max:	Maximum voltage in microvolts corresponding to this OPP
> >+ * @u_amp:	Maximum current drawn by the device in microamperes
> >+ *
> >+ * This structure stores the voltage/current values for a single power supply.
> >+ */
> >+struct dev_pm_opp_supply {
> >+	unsigned long u_volt;
> >+	unsigned long u_volt_min;
> >+	unsigned long u_volt_max;
> >+	unsigned long u_amp;
> >+};
> >+
> >+/**
> 
> This needs to move to include/linux/pm_opp.h, does it not? We need access to
> the actual voltage values from outside of the OPP core if we are going to be
> setting regulators from the platform provided opp_set_rate callback
> described in patch 7.

Yes, will do in V2.

-- 
viresh

Thread

[PATCH 0/8] PM / OPP: Multiple regulator support Viresh Kumar <viresh.kumar@linaro.org> - 2016-10-04 14:00 +0200
  [PATCH 5/8] PM / OPP: Add infrastructure to manage multiple regulators Viresh Kumar <viresh.kumar@linaro.org> - 2016-10-04 14:00 +0200
  [PATCH 3/8] PM / OPP: Manage supply's voltage/current in a separate structure Viresh Kumar <viresh.kumar@linaro.org> - 2016-10-04 14:00 +0200
    Re: [PATCH 3/8] PM / OPP: Manage supply's voltage/current in a  separate structure Dave Gerlach <d-gerlach@ti.com> - 2016-10-12 23:40 +0200
      Re: [PATCH 3/8] PM / OPP: Manage supply's voltage/current in a  separate structure Viresh Kumar <viresh.kumar@linaro.org> - 2016-10-13 07:50 +0200
  [PATCH 1/8] PM / OPP: Reword binding supporting multiple regulators per device Viresh Kumar <viresh.kumar@linaro.org> - 2016-10-04 14:00 +0200
    Re: [PATCH 1/8] PM / OPP: Reword binding supporting multiple  regulators per device Rob Herring <robh@kernel.org> - 2016-10-09 03:50 +0200
  [PATCH 8/8] PM / OPP: Don't WARN on multiple calls to dev_pm_opp_set_regulators() Viresh Kumar <viresh.kumar@linaro.org> - 2016-10-04 14:00 +0200
  Re: [PATCH 0/8] PM / OPP: Multiple regulator support Dave Gerlach <d-gerlach@ti.com> - 2016-10-10 21:30 +0200
    Re: [PATCH 0/8] PM / OPP: Multiple regulator support Viresh Kumar <viresh.kumar@linaro.org> - 2016-10-12 04:20 +0200

csiph-web