Re: [PATCH v6 1/3] mfd: devicetree: bindings: Add Qualcomm RPM DT binding
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
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
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
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
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
