Re: [PATCH v6 1/3] mfd: devicetree: bindings: Add Qualcomm RPM DT binding

2014-09-17 Thread Mark Brown
On Wed, Sep 17, 2014 at 11:19:40AM -0700, Bjorn Andersson wrote:

> The regulator driver have a strong dependency on the mfd driver, but Kconfig
> will make sure it's not selectable until the two parts merges again, so it
> should be fine. But I guess it's up to Mark?

Yes, that's the normal approach.  I'd have applied it but a lot of the
early discussion seemed to be about the register I/O interface.


signature.asc
Description: Digital signature


Re: [PATCH v6 1/3] mfd: devicetree: bindings: Add Qualcomm RPM DT binding

2014-09-17 Thread Bjorn Andersson
On Wed 17 Sep 09:15 PDT 2014, Lee Jones wrote:

> On Tue, 26 Aug 2014, Lee Jones wrote:
> 
> > On Mon, 25 Aug 2014, Bjorn Andersson wrote:
> > 
> > > Add binding for the Qualcomm Resource Power Manager (RPM) found in 8660,
> > > 8960 and 8064 based devices. The binding currently describes the rpm
> > > itself and the regulator subnodes.
> > > 
> > > Signed-off-by: Bjorn Andersson 
> > > ---
> > >  Documentation/devicetree/bindings/mfd/qcom,rpm.txt |  246 
> > > 
> > >  include/dt-bindings/mfd/qcom,rpm.h |  154 
> > >  2 files changed, 400 insertions(+)
> > >  create mode 100644 Documentation/devicetree/bindings/mfd/qcom,rpm.txt
> > >  create mode 100644 include/dt-bindings/mfd/qcom,rpm.h
> > 
> > Please use the same filename formatting as the existing files.
> 
> Are you going to resubmit this (and answer my question on the
> regulator patch)?
> 

I'll resubmit all three parts after changing the filename of the dt header.

The regulator driver have a strong dependency on the mfd driver, but Kconfig
will make sure it's not selectable until the two parts merges again, so it
should be fine. But I guess it's up to Mark?

Regards,
Bjorn

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH v6 1/3] mfd: devicetree: bindings: Add Qualcomm RPM DT binding

2014-09-17 Thread Lee Jones
On Tue, 26 Aug 2014, Lee Jones wrote:

> On Mon, 25 Aug 2014, Bjorn Andersson wrote:
> 
> > Add binding for the Qualcomm Resource Power Manager (RPM) found in 8660,
> > 8960 and 8064 based devices. The binding currently describes the rpm
> > itself and the regulator subnodes.
> > 
> > Signed-off-by: Bjorn Andersson 
> > ---
> >  Documentation/devicetree/bindings/mfd/qcom,rpm.txt |  246 
> > 
> >  include/dt-bindings/mfd/qcom,rpm.h |  154 
> >  2 files changed, 400 insertions(+)
> >  create mode 100644 Documentation/devicetree/bindings/mfd/qcom,rpm.txt
> >  create mode 100644 include/dt-bindings/mfd/qcom,rpm.h
> 
> Please use the same filename formatting as the existing files.

Are you going to resubmit this (and answer my question on the
regulator patch)?

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH v6 1/3] mfd: devicetree: bindings: Add Qualcomm RPM DT binding

2014-08-26 Thread Lee Jones
On Mon, 25 Aug 2014, Bjorn Andersson wrote:

> Add binding for the Qualcomm Resource Power Manager (RPM) found in 8660,
> 8960 and 8064 based devices. The binding currently describes the rpm
> itself and the regulator subnodes.
> 
> Signed-off-by: Bjorn Andersson 
> ---
>  Documentation/devicetree/bindings/mfd/qcom,rpm.txt |  246 
> 
>  include/dt-bindings/mfd/qcom,rpm.h |  154 
>  2 files changed, 400 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/mfd/qcom,rpm.txt
>  create mode 100644 include/dt-bindings/mfd/qcom,rpm.h

Please use the same filename formatting as the existing files.

[...]

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[PATCH v6 1/3] mfd: devicetree: bindings: Add Qualcomm RPM DT binding

2014-08-25 Thread Bjorn Andersson
Add binding for the Qualcomm Resource Power Manager (RPM) found in 8660,
8960 and 8064 based devices. The binding currently describes the rpm
itself and the regulator subnodes.

Signed-off-by: Bjorn Andersson 
---
 Documentation/devicetree/bindings/mfd/qcom,rpm.txt |  246 
 include/dt-bindings/mfd/qcom,rpm.h |  154 
 2 files changed, 400 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/mfd/qcom,rpm.txt
 create mode 100644 include/dt-bindings/mfd/qcom,rpm.h

diff --git a/Documentation/devicetree/bindings/mfd/qcom,rpm.txt 
b/Documentation/devicetree/bindings/mfd/qcom,rpm.txt
new file mode 100644
index 000..e2cc987
--- /dev/null
+++ b/Documentation/devicetree/bindings/mfd/qcom,rpm.txt
@@ -0,0 +1,246 @@
+Qualcomm Resource Power Manager (RPM)
+
+This driver is used to interface with the Resource Power Manager (RPM) found in
+various Qualcomm platforms. The RPM allows each component in the system to vote
+for state of the system resources, such as clocks, regulators and bus
+frequencies.
+
+- compatible:
+   Usage: required
+   Value type: 
+   Definition: must be one of:
+   "qcom,rpm-apq8064"
+   "qcom,rpm-msm8660"
+   "qcom,rpm-msm8960"
+
+- reg:
+   Usage: required
+   Value type: 
+   Definition: base address and size of the RPM's message ram
+
+- interrupts:
+   Usage: required
+   Value type: 
+   Definition: three entries specifying the RPM's:
+   1. acknowledgement interrupt
+   2. error interrupt
+   3. wakeup interrupt
+
+- interrupt-names:
+   Usage: required
+   Value type: 
+   Definition: must be the three strings "ack", "err" and "wakeup", in 
order
+
+- #address-cells:
+   Usage: required
+   Value type: 
+   Definition: must be 1
+
+- #size-cells:
+   Usage: required
+   Value type: 
+   Definition: must be 0
+
+- qcom,ipc:
+   Usage: required
+   Value type: 
+
+   Definition: three entries specifying the outgoing ipc bit used for
+   signaling the RPM:
+   - phandle to a syscon node representing the apcs registers
+   - u32 representing offset to the register within the syscon
+   - u32 representing the ipc bit within the register
+
+
+= SUBDEVICES
+
+The RPM exposes resources to its subnodes. The below bindings specify the set
+of valid subnodes that can operate on these resources.
+
+== Switch-mode Power Supply regulator
+
+- compatible:
+   Usage: required
+   Value type: 
+   Definition: must be one of:
+   "qcom,rpm-pm8058-smps"
+   "qcom,rpm-pm8901-ftsmps"
+   "qcom,rpm-pm8921-smps"
+   "qcom,rpm-pm8921-ftsmps"
+
+- reg:
+   Usage: required
+   Value type: 
+   Definition: resource as defined in 
+   must be one of:
+   QCOM_RPM_PM8058_SMPS0 - QCOM_RPM_PM8058_SMPS4,
+   QCOM_RPM_PM8821_SMPS1 - QCOM_RPM_PM8821_SMPS2,
+   QCOM_RPM_PM8901_SMPS0 - QCOM_RPM_PM8901_SMPS4,
+   QCOM_RPM_PM8921_SMPS1 - QCOM_RPM_PM8921_SMPS8
+
+- bias-pull-down:
+   Usage: optional
+   Value type: 
+   Definition: enable pull down of the regulator when inactive
+
+- qcom,switch-mode-frequency:
+   Usage: required
+   Value type: 
+   Definition: Frequency (Hz) of the switch-mode power supply;
+   must be one of:
+   1920, 960, 640, 480, 384, 320,
+   274, 240, 213, 192, 175, 160,
+   148, 137, 128, 120
+
+- qcom,force-mode:
+   Usage: optional (default if no other qcom,force-mode is specified)
+   Value type: 
+   Defintion: indicates that the regulator should be forced to a
+  particular mode, valid values are:
+  QCOM_RPM_FORCE_MODE_NONE - do not force any mode
+  QCOM_RPM_FORCE_MODE_LPM - force into low power mode
+  QCOM_RPM_FORCE_MODE_HPM - force into high power mode
+  QCOM_RPM_FORCE_MODE_AUTO - allow regulator to automatically
+ select its own mode based on
+ realtime current draw, only for:
+ qcom,rpm-pm8921-smps,
+ qcom,rpm-pm8921-ftsmps
+
+- qcom,power-mode-hysteretic:
+   Usage: optional
+   Value type: 
+   Definition: select that the power supply should operate in hysteretic
+   mode, instead of the default pwm mode
+
+Standard regulator bindings are used inside switch mode power supply subnodes.
+Check Documentation/devicetree/bindings/regulator/regulator.txt for more
+details.
+
+== Lo