All of lore.kernel.org
 help / color / mirror / Atom feed
From: Pankaj Dubey <pankaj.dubey@samsung.com>
To: kgene.kim@samsung.com, linux-arm-kernel@lists.infradead.org,
	linux-samsung-soc@vger.kernel.org
Cc: tomasz.figa@gmail.com, linux@arm.linux.org.uk,
	thomas.ab@samsung.com, vikas.sajjan@samsung.com,
	linux@roeck-us.net, naushad@samsung.com
Subject: RE: [PATCH v3 0/2] Handle reboot for Exynos SoC via restart_handler
Date: Mon, 10 Nov 2014 16:58:17 +0530	[thread overview]
Message-ID: <000f01cffcd9$9f5dc860$de195920$@samsung.com> (raw)
In-Reply-To: <1412668401-21837-1-git-send-email-pankaj.dubey@samsung.com>

Hi Kukjin,

On Tuesday, October 07, 2014 1:23 PM, Pankaj Dubey wrote:
> Subject: [PATCH v3 0/2] Handle reboot for Exynos SoC via restart_handler
> 
> This patch removes restart hook from machine_desc of Exynos, and moves
respective
> code into reboot_notifiers.
> Exynos5440 handles reboot via clock register so let's register a
restart_handler in
> Exynos5440 clock driver.
> For rest Exynos SoC, reboot is handled via PMU SWRESET register so let's
register a
> restart_handler in PMU driver for handling this.
> 
> This patch is inspired and dependent on following patch series[1] from
Geunter
> Roeck.
> 
> Also this patch is dependent on PMU platform driver patch [2] by me, and
has been
> prepared on top of it.
> 
> [1]: kernel: Add support for kernel restart handler call chain
> https://lkml.org/lkml/2014/8/19/652
> 
> [2]: ARM: Exynos: Convert PMU implementation into a platform driver
> https://lkml.org/lkml/2014/10/6/89
> 
> I have tested reboot functionality on Exynos3250 based board, on
kgene/for-next as
> well as linux-next's next-20140925 tag.
> 
> Patch v2 and related discussion can be found here
> http://www.spinics.net/lists/linux-samsung-soc/msg37457.html
> 
> Changes since v2:
>   - Used register_restart_handler instead of register_reboot_notifier.
>   - Updated notifier names and commit message accordingly.
> 
> Changes since v1:
>   - Addressed review comments from Tomasz Figa. Removed usage of
>     local variables where ever unnecessary.
>   - Make reg_base as global in clk-exynos5440.c file, to avoid
>     iomapping it again in reboot_notifier handler.
> 
> Pankaj Dubey (2):
>   clk: samsung: exynos5440: move restart code into clock driver
>   ARM: EXYNOS: PMU: move restart code into pmu driver
> 
>  arch/arm/mach-exynos/common.h        |    1 -
>  arch/arm/mach-exynos/exynos.c        |   23 -----------------------
>  arch/arm/mach-exynos/pmu.c           |   24
> ++++++++++++++++++++++++
>  drivers/clk/samsung/clk-exynos5440.c |   29
> ++++++++++++++++++++++++++++-
>  4 files changed, 52 insertions(+), 25 deletions(-)
> 

Will you please look into this patch series? 

> --
> 1.7.9.5

WARNING: multiple messages have this Message-ID (diff)
From: pankaj.dubey@samsung.com (Pankaj Dubey)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v3 0/2] Handle reboot for Exynos SoC via restart_handler
Date: Mon, 10 Nov 2014 16:58:17 +0530	[thread overview]
Message-ID: <000f01cffcd9$9f5dc860$de195920$@samsung.com> (raw)
In-Reply-To: <1412668401-21837-1-git-send-email-pankaj.dubey@samsung.com>

Hi Kukjin,

On Tuesday, October 07, 2014 1:23 PM, Pankaj Dubey wrote:
> Subject: [PATCH v3 0/2] Handle reboot for Exynos SoC via restart_handler
> 
> This patch removes restart hook from machine_desc of Exynos, and moves
respective
> code into reboot_notifiers.
> Exynos5440 handles reboot via clock register so let's register a
restart_handler in
> Exynos5440 clock driver.
> For rest Exynos SoC, reboot is handled via PMU SWRESET register so let's
register a
> restart_handler in PMU driver for handling this.
> 
> This patch is inspired and dependent on following patch series[1] from
Geunter
> Roeck.
> 
> Also this patch is dependent on PMU platform driver patch [2] by me, and
has been
> prepared on top of it.
> 
> [1]: kernel: Add support for kernel restart handler call chain
> https://lkml.org/lkml/2014/8/19/652
> 
> [2]: ARM: Exynos: Convert PMU implementation into a platform driver
> https://lkml.org/lkml/2014/10/6/89
> 
> I have tested reboot functionality on Exynos3250 based board, on
kgene/for-next as
> well as linux-next's next-20140925 tag.
> 
> Patch v2 and related discussion can be found here
> http://www.spinics.net/lists/linux-samsung-soc/msg37457.html
> 
> Changes since v2:
>   - Used register_restart_handler instead of register_reboot_notifier.
>   - Updated notifier names and commit message accordingly.
> 
> Changes since v1:
>   - Addressed review comments from Tomasz Figa. Removed usage of
>     local variables where ever unnecessary.
>   - Make reg_base as global in clk-exynos5440.c file, to avoid
>     iomapping it again in reboot_notifier handler.
> 
> Pankaj Dubey (2):
>   clk: samsung: exynos5440: move restart code into clock driver
>   ARM: EXYNOS: PMU: move restart code into pmu driver
> 
>  arch/arm/mach-exynos/common.h        |    1 -
>  arch/arm/mach-exynos/exynos.c        |   23 -----------------------
>  arch/arm/mach-exynos/pmu.c           |   24
> ++++++++++++++++++++++++
>  drivers/clk/samsung/clk-exynos5440.c |   29
> ++++++++++++++++++++++++++++-
>  4 files changed, 52 insertions(+), 25 deletions(-)
> 

Will you please look into this patch series? 

> --
> 1.7.9.5

  parent reply	other threads:[~2014-11-10 11:29 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-10-07  7:53 [PATCH v3 0/2] Handle reboot for Exynos SoC via restart_handler Pankaj Dubey
2014-10-07  7:53 ` Pankaj Dubey
2014-10-07  7:53 ` [PATCH v3 1/2] clk: samsung: exynos5440: move restart code into clock driver Pankaj Dubey
2014-10-07  7:53   ` Pankaj Dubey
2014-10-07  7:53 ` [PATCH v3 2/2] ARM: EXYNOS: PMU: move restart code into pmu driver Pankaj Dubey
2014-10-07  7:53   ` Pankaj Dubey
2014-10-07 16:51 ` [PATCH v3 0/2] Handle reboot for Exynos SoC via restart_handler Guenter Roeck
2014-10-07 16:51   ` Guenter Roeck
2014-11-10 11:28 ` Pankaj Dubey [this message]
2014-11-10 11:28   ` Pankaj Dubey

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='000f01cffcd9$9f5dc860$de195920$@samsung.com' \
    --to=pankaj.dubey@samsung.com \
    --cc=kgene.kim@samsung.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-samsung-soc@vger.kernel.org \
    --cc=linux@arm.linux.org.uk \
    --cc=linux@roeck-us.net \
    --cc=naushad@samsung.com \
    --cc=thomas.ab@samsung.com \
    --cc=tomasz.figa@gmail.com \
    --cc=vikas.sajjan@samsung.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.