* [PATCH 1/2] ARM: dts: Fix bootup issue on smdk5250 @ 2014-11-18 12:08 Yadwinder Singh Brar 2014-11-18 12:08 ` [PATCH 2/2] ARM: dts: Add missing irq pinctrl for max77686 " Yadwinder Singh Brar 2014-11-20 4:01 ` [1/2] ARM: dts: Fix bootup issue " Pankaj Dubey 0 siblings, 2 replies; 14+ messages in thread From: Yadwinder Singh Brar @ 2014-11-18 12:08 UTC (permalink / raw) To: linux-arm-kernel With default config on smdk5250 latest tree throws below message : [ 2.226049] thermal thermal_zone0: critical temperature reached(224 C),shutting down [ 2.227840] reboot: Failed to start orderly shutdown: forcing the issue and hangs randomly because it reads wrong temperature value. I can't figure out any direct relation between LDO10 and TMU from board schematics which I have. So making LDO10 always-on to fix issue for now. Signed-off-by: Yadwinder Singh Brar <yadi.brar@samsung.com> --- arch/arm/boot/dts/exynos5250-smdk5250.dts | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/arch/arm/boot/dts/exynos5250-smdk5250.dts b/arch/arm/boot/dts/exynos5250-smdk5250.dts index bc27cc2..95b5b51 100644 --- a/arch/arm/boot/dts/exynos5250-smdk5250.dts +++ b/arch/arm/boot/dts/exynos5250-smdk5250.dts @@ -190,6 +190,7 @@ regulator-name = "P1.8V_LDO_OUT10"; regulator-min-microvolt = <1800000>; regulator-max-microvolt = <1800000>; + regulator-always-on; }; ldo11_reg: LDO11 { -- 1.7.0.4 ^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH 2/2] ARM: dts: Add missing irq pinctrl for max77686 on smdk5250 2014-11-18 12:08 [PATCH 1/2] ARM: dts: Fix bootup issue on smdk5250 Yadwinder Singh Brar @ 2014-11-18 12:08 ` Yadwinder Singh Brar 2014-11-18 14:02 ` Javier Martinez Canillas ` (2 more replies) 2014-11-20 4:01 ` [1/2] ARM: dts: Fix bootup issue " Pankaj Dubey 1 sibling, 3 replies; 14+ messages in thread From: Yadwinder Singh Brar @ 2014-11-18 12:08 UTC (permalink / raw) To: linux-arm-kernel This patch adds pinctrl configuration for using configuring gpx3-2 as an external interrupt from max77686. Though max77686 RTC is enabled and gets probed by default, it doesnt work as its unable to get interrupt. This patch makes max77686 RTC work and also configures it as wakeup source. Signed-off-by: Yadwinder Singh Brar <yadi.brar@samsung.com> --- arch/arm/boot/dts/exynos5250-smdk5250.dts | 12 ++++++++++++ 1 files changed, 12 insertions(+), 0 deletions(-) diff --git a/arch/arm/boot/dts/exynos5250-smdk5250.dts b/arch/arm/boot/dts/exynos5250-smdk5250.dts index 95b5b51..19cd918 100644 --- a/arch/arm/boot/dts/exynos5250-smdk5250.dts +++ b/arch/arm/boot/dts/exynos5250-smdk5250.dts @@ -131,6 +131,9 @@ reg = <0x09>; interrupt-parent = <&gpx3>; interrupts = <2 IRQ_TYPE_NONE>; + pinctrl-names = "default"; + pinctrl-0 = <&max77686_irq>; + wakeup-source; voltage-regulators { ldo1_reg: LDO1 { @@ -411,3 +414,12 @@ }; }; }; + +&pinctrl_0 { + max77686_irq: max77686-irq { + samsung,pins = "gpx3-2"; + samsung,pin-function = <0xf>; + samsung,pin-pud = <0>; + samsung,pin-drv = <0>; + }; +}; -- 1.7.0.4 ^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH 2/2] ARM: dts: Add missing irq pinctrl for max77686 on smdk5250 2014-11-18 12:08 ` [PATCH 2/2] ARM: dts: Add missing irq pinctrl for max77686 " Yadwinder Singh Brar @ 2014-11-18 14:02 ` Javier Martinez Canillas 2014-11-20 4:06 ` [2/2] " Pankaj Dubey 2015-04-28 11:02 ` Pankaj Dubey 2 siblings, 0 replies; 14+ messages in thread From: Javier Martinez Canillas @ 2014-11-18 14:02 UTC (permalink / raw) To: linux-arm-kernel Hello Yadwinder, On Tue, Nov 18, 2014 at 1:08 PM, Yadwinder Singh Brar <yadi.brar@samsung.com> wrote: > This patch adds pinctrl configuration for using configuring gpx3-2 as an > external interrupt from max77686. Though max77686 RTC is enabled and gets > probed by default, it doesnt work as its unable to get interrupt. > I'm not familiar with this board but the patch looks good to me assuming the IRQ is the correct one. Reviewed-by: Javier Martinez Canillas <javier.martinez@collabora.co.uk> Best regards, Javier ^ permalink raw reply [flat|nested] 14+ messages in thread
* [2/2] ARM: dts: Add missing irq pinctrl for max77686 on smdk5250 2014-11-18 12:08 ` [PATCH 2/2] ARM: dts: Add missing irq pinctrl for max77686 " Yadwinder Singh Brar 2014-11-18 14:02 ` Javier Martinez Canillas @ 2014-11-20 4:06 ` Pankaj Dubey 2015-04-28 11:02 ` Pankaj Dubey 2 siblings, 0 replies; 14+ messages in thread From: Pankaj Dubey @ 2014-11-20 4:06 UTC (permalink / raw) To: linux-arm-kernel Hi, On Tuesday 18 November 2014 05:38 PM, Yadwinder Singh Brar wrote: > This patch adds pinctrl configuration for using configuring gpx3-2 as an > external interrupt from max77686. Though max77686 RTC is enabled and gets > probed by default, it doesnt work as its unable to get interrupt. > > This patch makes max77686 RTC work and also configures it as wakeup source. > > Signed-off-by: Yadwinder Singh Brar <yadi.brar@samsung.com> > Reviewed-by: Javier Martinez Canillas <javier.martinez@collabora.co.uk> > Tested this on SMDK5250, and with this PMIC RTC is working fine. Tested-by: Pankaj Dubey <pankaj.dubey@samsung.com> Thanks, Pankaj Dubey > --- > arch/arm/boot/dts/exynos5250-smdk5250.dts | 12 ++++++++++++ > 1 files changed, 12 insertions(+), 0 deletions(-) > > diff --git a/arch/arm/boot/dts/exynos5250-smdk5250.dts b/arch/arm/boot/dts/exynos5250-smdk5250.dts > index 95b5b51..19cd918 100644 > --- a/arch/arm/boot/dts/exynos5250-smdk5250.dts > +++ b/arch/arm/boot/dts/exynos5250-smdk5250.dts > @@ -131,6 +131,9 @@ > reg = <0x09>; > interrupt-parent = <&gpx3>; > interrupts = <2 IRQ_TYPE_NONE>; > + pinctrl-names = "default"; > + pinctrl-0 = <&max77686_irq>; > + wakeup-source; > > voltage-regulators { > ldo1_reg: LDO1 { > @@ -411,3 +414,12 @@ > }; > }; > }; > + > +&pinctrl_0 { > + max77686_irq: max77686-irq { > + samsung,pins = "gpx3-2"; > + samsung,pin-function = <0xf>; > + samsung,pin-pud = <0>; > + samsung,pin-drv = <0>; > + }; > +}; > ^ permalink raw reply [flat|nested] 14+ messages in thread
* [2/2] ARM: dts: Add missing irq pinctrl for max77686 on smdk5250 2014-11-18 12:08 ` [PATCH 2/2] ARM: dts: Add missing irq pinctrl for max77686 " Yadwinder Singh Brar 2014-11-18 14:02 ` Javier Martinez Canillas 2014-11-20 4:06 ` [2/2] " Pankaj Dubey @ 2015-04-28 11:02 ` Pankaj Dubey 2015-05-02 5:49 ` Krzysztof Kozlowski 2 siblings, 1 reply; 14+ messages in thread From: Pankaj Dubey @ 2015-04-28 11:02 UTC (permalink / raw) To: linux-arm-kernel Hi Kukjin, Any updated for this patch. Without this PMIC RTC is not working on SMDK5250. Thanks, Pankaj Dubey On Tuesday 18 November 2014 05:38 PM, Yadwinder Singh Brar wrote: > This patch adds pinctrl configuration for using configuring gpx3-2 as an > external interrupt from max77686. Though max77686 RTC is enabled and gets > probed by default, it doesnt work as its unable to get interrupt. > > This patch makes max77686 RTC work and also configures it as wakeup source. > > Signed-off-by: Yadwinder Singh Brar <yadi.brar@samsung.com> > Reviewed-by: Javier Martinez Canillas <javier.martinez@collabora.co.uk> > Tested-by: Pankaj Dubey <pankaj.dubey@samsung.com> > > --- > arch/arm/boot/dts/exynos5250-smdk5250.dts | 12 ++++++++++++ > 1 files changed, 12 insertions(+), 0 deletions(-) > > diff --git a/arch/arm/boot/dts/exynos5250-smdk5250.dts b/arch/arm/boot/dts/exynos5250-smdk5250.dts > index 95b5b51..19cd918 100644 > --- a/arch/arm/boot/dts/exynos5250-smdk5250.dts > +++ b/arch/arm/boot/dts/exynos5250-smdk5250.dts > @@ -131,6 +131,9 @@ > reg = <0x09>; > interrupt-parent = <&gpx3>; > interrupts = <2 IRQ_TYPE_NONE>; > + pinctrl-names = "default"; > + pinctrl-0 = <&max77686_irq>; > + wakeup-source; > > voltage-regulators { > ldo1_reg: LDO1 { > @@ -411,3 +414,12 @@ > }; > }; > }; > + > +&pinctrl_0 { > + max77686_irq: max77686-irq { > + samsung,pins = "gpx3-2"; > + samsung,pin-function = <0xf>; > + samsung,pin-pud = <0>; > + samsung,pin-drv = <0>; > + }; > +}; > ^ permalink raw reply [flat|nested] 14+ messages in thread
* [2/2] ARM: dts: Add missing irq pinctrl for max77686 on smdk5250 2015-04-28 11:02 ` Pankaj Dubey @ 2015-05-02 5:49 ` Krzysztof Kozlowski 2015-08-17 5:50 ` Krzysztof Kozlowski 0 siblings, 1 reply; 14+ messages in thread From: Krzysztof Kozlowski @ 2015-05-02 5:49 UTC (permalink / raw) To: linux-arm-kernel W dniu 28.04.2015 o 20:02, Pankaj Dubey pisze: > Hi Kukjin, > > Any updated for this patch. Without this PMIC RTC is not working on > SMDK5250. > Could you resend the patch after rebasing it? Best regards, Krzysztof > Thanks, > Pankaj Dubey > > On Tuesday 18 November 2014 05:38 PM, Yadwinder Singh Brar wrote: >> This patch adds pinctrl configuration for using configuring gpx3-2 as an >> external interrupt from max77686. Though max77686 RTC is enabled and gets >> probed by default, it doesnt work as its unable to get interrupt. >> >> This patch makes max77686 RTC work and also configures it as wakeup >> source. >> >> Signed-off-by: Yadwinder Singh Brar <yadi.brar@samsung.com> >> Reviewed-by: Javier Martinez Canillas <javier.martinez@collabora.co.uk> >> Tested-by: Pankaj Dubey <pankaj.dubey@samsung.com> >> >> --- >> arch/arm/boot/dts/exynos5250-smdk5250.dts | 12 ++++++++++++ >> 1 files changed, 12 insertions(+), 0 deletions(-) >> >> diff --git a/arch/arm/boot/dts/exynos5250-smdk5250.dts >> b/arch/arm/boot/dts/exynos5250-smdk5250.dts >> index 95b5b51..19cd918 100644 >> --- a/arch/arm/boot/dts/exynos5250-smdk5250.dts >> +++ b/arch/arm/boot/dts/exynos5250-smdk5250.dts >> @@ -131,6 +131,9 @@ >> reg = <0x09>; >> interrupt-parent = <&gpx3>; >> interrupts = <2 IRQ_TYPE_NONE>; >> + pinctrl-names = "default"; >> + pinctrl-0 = <&max77686_irq>; >> + wakeup-source; >> >> voltage-regulators { >> ldo1_reg: LDO1 { >> @@ -411,3 +414,12 @@ >> }; >> }; >> }; >> + >> +&pinctrl_0 { >> + max77686_irq: max77686-irq { >> + samsung,pins = "gpx3-2"; >> + samsung,pin-function = <0xf>; >> + samsung,pin-pud = <0>; >> + samsung,pin-drv = <0>; >> + }; >> +}; >> > -- > To unsubscribe from this list: send the line "unsubscribe > linux-samsung-soc" in > the body of a message to majordomo at vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 14+ messages in thread
* [2/2] ARM: dts: Add missing irq pinctrl for max77686 on smdk5250 2015-05-02 5:49 ` Krzysztof Kozlowski @ 2015-08-17 5:50 ` Krzysztof Kozlowski 2015-08-18 6:13 ` Pankaj Dubey 2015-08-18 6:27 ` Pankaj Dubey 0 siblings, 2 replies; 14+ messages in thread From: Krzysztof Kozlowski @ 2015-08-17 5:50 UTC (permalink / raw) To: linux-arm-kernel 2015-05-02 14:49 GMT+09:00 Krzysztof Kozlowski <k.kozlowski.k@gmail.com>: > W dniu 28.04.2015 o 20:02, Pankaj Dubey pisze: >> Hi Kukjin, >> >> Any updated for this patch. Without this PMIC RTC is not working on >> SMDK5250. >> > > Could you resend the patch after rebasing it? > > Best regards, > Krzysztof Dear Pankaj and Yadwinder, Do you plan to do anything with this patch? Best regards, Krzysztof >> Thanks, >> Pankaj Dubey >> >> On Tuesday 18 November 2014 05:38 PM, Yadwinder Singh Brar wrote: >>> This patch adds pinctrl configuration for using configuring gpx3-2 as an >>> external interrupt from max77686. Though max77686 RTC is enabled and gets >>> probed by default, it doesnt work as its unable to get interrupt. >>> >>> This patch makes max77686 RTC work and also configures it as wakeup >>> source. >>> >>> Signed-off-by: Yadwinder Singh Brar <yadi.brar@samsung.com> >>> Reviewed-by: Javier Martinez Canillas <javier.martinez@collabora.co.uk> >>> Tested-by: Pankaj Dubey <pankaj.dubey@samsung.com> >>> >>> --- >>> arch/arm/boot/dts/exynos5250-smdk5250.dts | 12 ++++++++++++ >>> 1 files changed, 12 insertions(+), 0 deletions(-) >>> >>> diff --git a/arch/arm/boot/dts/exynos5250-smdk5250.dts >>> b/arch/arm/boot/dts/exynos5250-smdk5250.dts >>> index 95b5b51..19cd918 100644 >>> --- a/arch/arm/boot/dts/exynos5250-smdk5250.dts >>> +++ b/arch/arm/boot/dts/exynos5250-smdk5250.dts >>> @@ -131,6 +131,9 @@ >>> reg = <0x09>; >>> interrupt-parent = <&gpx3>; >>> interrupts = <2 IRQ_TYPE_NONE>; >>> + pinctrl-names = "default"; >>> + pinctrl-0 = <&max77686_irq>; >>> + wakeup-source; >>> >>> voltage-regulators { >>> ldo1_reg: LDO1 { >>> @@ -411,3 +414,12 @@ >>> }; >>> }; >>> }; >>> + >>> +&pinctrl_0 { >>> + max77686_irq: max77686-irq { >>> + samsung,pins = "gpx3-2"; >>> + samsung,pin-function = <0xf>; >>> + samsung,pin-pud = <0>; >>> + samsung,pin-drv = <0>; >>> + }; >>> +}; >>> >> -- >> To unsubscribe from this list: send the line "unsubscribe >> linux-samsung-soc" in >> the body of a message to majordomo at vger.kernel.org >> More majordomo info at http://vger.kernel.org/majordomo-info.html > ^ permalink raw reply [flat|nested] 14+ messages in thread
* [2/2] ARM: dts: Add missing irq pinctrl for max77686 on smdk5250 2015-08-17 5:50 ` Krzysztof Kozlowski @ 2015-08-18 6:13 ` Pankaj Dubey 2015-08-18 6:27 ` Pankaj Dubey 1 sibling, 0 replies; 14+ messages in thread From: Pankaj Dubey @ 2015-08-18 6:13 UTC (permalink / raw) To: linux-arm-kernel Hi Krzysztof, On Monday 17 August 2015 11:20 AM, Krzysztof Kozlowski wrote: > 2015-05-02 14:49 GMT+09:00 Krzysztof Kozlowski <k.kozlowski.k@gmail.com>: >> W dniu 28.04.2015 o 20:02, Pankaj Dubey pisze: >>> Hi Kukjin, >>> >>> Any updated for this patch. Without this PMIC RTC is not working on >>> SMDK5250. >>> >> >> Could you resend the patch after rebasing it? >> >> Best regards, >> Krzysztof > > Dear Pankaj and Yadwinder, > > Do you plan to do anything with this patch? > Thanks for reminding us. Somehow I missed this. I will resend the patch shortly after rebasing and testing. Thanks, Pankaj Dubey > Best regards, > Krzysztof > > >>> Thanks, >>> Pankaj Dubey >>> >>> On Tuesday 18 November 2014 05:38 PM, Yadwinder Singh Brar wrote: >>>> This patch adds pinctrl configuration for using configuring gpx3-2 as an >>>> external interrupt from max77686. Though max77686 RTC is enabled and gets >>>> probed by default, it doesnt work as its unable to get interrupt. >>>> >>>> This patch makes max77686 RTC work and also configures it as wakeup >>>> source. >>>> >>>> Signed-off-by: Yadwinder Singh Brar <yadi.brar@samsung.com> >>>> Reviewed-by: Javier Martinez Canillas <javier.martinez@collabora.co.uk> >>>> Tested-by: Pankaj Dubey <pankaj.dubey@samsung.com> >>>> >>>> --- >>>> arch/arm/boot/dts/exynos5250-smdk5250.dts | 12 ++++++++++++ >>>> 1 files changed, 12 insertions(+), 0 deletions(-) >>>> >>>> diff --git a/arch/arm/boot/dts/exynos5250-smdk5250.dts >>>> b/arch/arm/boot/dts/exynos5250-smdk5250.dts >>>> index 95b5b51..19cd918 100644 >>>> --- a/arch/arm/boot/dts/exynos5250-smdk5250.dts >>>> +++ b/arch/arm/boot/dts/exynos5250-smdk5250.dts >>>> @@ -131,6 +131,9 @@ >>>> reg = <0x09>; >>>> interrupt-parent = <&gpx3>; >>>> interrupts = <2 IRQ_TYPE_NONE>; >>>> + pinctrl-names = "default"; >>>> + pinctrl-0 = <&max77686_irq>; >>>> + wakeup-source; >>>> >>>> voltage-regulators { >>>> ldo1_reg: LDO1 { >>>> @@ -411,3 +414,12 @@ >>>> }; >>>> }; >>>> }; >>>> + >>>> +&pinctrl_0 { >>>> + max77686_irq: max77686-irq { >>>> + samsung,pins = "gpx3-2"; >>>> + samsung,pin-function = <0xf>; >>>> + samsung,pin-pud = <0>; >>>> + samsung,pin-drv = <0>; >>>> + }; >>>> +}; >>>> >>> -- >>> To unsubscribe from this list: send the line "unsubscribe >>> linux-samsung-soc" in >>> the body of a message to majordomo at vger.kernel.org >>> More majordomo info at http://vger.kernel.org/majordomo-info.html >> > ^ permalink raw reply [flat|nested] 14+ messages in thread
* [2/2] ARM: dts: Add missing irq pinctrl for max77686 on smdk5250 2015-08-17 5:50 ` Krzysztof Kozlowski 2015-08-18 6:13 ` Pankaj Dubey @ 2015-08-18 6:27 ` Pankaj Dubey 2015-08-18 7:00 ` Krzysztof Kozlowski 1 sibling, 1 reply; 14+ messages in thread From: Pankaj Dubey @ 2015-08-18 6:27 UTC (permalink / raw) To: linux-arm-kernel Hi Krzysztof, On Monday 17 August 2015 11:20 AM, Krzysztof Kozlowski wrote: > 2015-05-02 14:49 GMT+09:00 Krzysztof Kozlowski <k.kozlowski.k@gmail.com>: >> W dniu 28.04.2015 o 20:02, Pankaj Dubey pisze: >>> Hi Kukjin, >>> >>> Any updated for this patch. Without this PMIC RTC is not working on >>> SMDK5250. >>> >> >> Could you resend the patch after rebasing it? >> >> Best regards, >> Krzysztof > > Dear Pankaj and Yadwinder, > > Do you plan to do anything with this patch? > > Best regards, > Krzysztof > > Just noticed that the patch in subject is already merged in kgene tree. There was another patch "[1/2]: ARM: dts: Fix bootup issue on smdk5250" which was posted along with this is still not merged. Resending it once again. [1]: http://www.spinics.net/lists/linux-samsung-soc/msg39100.html Thanks, Pankaj Dubey >>> Thanks, >>> Pankaj Dubey >>> >>> On Tuesday 18 November 2014 05:38 PM, Yadwinder Singh Brar wrote: >>>> This patch adds pinctrl configuration for using configuring gpx3-2 as an >>>> external interrupt from max77686. Though max77686 RTC is enabled and gets >>>> probed by default, it doesnt work as its unable to get interrupt. >>>> >>>> This patch makes max77686 RTC work and also configures it as wakeup >>>> source. >>>> >>>> Signed-off-by: Yadwinder Singh Brar <yadi.brar@samsung.com> >>>> Reviewed-by: Javier Martinez Canillas <javier.martinez@collabora.co.uk> >>>> Tested-by: Pankaj Dubey <pankaj.dubey@samsung.com> >>>> >>>> --- >>>> arch/arm/boot/dts/exynos5250-smdk5250.dts | 12 ++++++++++++ >>>> 1 files changed, 12 insertions(+), 0 deletions(-) >>>> >>>> diff --git a/arch/arm/boot/dts/exynos5250-smdk5250.dts >>>> b/arch/arm/boot/dts/exynos5250-smdk5250.dts >>>> index 95b5b51..19cd918 100644 >>>> --- a/arch/arm/boot/dts/exynos5250-smdk5250.dts >>>> +++ b/arch/arm/boot/dts/exynos5250-smdk5250.dts >>>> @@ -131,6 +131,9 @@ >>>> reg = <0x09>; >>>> interrupt-parent = <&gpx3>; >>>> interrupts = <2 IRQ_TYPE_NONE>; >>>> + pinctrl-names = "default"; >>>> + pinctrl-0 = <&max77686_irq>; >>>> + wakeup-source; >>>> >>>> voltage-regulators { >>>> ldo1_reg: LDO1 { >>>> @@ -411,3 +414,12 @@ >>>> }; >>>> }; >>>> }; >>>> + >>>> +&pinctrl_0 { >>>> + max77686_irq: max77686-irq { >>>> + samsung,pins = "gpx3-2"; >>>> + samsung,pin-function = <0xf>; >>>> + samsung,pin-pud = <0>; >>>> + samsung,pin-drv = <0>; >>>> + }; >>>> +}; >>>> >>> -- >>> To unsubscribe from this list: send the line "unsubscribe >>> linux-samsung-soc" in >>> the body of a message to majordomo at vger.kernel.org >>> More majordomo info at http://vger.kernel.org/majordomo-info.html >> > ^ permalink raw reply [flat|nested] 14+ messages in thread
* [2/2] ARM: dts: Add missing irq pinctrl for max77686 on smdk5250 2015-08-18 6:27 ` Pankaj Dubey @ 2015-08-18 7:00 ` Krzysztof Kozlowski 0 siblings, 0 replies; 14+ messages in thread From: Krzysztof Kozlowski @ 2015-08-18 7:00 UTC (permalink / raw) To: linux-arm-kernel On 18.08.2015 15:27, Pankaj Dubey wrote: > Hi Krzysztof, > > On Monday 17 August 2015 11:20 AM, Krzysztof Kozlowski wrote: >> 2015-05-02 14:49 GMT+09:00 Krzysztof Kozlowski <k.kozlowski.k@gmail.com>: >>> W dniu 28.04.2015 o 20:02, Pankaj Dubey pisze: >>>> Hi Kukjin, >>>> >>>> Any updated for this patch. Without this PMIC RTC is not working on >>>> SMDK5250. >>>> >>> >>> Could you resend the patch after rebasing it? >>> >>> Best regards, >>> Krzysztof >> >> Dear Pankaj and Yadwinder, >> >> Do you plan to do anything with this patch? >> >> Best regards, >> Krzysztof >> >> > > Just noticed that the patch in subject is already merged in kgene tree. Yes, you're right. > There was another patch "[1/2]: ARM: dts: Fix bootup issue on smdk5250" > which was posted along with this is still not merged. Resending it once > again. > > [1]: http://www.spinics.net/lists/linux-samsung-soc/msg39100.html Thanks! Best regards, Krzysztof ^ permalink raw reply [flat|nested] 14+ messages in thread
* [1/2] ARM: dts: Fix bootup issue on smdk5250 2014-11-18 12:08 [PATCH 1/2] ARM: dts: Fix bootup issue on smdk5250 Yadwinder Singh Brar 2014-11-18 12:08 ` [PATCH 2/2] ARM: dts: Add missing irq pinctrl for max77686 " Yadwinder Singh Brar @ 2014-11-20 4:01 ` Pankaj Dubey 2014-11-22 3:25 ` Kukjin Kim 1 sibling, 1 reply; 14+ messages in thread From: Pankaj Dubey @ 2014-11-20 4:01 UTC (permalink / raw) To: linux-arm-kernel Hi, On Tuesday 18 November 2014 05:38 PM, Yadwinder Singh Brar wrote: > With default config on smdk5250 latest tree throws below message : > > [ 2.226049] thermal thermal_zone0: critical temperature reached(224 C),shutting down > [ 2.227840] reboot: Failed to start orderly shutdown: forcing the issue > > and hangs randomly because it reads wrong temperature value. > > I can't figure out any direct relation between LDO10 and TMU from board > schematics which I have. So making LDO10 always-on to fix issue for now. > > Signed-off-by: Yadwinder Singh Brar <yadi.brar@samsung.com> > Tested this on SMDK5250 board, system boot is fine now, hence Tested-by: Pankaj Dubey <pankaj.dubey@samsung.com> > --- > arch/arm/boot/dts/exynos5250-smdk5250.dts | 1 + > 1 files changed, 1 insertions(+), 0 deletions(-) > > diff --git a/arch/arm/boot/dts/exynos5250-smdk5250.dts b/arch/arm/boot/dts/exynos5250-smdk5250.dts > index bc27cc2..95b5b51 100644 > --- a/arch/arm/boot/dts/exynos5250-smdk5250.dts > +++ b/arch/arm/boot/dts/exynos5250-smdk5250.dts > @@ -190,6 +190,7 @@ > regulator-name = "P1.8V_LDO_OUT10"; > regulator-min-microvolt = <1800000>; > regulator-max-microvolt = <1800000>; > + regulator-always-on; > }; > > ldo11_reg: LDO11 { > ^ permalink raw reply [flat|nested] 14+ messages in thread
* [1/2] ARM: dts: Fix bootup issue on smdk5250 2014-11-20 4:01 ` [1/2] ARM: dts: Fix bootup issue " Pankaj Dubey @ 2014-11-22 3:25 ` Kukjin Kim 2014-11-25 13:16 ` Yadwinder Singh Brar 2015-04-28 11:05 ` Pankaj Dubey 0 siblings, 2 replies; 14+ messages in thread From: Kukjin Kim @ 2014-11-22 3:25 UTC (permalink / raw) To: linux-arm-kernel Pankaj Dubey wrote: > > Hi, > Hi, Yadwinder, please don't miss [PATCH 1/2] in subject, so that my e-mail client can't filter wrong ;) > On Tuesday 18 November 2014 05:38 PM, Yadwinder Singh Brar wrote: > > With default config on smdk5250 latest tree throws below message : > > > > [ 2.226049] thermal thermal_zone0: critical temperature reached(224 C),shutting down > > [ 2.227840] reboot: Failed to start orderly shutdown: forcing the issue > > > > and hangs randomly because it reads wrong temperature value. > > > > I can't figure out any direct relation between LDO10 and TMU from board > > schematics which I have. So making LDO10 always-on to fix issue for now. > > Hmm...strange, I also can't see any relations now...but I need to contact to hardware guy before applying this, this fixes the problem though. Thanks, Kukjin > > Signed-off-by: Yadwinder Singh Brar <yadi.brar@samsung.com> > > > > Tested this on SMDK5250 board, system boot is fine now, hence > > Tested-by: Pankaj Dubey <pankaj.dubey@samsung.com> > > > --- > > arch/arm/boot/dts/exynos5250-smdk5250.dts | 1 + > > 1 files changed, 1 insertions(+), 0 deletions(-) > > > > diff --git a/arch/arm/boot/dts/exynos5250-smdk5250.dts b/arch/arm/boot/dts/exynos5250-smdk5250.dts > > index bc27cc2..95b5b51 100644 > > --- a/arch/arm/boot/dts/exynos5250-smdk5250.dts > > +++ b/arch/arm/boot/dts/exynos5250-smdk5250.dts > > @@ -190,6 +190,7 @@ > > regulator-name = "P1.8V_LDO_OUT10"; > > regulator-min-microvolt = <1800000>; > > regulator-max-microvolt = <1800000>; > > + regulator-always-on; > > }; > > > > ldo11_reg: LDO11 { > > ^ permalink raw reply [flat|nested] 14+ messages in thread
* [1/2] ARM: dts: Fix bootup issue on smdk5250 2014-11-22 3:25 ` Kukjin Kim @ 2014-11-25 13:16 ` Yadwinder Singh Brar 2015-04-28 11:05 ` Pankaj Dubey 1 sibling, 0 replies; 14+ messages in thread From: Yadwinder Singh Brar @ 2014-11-25 13:16 UTC (permalink / raw) To: linux-arm-kernel Hi Kukjin, > Subject: RE: [1/2] ARM: dts: Fix bootup issue on smdk5250 > > Pankaj Dubey wrote: > > > > Hi, > > > Hi, > > Yadwinder, please don't miss [PATCH 1/2] in subject, so that my e-mail > client can't filter wrong ;) > Sure, but I think your mail client had played some trick, as patches were posted with [PATCH */2] prefix. > > On Tuesday 18 November 2014 05:38 PM, Yadwinder Singh Brar wrote: > > > With default config on smdk5250 latest tree throws below message : > > > > > > [ 2.226049] thermal thermal_zone0: critical temperature > reached(224 C),shutting down > > > [ 2.227840] reboot: Failed to start orderly shutdown: forcing > the issue > > > > > > and hangs randomly because it reads wrong temperature value. > > > > > > I can't figure out any direct relation between LDO10 and TMU from > > > board schematics which I have. So making LDO10 always-on to fix > issue for now. > > > > Hmm...strange, I also can't see any relations now...but I need to > contact to hardware guy before applying this, this fixes the problem > though. > Ok, sure it will be good know actual relation from hardware guy. Meanwhile can we get another patch[1] merged which was posted along with patch just because both patches were on same file. Otherwise there is no dependency between them and can be applied independently. Best Regards, Yadwinder [1] : http://www.spinics.net/lists/linux-samsung-soc/msg39107.html > Thanks, > Kukjin > > > > Signed-off-by: Yadwinder Singh Brar <yadi.brar@samsung.com> > > > > > > > Tested this on SMDK5250 board, system boot is fine now, hence > > > > Tested-by: Pankaj Dubey <pankaj.dubey@samsung.com> > > > > > --- > > > arch/arm/boot/dts/exynos5250-smdk5250.dts | 1 + > > > 1 files changed, 1 insertions(+), 0 deletions(-) > > > > > > diff --git a/arch/arm/boot/dts/exynos5250-smdk5250.dts > > > b/arch/arm/boot/dts/exynos5250-smdk5250.dts > > > index bc27cc2..95b5b51 100644 > > > --- a/arch/arm/boot/dts/exynos5250-smdk5250.dts > > > +++ b/arch/arm/boot/dts/exynos5250-smdk5250.dts > > > @@ -190,6 +190,7 @@ > > > regulator-name = "P1.8V_LDO_OUT10"; > > > regulator-min-microvolt = <1800000>; > > > regulator-max-microvolt = <1800000>; > > > + regulator-always-on; > > > }; > > > > > > ldo11_reg: LDO11 { > > > ^ permalink raw reply [flat|nested] 14+ messages in thread
* [1/2] ARM: dts: Fix bootup issue on smdk5250 2014-11-22 3:25 ` Kukjin Kim 2014-11-25 13:16 ` Yadwinder Singh Brar @ 2015-04-28 11:05 ` Pankaj Dubey 1 sibling, 0 replies; 14+ messages in thread From: Pankaj Dubey @ 2015-04-28 11:05 UTC (permalink / raw) To: linux-arm-kernel Hi Kukjin, On Saturday 22 November 2014 08:55 AM, Kukjin Kim wrote: > Pankaj Dubey wrote: >> >> Hi, >> > Hi, > > Yadwinder, please don't miss [PATCH 1/2] in subject, so that my e-mail client > can't filter wrong ;) > >> On Tuesday 18 November 2014 05:38 PM, Yadwinder Singh Brar wrote: >>> With default config on smdk5250 latest tree throws below message : >>> >>> [ 2.226049] thermal thermal_zone0: critical temperature reached(224 C),shutting down >>> [ 2.227840] reboot: Failed to start orderly shutdown: forcing the issue >>> >>> and hangs randomly because it reads wrong temperature value. >>> >>> I can't figure out any direct relation between LDO10 and TMU from board >>> schematics which I have. So making LDO10 always-on to fix issue for now. >>> > Hmm...strange, I also can't see any relations now...but I need to contact to > hardware guy before applying this, this fixes the problem though. > By any chance you got time to check with HW engineer, as issue is there without this patch, SMDK5250 fails to boot and hangs. Thanks, Pankaj Dubey > Thanks, > Kukjin > >>> Signed-off-by: Yadwinder Singh Brar <yadi.brar@samsung.com> >>> >> >> Tested this on SMDK5250 board, system boot is fine now, hence >> >> Tested-by: Pankaj Dubey <pankaj.dubey@samsung.com> >> >>> --- >>> arch/arm/boot/dts/exynos5250-smdk5250.dts | 1 + >>> 1 files changed, 1 insertions(+), 0 deletions(-) >>> >>> diff --git a/arch/arm/boot/dts/exynos5250-smdk5250.dts b/arch/arm/boot/dts/exynos5250-smdk5250.dts >>> index bc27cc2..95b5b51 100644 >>> --- a/arch/arm/boot/dts/exynos5250-smdk5250.dts >>> +++ b/arch/arm/boot/dts/exynos5250-smdk5250.dts >>> @@ -190,6 +190,7 @@ >>> regulator-name = "P1.8V_LDO_OUT10"; >>> regulator-min-microvolt = <1800000>; >>> regulator-max-microvolt = <1800000>; >>> + regulator-always-on; >>> }; >>> >>> ldo11_reg: LDO11 { >>> > > ^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2015-08-18 7:00 UTC | newest] Thread overview: 14+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2014-11-18 12:08 [PATCH 1/2] ARM: dts: Fix bootup issue on smdk5250 Yadwinder Singh Brar 2014-11-18 12:08 ` [PATCH 2/2] ARM: dts: Add missing irq pinctrl for max77686 " Yadwinder Singh Brar 2014-11-18 14:02 ` Javier Martinez Canillas 2014-11-20 4:06 ` [2/2] " Pankaj Dubey 2015-04-28 11:02 ` Pankaj Dubey 2015-05-02 5:49 ` Krzysztof Kozlowski 2015-08-17 5:50 ` Krzysztof Kozlowski 2015-08-18 6:13 ` Pankaj Dubey 2015-08-18 6:27 ` Pankaj Dubey 2015-08-18 7:00 ` Krzysztof Kozlowski 2014-11-20 4:01 ` [1/2] ARM: dts: Fix bootup issue " Pankaj Dubey 2014-11-22 3:25 ` Kukjin Kim 2014-11-25 13:16 ` Yadwinder Singh Brar 2015-04-28 11:05 ` Pankaj Dubey
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).