Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1458282 > unrolled thread

[PATCH 2/2] device-tree: nexus7: Add IMEM syscon and reboot reason support

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  [PATCH 2/2] device-tree: nexus7: Add IMEM syscon and reboot reason support John Stultz <john.stultz@linaro.org> - 2016-08-09 00:40 +0200
    Re: [PATCH 2/2] device-tree: nexus7: Add IMEM syscon and reboot  reason support Bjorn Andersson <bjorn.andersson@linaro.org> - 2016-08-10 06:30 +0200
      Re: [PATCH 2/2] device-tree: nexus7: Add IMEM syscon and reboot  reason support John Stultz <john.stultz@linaro.org> - 2016-08-10 20:30 +0200
        Re: [PATCH 2/2] device-tree: nexus7: Add IMEM syscon and reboot  reason support Bjorn Andersson <bjorn.andersson@linaro.org> - 2016-08-10 22:00 +0200

#1458282 — [PATCH 2/2] device-tree: nexus7: Add IMEM syscon and reboot reason support

This patch add the IMEM syscon memory region to the DT,
as well as addds support for the magic reboot reason
values that are written to the address for each mode.

Cc: Rob Herring <robh+dt@kernel.org>
Cc: Andy Gross <agross@codeaurora.org>
Cc: Bjorn Andersson <bjorn.andersson@linaro.org>
Cc: Stephen Boyd <stephen.boyd@linaro.org>
Cc: linux-arm-msm@vger.kernel.org
Cc: devicetree@vger.kernel.org
Signed-off-by: John Stultz <john.stultz@linaro.org>
---
 arch/arm/boot/dts/qcom-apq8064-asus-nexus7-flo.dts | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

diff --git a/arch/arm/boot/dts/qcom-apq8064-asus-nexus7-flo.dts b/arch/arm/boot/dts/qcom-apq8064-asus-nexus7-flo.dts
index 7b05f07..ff856c3 100644
--- a/arch/arm/boot/dts/qcom-apq8064-asus-nexus7-flo.dts
+++ b/arch/arm/boot/dts/qcom-apq8064-asus-nexus7-flo.dts
@@ -272,5 +272,19 @@
 				vqmmc-supply = <&pm8921_s4>;
 			};
 		};
+
+		imem@2a03f000 {
+			compatible = "syscon", "simple-mfd";
+			reg = <0x2a03f000 0x1000>;
+
+			reboot-mode {
+				compatible = "syscon-reboot-mode";
+				offset = <0x65c>;
+
+				mode-normal	= <0x77665501>;
+				mode-bootloader	= <0x77665500>;
+				mode-recovery	= <0x77665502>;
+			};
+		};
 	};
 };
-- 
1.9.1

[toc] | [next] | [standalone]

#1459285 — Re: [PATCH 2/2] device-tree: nexus7: Add IMEM syscon and reboot reason support

On Mon 08 Aug 15:34 PDT 2016, John Stultz wrote:

> This patch add the IMEM syscon memory region to the DT,
> as well as addds support for the magic reboot reason
> values that are written to the address for each mode.
>

This looks good, double checked the addresses and magics. But I think
you should move the entire thing to qcom-apq8064.dtsi, as this is common
to the base platform.

And I would prefer if you updated the subject prefix...

With the move and subject update:
Reviewed-by: Bjorn Andersson <bjorn.andersson@linaro.org>

Regards,
Bjorn

> Cc: Rob Herring <robh+dt@kernel.org>
> Cc: Andy Gross <agross@codeaurora.org>
> Cc: Bjorn Andersson <bjorn.andersson@linaro.org>
> Cc: Stephen Boyd <stephen.boyd@linaro.org>
> Cc: linux-arm-msm@vger.kernel.org
> Cc: devicetree@vger.kernel.org
> Signed-off-by: John Stultz <john.stultz@linaro.org>
> ---
>  arch/arm/boot/dts/qcom-apq8064-asus-nexus7-flo.dts | 14 ++++++++++++++
>  1 file changed, 14 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/qcom-apq8064-asus-nexus7-flo.dts b/arch/arm/boot/dts/qcom-apq8064-asus-nexus7-flo.dts
> index 7b05f07..ff856c3 100644
> --- a/arch/arm/boot/dts/qcom-apq8064-asus-nexus7-flo.dts
> +++ b/arch/arm/boot/dts/qcom-apq8064-asus-nexus7-flo.dts
> @@ -272,5 +272,19 @@
>  				vqmmc-supply = <&pm8921_s4>;
>  			};
>  		};
> +
> +		imem@2a03f000 {
> +			compatible = "syscon", "simple-mfd";
> +			reg = <0x2a03f000 0x1000>;
> +
> +			reboot-mode {
> +				compatible = "syscon-reboot-mode";
> +				offset = <0x65c>;
> +
> +				mode-normal	= <0x77665501>;
> +				mode-bootloader	= <0x77665500>;
> +				mode-recovery	= <0x77665502>;
> +			};
> +		};
>  	};
>  };
> -- 
> 1.9.1
> 

[toc] | [prev] | [next] | [standalone]

#1459405 — Re: [PATCH 2/2] device-tree: nexus7: Add IMEM syscon and reboot reason support

On Tue, Aug 9, 2016 at 9:28 PM, Bjorn Andersson
<bjorn.andersson@linaro.org> wrote:
> On Mon 08 Aug 15:34 PDT 2016, John Stultz wrote:
>
>> This patch add the IMEM syscon memory region to the DT,
>> as well as addds support for the magic reboot reason
>> values that are written to the address for each mode.
>>
>
> This looks good, double checked the addresses and magics. But I think
> you should move the entire thing to qcom-apq8064.dtsi, as this is common
> to the base platform.

Interesting, so do all 8064's use the same bootloader?

> And I would prefer if you updated the subject prefix...
>
> With the move and subject update:
> Reviewed-by: Bjorn Andersson <bjorn.andersson@linaro.org>

thanks!
-john

[toc] | [prev] | [next] | [standalone]

#1459731 — Re: [PATCH 2/2] device-tree: nexus7: Add IMEM syscon and reboot reason support

On Tue 09 Aug 22:04 PDT 2016, John Stultz wrote:

> On Tue, Aug 9, 2016 at 9:28 PM, Bjorn Andersson
> <bjorn.andersson@linaro.org> wrote:
> > On Mon 08 Aug 15:34 PDT 2016, John Stultz wrote:
> >
> >> This patch add the IMEM syscon memory region to the DT,
> >> as well as addds support for the magic reboot reason
> >> values that are written to the address for each mode.
> >>
> >
> > This looks good, double checked the addresses and magics. But I think
> > you should move the entire thing to qcom-apq8064.dtsi, as this is common
> > to the base platform.
> 
> Interesting, so do all 8064's use the same bootloader?
> 

As far as I can tell, the APQ8064 comes with 4kb of IMEM, so that part
is common regardless.

There is no indication of the restart reason being "optional" in the
downstream kernel, but that's not necessarily a "yes" to your question.

So, please move it to the apq8064.dtsi, as there are no indications of
systems not having this setup. Worst case would be that I'm wrong and we
have to move it out to all current targets sometime in the future.

Regards,
Bjorn

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web