* [PATCH 0/3] Add 'qcom,pon-reboot-not-used' support in PON driver @ 2021-07-05 2:50 Shawn Guo 2021-07-05 2:50 ` [PATCH 1/3] dt-bindings: power: reset: Convert qcom,pon to DT schema Shawn Guo ` (2 more replies) 0 siblings, 3 replies; 9+ messages in thread From: Shawn Guo @ 2021-07-05 2:50 UTC (permalink / raw) To: Sebastian Reichel Cc: Stephan Gerhold, Rob Herring, Vinod Koul, Bjorn Andersson, devicetree, linux-arm-msm, linux-pm, Shawn Guo It's not always the case that reboot mode value gets stored in PON register. For example, Sony Xperia M4 Aqua phone (MSM8939) uses a different set of mode values and stores them in IMEM. The series adds an optional DT property 'qcom,pon-reboot-not-used', and have PON driver skip reboot-mode setup if the property is present. Shawn Guo (3): dt-bindings: power: reset: Convert qcom,pon to DT schema dt-bindings: qcom,pon: Add 'qcom,pon-reboot-not-used' property power: reset: qcom-pon: Add support for 'qcom,pon-reboot-not-used' .../bindings/power/reset/qcom,pon.txt | 49 ----------------- .../bindings/power/reset/qcom,pon.yaml | 53 +++++++++++++++++++ drivers/power/reset/qcom-pon.c | 9 ++++ 3 files changed, 62 insertions(+), 49 deletions(-) delete mode 100644 Documentation/devicetree/bindings/power/reset/qcom,pon.txt create mode 100644 Documentation/devicetree/bindings/power/reset/qcom,pon.yaml -- 2.17.1 ^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH 1/3] dt-bindings: power: reset: Convert qcom,pon to DT schema 2021-07-05 2:50 [PATCH 0/3] Add 'qcom,pon-reboot-not-used' support in PON driver Shawn Guo @ 2021-07-05 2:50 ` Shawn Guo 2021-07-12 15:45 ` Rob Herring 2021-07-05 2:50 ` [PATCH 2/3] dt-bindings: qcom,pon: Add 'qcom,pon-reboot-not-used' property Shawn Guo 2021-07-05 2:50 ` [PATCH 3/3] power: reset: qcom-pon: Add support for 'qcom,pon-reboot-not-used' Shawn Guo 2 siblings, 1 reply; 9+ messages in thread From: Shawn Guo @ 2021-07-05 2:50 UTC (permalink / raw) To: Sebastian Reichel Cc: Stephan Gerhold, Rob Herring, Vinod Koul, Bjorn Andersson, devicetree, linux-arm-msm, linux-pm, Shawn Guo It converts qcom,pon DT bindings from legacy bare text to YAML format. Signed-off-by: Shawn Guo <shawn.guo@linaro.org> --- .../bindings/power/reset/qcom,pon.txt | 49 ------------------- .../bindings/power/reset/qcom,pon.yaml | 49 +++++++++++++++++++ 2 files changed, 49 insertions(+), 49 deletions(-) delete mode 100644 Documentation/devicetree/bindings/power/reset/qcom,pon.txt create mode 100644 Documentation/devicetree/bindings/power/reset/qcom,pon.yaml diff --git a/Documentation/devicetree/bindings/power/reset/qcom,pon.txt b/Documentation/devicetree/bindings/power/reset/qcom,pon.txt deleted file mode 100644 index 0c0dc3a1e693..000000000000 --- a/Documentation/devicetree/bindings/power/reset/qcom,pon.txt +++ /dev/null @@ -1,49 +0,0 @@ -Qualcomm PON Device - -The Power On device for Qualcomm PM8xxx is MFD supporting pwrkey -and resin along with the Android reboot-mode. - -This DT node has pwrkey and resin as sub nodes. - -Required Properties: --compatible: Must be one of: - "qcom,pm8916-pon" - "qcom,pms405-pon" - "qcom,pm8998-pon" - --reg: Specifies the physical address of the pon register - -Optional subnode: --pwrkey: Specifies the subnode pwrkey and should follow the - qcom,pm8941-pwrkey.txt description. --resin: Specifies the subnode resin and should follow the - qcom,pm8xxx-pwrkey.txt description. - -The rest of the properties should follow the generic reboot-mode description -found in reboot-mode.txt - -Example: - - pon@800 { - compatible = "qcom,pm8916-pon"; - - reg = <0x800>; - mode-bootloader = <0x2>; - mode-recovery = <0x1>; - - pwrkey { - compatible = "qcom,pm8941-pwrkey"; - interrupts = <0x0 0x8 0 IRQ_TYPE_EDGE_BOTH>; - debounce = <15625>; - bias-pull-up; - linux,code = <KEY_POWER>; - }; - - resin { - compatible = "qcom,pm8941-resin"; - interrupts = <0x0 0x8 1 IRQ_TYPE_EDGE_BOTH>; - debounce = <15625>; - bias-pull-up; - linux,code = <KEY_VOLUMEDOWN>; - }; - }; diff --git a/Documentation/devicetree/bindings/power/reset/qcom,pon.yaml b/Documentation/devicetree/bindings/power/reset/qcom,pon.yaml new file mode 100644 index 000000000000..7764c804af1d --- /dev/null +++ b/Documentation/devicetree/bindings/power/reset/qcom,pon.yaml @@ -0,0 +1,49 @@ +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/power/reset/qcom,pon.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: Qualcomm PON Device + +maintainers: + - Vinod Koul <vkoul@kernel.org> + +description: | + The Power On device for Qualcomm PM8xxx is MFD supporting pwrkey + and resin along with the Android reboot-mode. + +properties: + compatible: + enum: + - qcom,pm8916-pon + - qcom,pms405-pon + - qcom,pm8998-pon + + reg: + maxItems: 1 + +patternProperties: + "^mode-.+": + $ref: /schemas/types.yaml#/definitions/uint32 + description: Vendor-specific mode value written to the mode register + +required: + - compatible + - reg + +additionalProperties: false + +examples: + - | + pmic { + #address-cells = <1>; + #size-cells = <0>; + + pon@800 { + compatible = "qcom,pm8916-pon"; + reg = <0x800>; + mode-bootloader = <0x2>; + mode-recovery = <0x1>; + }; + }; -- 2.17.1 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH 1/3] dt-bindings: power: reset: Convert qcom,pon to DT schema 2021-07-05 2:50 ` [PATCH 1/3] dt-bindings: power: reset: Convert qcom,pon to DT schema Shawn Guo @ 2021-07-12 15:45 ` Rob Herring 2021-07-14 11:47 ` Shawn Guo 0 siblings, 1 reply; 9+ messages in thread From: Rob Herring @ 2021-07-12 15:45 UTC (permalink / raw) To: Shawn Guo Cc: Sebastian Reichel, Stephan Gerhold, Vinod Koul, Bjorn Andersson, devicetree, linux-arm-msm, linux-pm On Mon, Jul 05, 2021 at 10:50:30AM +0800, Shawn Guo wrote: > It converts qcom,pon DT bindings from legacy bare text to YAML format. > > Signed-off-by: Shawn Guo <shawn.guo@linaro.org> > --- > .../bindings/power/reset/qcom,pon.txt | 49 ------------------- > .../bindings/power/reset/qcom,pon.yaml | 49 +++++++++++++++++++ > 2 files changed, 49 insertions(+), 49 deletions(-) > delete mode 100644 Documentation/devicetree/bindings/power/reset/qcom,pon.txt > create mode 100644 Documentation/devicetree/bindings/power/reset/qcom,pon.yaml > > diff --git a/Documentation/devicetree/bindings/power/reset/qcom,pon.txt b/Documentation/devicetree/bindings/power/reset/qcom,pon.txt > deleted file mode 100644 > index 0c0dc3a1e693..000000000000 > --- a/Documentation/devicetree/bindings/power/reset/qcom,pon.txt > +++ /dev/null > @@ -1,49 +0,0 @@ > -Qualcomm PON Device > - > -The Power On device for Qualcomm PM8xxx is MFD supporting pwrkey > -and resin along with the Android reboot-mode. > - > -This DT node has pwrkey and resin as sub nodes. > - > -Required Properties: > --compatible: Must be one of: > - "qcom,pm8916-pon" > - "qcom,pms405-pon" > - "qcom,pm8998-pon" > - > --reg: Specifies the physical address of the pon register > - > -Optional subnode: > --pwrkey: Specifies the subnode pwrkey and should follow the > - qcom,pm8941-pwrkey.txt description. > --resin: Specifies the subnode resin and should follow the > - qcom,pm8xxx-pwrkey.txt description. > - > -The rest of the properties should follow the generic reboot-mode description > -found in reboot-mode.txt > - > -Example: > - > - pon@800 { > - compatible = "qcom,pm8916-pon"; > - > - reg = <0x800>; > - mode-bootloader = <0x2>; > - mode-recovery = <0x1>; > - > - pwrkey { > - compatible = "qcom,pm8941-pwrkey"; > - interrupts = <0x0 0x8 0 IRQ_TYPE_EDGE_BOTH>; > - debounce = <15625>; > - bias-pull-up; > - linux,code = <KEY_POWER>; > - }; > - > - resin { > - compatible = "qcom,pm8941-resin"; > - interrupts = <0x0 0x8 1 IRQ_TYPE_EDGE_BOTH>; > - debounce = <15625>; > - bias-pull-up; > - linux,code = <KEY_VOLUMEDOWN>; > - }; > - }; > diff --git a/Documentation/devicetree/bindings/power/reset/qcom,pon.yaml b/Documentation/devicetree/bindings/power/reset/qcom,pon.yaml > new file mode 100644 > index 000000000000..7764c804af1d > --- /dev/null > +++ b/Documentation/devicetree/bindings/power/reset/qcom,pon.yaml > @@ -0,0 +1,49 @@ > +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/power/reset/qcom,pon.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Qualcomm PON Device > + > +maintainers: > + - Vinod Koul <vkoul@kernel.org> > + > +description: | > + The Power On device for Qualcomm PM8xxx is MFD supporting pwrkey > + and resin along with the Android reboot-mode. allOf: - $ref: /schemas/power/reset/reboot-mode.yaml# And then... > + > +properties: > + compatible: > + enum: > + - qcom,pm8916-pon > + - qcom,pms405-pon > + - qcom,pm8998-pon > + > + reg: > + maxItems: 1 > + > +patternProperties: > + "^mode-.+": > + $ref: /schemas/types.yaml#/definitions/uint32 > + description: Vendor-specific mode value written to the mode register Drop this. > + > +required: > + - compatible > + - reg > + > +additionalProperties: false unevaluatedProperties: false > + > +examples: > + - | > + pmic { > + #address-cells = <1>; > + #size-cells = <0>; > + > + pon@800 { > + compatible = "qcom,pm8916-pon"; > + reg = <0x800>; > + mode-bootloader = <0x2>; > + mode-recovery = <0x1>; > + }; > + }; > -- > 2.17.1 > > ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 1/3] dt-bindings: power: reset: Convert qcom,pon to DT schema 2021-07-12 15:45 ` Rob Herring @ 2021-07-14 11:47 ` Shawn Guo 0 siblings, 0 replies; 9+ messages in thread From: Shawn Guo @ 2021-07-14 11:47 UTC (permalink / raw) To: Rob Herring Cc: Sebastian Reichel, Stephan Gerhold, Vinod Koul, Bjorn Andersson, devicetree, linux-arm-msm, linux-pm On Mon, Jul 12, 2021 at 09:45:21AM -0600, Rob Herring wrote: > On Mon, Jul 05, 2021 at 10:50:30AM +0800, Shawn Guo wrote: > > It converts qcom,pon DT bindings from legacy bare text to YAML format. > > > > Signed-off-by: Shawn Guo <shawn.guo@linaro.org> > > --- > > .../bindings/power/reset/qcom,pon.txt | 49 ------------------- > > .../bindings/power/reset/qcom,pon.yaml | 49 +++++++++++++++++++ > > 2 files changed, 49 insertions(+), 49 deletions(-) > > delete mode 100644 Documentation/devicetree/bindings/power/reset/qcom,pon.txt > > create mode 100644 Documentation/devicetree/bindings/power/reset/qcom,pon.yaml > > > > diff --git a/Documentation/devicetree/bindings/power/reset/qcom,pon.txt b/Documentation/devicetree/bindings/power/reset/qcom,pon.txt > > deleted file mode 100644 > > index 0c0dc3a1e693..000000000000 > > --- a/Documentation/devicetree/bindings/power/reset/qcom,pon.txt > > +++ /dev/null > > @@ -1,49 +0,0 @@ > > -Qualcomm PON Device > > - > > -The Power On device for Qualcomm PM8xxx is MFD supporting pwrkey > > -and resin along with the Android reboot-mode. > > - > > -This DT node has pwrkey and resin as sub nodes. > > - > > -Required Properties: > > --compatible: Must be one of: > > - "qcom,pm8916-pon" > > - "qcom,pms405-pon" > > - "qcom,pm8998-pon" > > - > > --reg: Specifies the physical address of the pon register > > - > > -Optional subnode: > > --pwrkey: Specifies the subnode pwrkey and should follow the > > - qcom,pm8941-pwrkey.txt description. > > --resin: Specifies the subnode resin and should follow the > > - qcom,pm8xxx-pwrkey.txt description. > > - > > -The rest of the properties should follow the generic reboot-mode description > > -found in reboot-mode.txt > > - > > -Example: > > - > > - pon@800 { > > - compatible = "qcom,pm8916-pon"; > > - > > - reg = <0x800>; > > - mode-bootloader = <0x2>; > > - mode-recovery = <0x1>; > > - > > - pwrkey { > > - compatible = "qcom,pm8941-pwrkey"; > > - interrupts = <0x0 0x8 0 IRQ_TYPE_EDGE_BOTH>; > > - debounce = <15625>; > > - bias-pull-up; > > - linux,code = <KEY_POWER>; > > - }; > > - > > - resin { > > - compatible = "qcom,pm8941-resin"; > > - interrupts = <0x0 0x8 1 IRQ_TYPE_EDGE_BOTH>; > > - debounce = <15625>; > > - bias-pull-up; > > - linux,code = <KEY_VOLUMEDOWN>; > > - }; > > - }; > > diff --git a/Documentation/devicetree/bindings/power/reset/qcom,pon.yaml b/Documentation/devicetree/bindings/power/reset/qcom,pon.yaml > > new file mode 100644 > > index 000000000000..7764c804af1d > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/power/reset/qcom,pon.yaml > > @@ -0,0 +1,49 @@ > > +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause > > +%YAML 1.2 > > +--- > > +$id: http://devicetree.org/schemas/power/reset/qcom,pon.yaml# > > +$schema: http://devicetree.org/meta-schemas/core.yaml# > > + > > +title: Qualcomm PON Device > > + > > +maintainers: > > + - Vinod Koul <vkoul@kernel.org> > > + > > +description: | > > + The Power On device for Qualcomm PM8xxx is MFD supporting pwrkey > > + and resin along with the Android reboot-mode. > > allOf: > - $ref: /schemas/power/reset/reboot-mode.yaml# > > And then... > > > + > > +properties: > > + compatible: > > + enum: > > + - qcom,pm8916-pon > > + - qcom,pms405-pon > > + - qcom,pm8998-pon > > + > > + reg: > > + maxItems: 1 > > + > > +patternProperties: > > + "^mode-.+": > > + $ref: /schemas/types.yaml#/definitions/uint32 > > + description: Vendor-specific mode value written to the mode register > > Drop this. Drop the description line only or the whole "^mode-.+" block? I got the following from dt_binding_check if dropping the whole block. pon@800: 'compatible', 'reg' do not match any of the regexes: '^mode-.*$', 'pinctrl-[0-9]+' > > > + > > +required: > > + - compatible > > + - reg > > + > > +additionalProperties: false > > unevaluatedProperties: false Replace additionalProperties with unevaluatedProperties, right? Shawn ^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH 2/3] dt-bindings: qcom,pon: Add 'qcom,pon-reboot-not-used' property 2021-07-05 2:50 [PATCH 0/3] Add 'qcom,pon-reboot-not-used' support in PON driver Shawn Guo 2021-07-05 2:50 ` [PATCH 1/3] dt-bindings: power: reset: Convert qcom,pon to DT schema Shawn Guo @ 2021-07-05 2:50 ` Shawn Guo 2021-07-12 17:08 ` Bjorn Andersson 2021-07-05 2:50 ` [PATCH 3/3] power: reset: qcom-pon: Add support for 'qcom,pon-reboot-not-used' Shawn Guo 2 siblings, 1 reply; 9+ messages in thread From: Shawn Guo @ 2021-07-05 2:50 UTC (permalink / raw) To: Sebastian Reichel Cc: Stephan Gerhold, Rob Herring, Vinod Koul, Bjorn Andersson, devicetree, linux-arm-msm, linux-pm, Shawn Guo Add an optional 'qcom,pon-reboot-not-used' property for devices, which do not use PON register to pass reboot mode but other mechanism, e.g. particular IMEM address. Signed-off-by: Shawn Guo <shawn.guo@linaro.org> --- Documentation/devicetree/bindings/power/reset/qcom,pon.yaml | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/Documentation/devicetree/bindings/power/reset/qcom,pon.yaml b/Documentation/devicetree/bindings/power/reset/qcom,pon.yaml index 7764c804af1d..584eff179904 100644 --- a/Documentation/devicetree/bindings/power/reset/qcom,pon.yaml +++ b/Documentation/devicetree/bindings/power/reset/qcom,pon.yaml @@ -23,6 +23,10 @@ properties: reg: maxItems: 1 + qcom,pon-reboot-not-used: + description: Support of reboot mode passing through PON register is not used + type: boolean + patternProperties: "^mode-.+": $ref: /schemas/types.yaml#/definitions/uint32 -- 2.17.1 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH 2/3] dt-bindings: qcom,pon: Add 'qcom,pon-reboot-not-used' property 2021-07-05 2:50 ` [PATCH 2/3] dt-bindings: qcom,pon: Add 'qcom,pon-reboot-not-used' property Shawn Guo @ 2021-07-12 17:08 ` Bjorn Andersson 0 siblings, 0 replies; 9+ messages in thread From: Bjorn Andersson @ 2021-07-12 17:08 UTC (permalink / raw) To: Shawn Guo Cc: Sebastian Reichel, Stephan Gerhold, Rob Herring, Vinod Koul, devicetree, linux-arm-msm, linux-pm On Sun 04 Jul 21:50 CDT 2021, Shawn Guo wrote: > Add an optional 'qcom,pon-reboot-not-used' property for devices, which > do not use PON register to pass reboot mode but other mechanism, e.g. > particular IMEM address. > > Signed-off-by: Shawn Guo <shawn.guo@linaro.org> > --- > Documentation/devicetree/bindings/power/reset/qcom,pon.yaml | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/Documentation/devicetree/bindings/power/reset/qcom,pon.yaml b/Documentation/devicetree/bindings/power/reset/qcom,pon.yaml > index 7764c804af1d..584eff179904 100644 > --- a/Documentation/devicetree/bindings/power/reset/qcom,pon.yaml > +++ b/Documentation/devicetree/bindings/power/reset/qcom,pon.yaml > @@ -23,6 +23,10 @@ properties: > reg: > maxItems: 1 > > + qcom,pon-reboot-not-used: > + description: Support of reboot mode passing through PON register is not used > + type: boolean > + How is this different from just not specifying any mode-* properties? Regards, Bjorn > patternProperties: > "^mode-.+": > $ref: /schemas/types.yaml#/definitions/uint32 > -- > 2.17.1 > ^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH 3/3] power: reset: qcom-pon: Add support for 'qcom,pon-reboot-not-used' 2021-07-05 2:50 [PATCH 0/3] Add 'qcom,pon-reboot-not-used' support in PON driver Shawn Guo 2021-07-05 2:50 ` [PATCH 1/3] dt-bindings: power: reset: Convert qcom,pon to DT schema Shawn Guo 2021-07-05 2:50 ` [PATCH 2/3] dt-bindings: qcom,pon: Add 'qcom,pon-reboot-not-used' property Shawn Guo @ 2021-07-05 2:50 ` Shawn Guo 2021-07-12 17:12 ` Bjorn Andersson 2 siblings, 1 reply; 9+ messages in thread From: Shawn Guo @ 2021-07-05 2:50 UTC (permalink / raw) To: Sebastian Reichel Cc: Stephan Gerhold, Rob Herring, Vinod Koul, Bjorn Andersson, devicetree, linux-arm-msm, linux-pm, Shawn Guo Check property 'qcom,pon-reboot-not-used' and skip reboot-mode setup if the property is present. This is useful for devices that use IMEM instead of PON register to pass reboot mode, but still want to populate pwrkey and resin devices. Signed-off-by: Shawn Guo <shawn.guo@linaro.org> --- drivers/power/reset/qcom-pon.c | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/drivers/power/reset/qcom-pon.c b/drivers/power/reset/qcom-pon.c index 4a688741a88a..85fae9e85878 100644 --- a/drivers/power/reset/qcom-pon.c +++ b/drivers/power/reset/qcom-pon.c @@ -47,6 +47,14 @@ static int pm8916_pon_probe(struct platform_device *pdev) struct pm8916_pon *pon; int error; + if (device_property_present(&pdev->dev, "qcom,pon-reboot-not-used")) { + /* + * Skip reboot-mode setup and registration if PON is not used + * for passing reboot mode at all. + */ + goto done; + } + pon = devm_kzalloc(&pdev->dev, sizeof(*pon), GFP_KERNEL); if (!pon) return -ENOMEM; @@ -75,6 +83,7 @@ static int pm8916_pon_probe(struct platform_device *pdev) platform_set_drvdata(pdev, pon); +done: return devm_of_platform_populate(&pdev->dev); } -- 2.17.1 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH 3/3] power: reset: qcom-pon: Add support for 'qcom,pon-reboot-not-used' 2021-07-05 2:50 ` [PATCH 3/3] power: reset: qcom-pon: Add support for 'qcom,pon-reboot-not-used' Shawn Guo @ 2021-07-12 17:12 ` Bjorn Andersson 2021-07-14 11:49 ` Shawn Guo 0 siblings, 1 reply; 9+ messages in thread From: Bjorn Andersson @ 2021-07-12 17:12 UTC (permalink / raw) To: Shawn Guo Cc: Sebastian Reichel, Stephan Gerhold, Rob Herring, Vinod Koul, devicetree, linux-arm-msm, linux-pm On Sun 04 Jul 21:50 CDT 2021, Shawn Guo wrote: > Check property 'qcom,pon-reboot-not-used' and skip reboot-mode setup > if the property is present. This is useful for devices that use IMEM > instead of PON register to pass reboot mode, but still want to populate > pwrkey and resin devices. > If IMEM is used instead, I think it should be sufficient for the DT author to omit mode-* properties to achieve this. If the PON hardware doesn't support mode-*, then that should be reflected in the DT binding and compatible. Regards, Bjorn > Signed-off-by: Shawn Guo <shawn.guo@linaro.org> > --- > drivers/power/reset/qcom-pon.c | 9 +++++++++ > 1 file changed, 9 insertions(+) > > diff --git a/drivers/power/reset/qcom-pon.c b/drivers/power/reset/qcom-pon.c > index 4a688741a88a..85fae9e85878 100644 > --- a/drivers/power/reset/qcom-pon.c > +++ b/drivers/power/reset/qcom-pon.c > @@ -47,6 +47,14 @@ static int pm8916_pon_probe(struct platform_device *pdev) > struct pm8916_pon *pon; > int error; > > + if (device_property_present(&pdev->dev, "qcom,pon-reboot-not-used")) { > + /* > + * Skip reboot-mode setup and registration if PON is not used > + * for passing reboot mode at all. > + */ > + goto done; > + } > + > pon = devm_kzalloc(&pdev->dev, sizeof(*pon), GFP_KERNEL); > if (!pon) > return -ENOMEM; > @@ -75,6 +83,7 @@ static int pm8916_pon_probe(struct platform_device *pdev) > > platform_set_drvdata(pdev, pon); > > +done: > return devm_of_platform_populate(&pdev->dev); > } > > -- > 2.17.1 > ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 3/3] power: reset: qcom-pon: Add support for 'qcom,pon-reboot-not-used' 2021-07-12 17:12 ` Bjorn Andersson @ 2021-07-14 11:49 ` Shawn Guo 0 siblings, 0 replies; 9+ messages in thread From: Shawn Guo @ 2021-07-14 11:49 UTC (permalink / raw) To: Bjorn Andersson Cc: Sebastian Reichel, Stephan Gerhold, Rob Herring, Vinod Koul, devicetree, linux-arm-msm, linux-pm On Mon, Jul 12, 2021 at 12:12:26PM -0500, Bjorn Andersson wrote: > On Sun 04 Jul 21:50 CDT 2021, Shawn Guo wrote: > > > Check property 'qcom,pon-reboot-not-used' and skip reboot-mode setup > > if the property is present. This is useful for devices that use IMEM > > instead of PON register to pass reboot mode, but still want to populate > > pwrkey and resin devices. > > > > If IMEM is used instead, I think it should be sufficient for the DT > author to omit mode-* properties to achieve this. Indeed! This is much better solution. Thanks for the suggestion! So please disregard patch #2 and #3, and I will resend #1 as a separate patch. Shawn > If the PON hardware > doesn't support mode-*, then that should be reflected in the DT binding > and compatible. ^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2021-07-14 11:50 UTC | newest] Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2021-07-05 2:50 [PATCH 0/3] Add 'qcom,pon-reboot-not-used' support in PON driver Shawn Guo 2021-07-05 2:50 ` [PATCH 1/3] dt-bindings: power: reset: Convert qcom,pon to DT schema Shawn Guo 2021-07-12 15:45 ` Rob Herring 2021-07-14 11:47 ` Shawn Guo 2021-07-05 2:50 ` [PATCH 2/3] dt-bindings: qcom,pon: Add 'qcom,pon-reboot-not-used' property Shawn Guo 2021-07-12 17:08 ` Bjorn Andersson 2021-07-05 2:50 ` [PATCH 3/3] power: reset: qcom-pon: Add support for 'qcom,pon-reboot-not-used' Shawn Guo 2021-07-12 17:12 ` Bjorn Andersson 2021-07-14 11:49 ` Shawn Guo
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).