All of lore.kernel.org
 help / color / mirror / Atom feed
From: Benoit Cousson <b-cousson@ti.com>
To: Paul Walmsley <paul@pwsan.com>
Cc: linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 0/7] ARM: OMAP: second set of PRM/CM/CGRM cleanup patches for 3.8
Date: Mon, 22 Oct 2012 14:14:23 +0200	[thread overview]
Message-ID: <5085389F.6070106@ti.com> (raw)
In-Reply-To: <20121016012448.21844.92339.stgit@dusk.lan>


Hi Paul,

What is CGRM? Is it a typo?

Regards,
Benoit


On 10/16/2012 03:32 AM, Paul Walmsley wrote:
> This series removes the omap_prcm_get_reset_sources() function.  This
> was exported from arch/arm/mach-omap2/prcm.c for use by the OMAP
> watchdog driver to report the "boot reason".  This series reimplements
> this using a platform_data function pointer for the time being, but
> after the upcoming PRM/CM drivers are merged, the watchdog driver can be
> patched to use an exported function from the drivers.
> 
> This series will also be used as a base for other PRM/CM cleanup during the
> 3.8 time frame, since it adds the prm_register() and prm_unregister() functions.
> These are called by SoC-specific PRM IP block drivers to register function
> pointers with the PRM subsystem.
> 
> This series changes the format of the watchdog's boot reason data to conform
> with the watchdog subsystem standard (the WDIOF_* flags).  If users need
> more detail than that interface provides, either the watchdog interface can
> be patched, or those users can get that information from the upcoming PRM
> drivers.
> 
> 
> - Paul
> 
> ---
> 
> prcm_cleanup_b_3.8
>    text	   data	    bss	    dec	    hex	filename
> 7519295	 696796	5613996	13830087	 d307c7	vmlinux.omap2plus_defconfig.orig
> 7520203	 696868	5613996	13831067	 d30b9b	vmlinux.omap2plus_defconfig
> 
> Paul Walmsley (7):
>       ARM: OMAP2+: PRM: prepare for use of prm_ll_data function pointers
>       ARM: OMAP2+: CM: prepare for use of cm_ll_data function pointers
>       ARM: OMAP1: create read_reset_sources() function (for initial use by watchdog)
>       ARM: OMAP2+: PRM: create PRM reset source API for the watchdog timer driver
>       ARM: OMAP2+: WDT: move init; add read_reset_sources pdata function pointer
>       watchdog: OMAP: use standard GETBOOTSTATUS interface; use platform_data fn ptr
>       ARM: OMAP2+: PRCM: remove omap_prcm_get_reset_sources()
> 
> 
>  arch/arm/mach-omap1/common.h                |    2 +
>  arch/arm/mach-omap1/devices.c               |   21 ++++++-
>  arch/arm/mach-omap1/reset.c                 |   39 +++++++++++++
>  arch/arm/mach-omap2/Makefile                |    2 -
>  arch/arm/mach-omap2/cm.h                    |   12 ++++
>  arch/arm/mach-omap2/cm_common.c             |   71 +++++++++++++++++++++++
>  arch/arm/mach-omap2/devices.c               |   26 --------
>  arch/arm/mach-omap2/prcm.c                  |   12 ----
>  arch/arm/mach-omap2/prm-regbits-24xx.h      |    4 +
>  arch/arm/mach-omap2/prm-regbits-34xx.h      |   10 +++
>  arch/arm/mach-omap2/prm.h                   |   53 +++++++++++++++++
>  arch/arm/mach-omap2/prm2xxx.c               |   68 ++++++++++++++++++++++
>  arch/arm/mach-omap2/prm2xxx.h               |    4 +
>  arch/arm/mach-omap2/prm2xxx_3xxx.h          |    2 +
>  arch/arm/mach-omap2/prm3xxx.c               |   65 +++++++++++++++++++++
>  arch/arm/mach-omap2/prm3xxx.h               |    2 +
>  arch/arm/mach-omap2/prm44xx.c               |   83 ++++++++++++++++++++++++++-
>  arch/arm/mach-omap2/prm44xx.h               |    2 +
>  arch/arm/mach-omap2/prm_common.c            |   78 +++++++++++++++++++++++++
>  arch/arm/mach-omap2/wd_timer.c              |   33 +++++++++++
>  arch/arm/plat-omap/include/plat/prcm.h      |    1 
>  drivers/watchdog/omap_wdt.c                 |   26 ++++----
>  include/linux/platform_data/omap-wd-timer.h |   38 ++++++++++++
>  23 files changed, 594 insertions(+), 60 deletions(-)
>  create mode 100644 arch/arm/mach-omap2/cm_common.c
>  create mode 100644 include/linux/platform_data/omap-wd-timer.h
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 


WARNING: multiple messages have this Message-ID (diff)
From: b-cousson@ti.com (Benoit Cousson)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 0/7] ARM: OMAP: second set of PRM/CM/CGRM cleanup patches for 3.8
Date: Mon, 22 Oct 2012 14:14:23 +0200	[thread overview]
Message-ID: <5085389F.6070106@ti.com> (raw)
In-Reply-To: <20121016012448.21844.92339.stgit@dusk.lan>


Hi Paul,

What is CGRM? Is it a typo?

Regards,
Benoit


On 10/16/2012 03:32 AM, Paul Walmsley wrote:
> This series removes the omap_prcm_get_reset_sources() function.  This
> was exported from arch/arm/mach-omap2/prcm.c for use by the OMAP
> watchdog driver to report the "boot reason".  This series reimplements
> this using a platform_data function pointer for the time being, but
> after the upcoming PRM/CM drivers are merged, the watchdog driver can be
> patched to use an exported function from the drivers.
> 
> This series will also be used as a base for other PRM/CM cleanup during the
> 3.8 time frame, since it adds the prm_register() and prm_unregister() functions.
> These are called by SoC-specific PRM IP block drivers to register function
> pointers with the PRM subsystem.
> 
> This series changes the format of the watchdog's boot reason data to conform
> with the watchdog subsystem standard (the WDIOF_* flags).  If users need
> more detail than that interface provides, either the watchdog interface can
> be patched, or those users can get that information from the upcoming PRM
> drivers.
> 
> 
> - Paul
> 
> ---
> 
> prcm_cleanup_b_3.8
>    text	   data	    bss	    dec	    hex	filename
> 7519295	 696796	5613996	13830087	 d307c7	vmlinux.omap2plus_defconfig.orig
> 7520203	 696868	5613996	13831067	 d30b9b	vmlinux.omap2plus_defconfig
> 
> Paul Walmsley (7):
>       ARM: OMAP2+: PRM: prepare for use of prm_ll_data function pointers
>       ARM: OMAP2+: CM: prepare for use of cm_ll_data function pointers
>       ARM: OMAP1: create read_reset_sources() function (for initial use by watchdog)
>       ARM: OMAP2+: PRM: create PRM reset source API for the watchdog timer driver
>       ARM: OMAP2+: WDT: move init; add read_reset_sources pdata function pointer
>       watchdog: OMAP: use standard GETBOOTSTATUS interface; use platform_data fn ptr
>       ARM: OMAP2+: PRCM: remove omap_prcm_get_reset_sources()
> 
> 
>  arch/arm/mach-omap1/common.h                |    2 +
>  arch/arm/mach-omap1/devices.c               |   21 ++++++-
>  arch/arm/mach-omap1/reset.c                 |   39 +++++++++++++
>  arch/arm/mach-omap2/Makefile                |    2 -
>  arch/arm/mach-omap2/cm.h                    |   12 ++++
>  arch/arm/mach-omap2/cm_common.c             |   71 +++++++++++++++++++++++
>  arch/arm/mach-omap2/devices.c               |   26 --------
>  arch/arm/mach-omap2/prcm.c                  |   12 ----
>  arch/arm/mach-omap2/prm-regbits-24xx.h      |    4 +
>  arch/arm/mach-omap2/prm-regbits-34xx.h      |   10 +++
>  arch/arm/mach-omap2/prm.h                   |   53 +++++++++++++++++
>  arch/arm/mach-omap2/prm2xxx.c               |   68 ++++++++++++++++++++++
>  arch/arm/mach-omap2/prm2xxx.h               |    4 +
>  arch/arm/mach-omap2/prm2xxx_3xxx.h          |    2 +
>  arch/arm/mach-omap2/prm3xxx.c               |   65 +++++++++++++++++++++
>  arch/arm/mach-omap2/prm3xxx.h               |    2 +
>  arch/arm/mach-omap2/prm44xx.c               |   83 ++++++++++++++++++++++++++-
>  arch/arm/mach-omap2/prm44xx.h               |    2 +
>  arch/arm/mach-omap2/prm_common.c            |   78 +++++++++++++++++++++++++
>  arch/arm/mach-omap2/wd_timer.c              |   33 +++++++++++
>  arch/arm/plat-omap/include/plat/prcm.h      |    1 
>  drivers/watchdog/omap_wdt.c                 |   26 ++++----
>  include/linux/platform_data/omap-wd-timer.h |   38 ++++++++++++
>  23 files changed, 594 insertions(+), 60 deletions(-)
>  create mode 100644 arch/arm/mach-omap2/cm_common.c
>  create mode 100644 include/linux/platform_data/omap-wd-timer.h
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> the body of a message to majordomo at vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 

  parent reply	other threads:[~2012-10-22 12:14 UTC|newest]

Thread overview: 60+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-10-16  1:32 [PATCH 0/7] ARM: OMAP: second set of PRM/CM/CGRM cleanup patches for 3.8 Paul Walmsley
2012-10-16  1:32 ` Paul Walmsley
2012-10-16  1:32 ` [PATCH 1/7] ARM: OMAP2+: PRM: prepare for use of prm_ll_data function pointers Paul Walmsley
2012-10-16  1:32   ` Paul Walmsley
2012-10-16  1:32 ` [PATCH 2/7] ARM: OMAP2+: CM: prepare for use of cm_ll_data " Paul Walmsley
2012-10-16  1:32   ` Paul Walmsley
2012-10-16  1:32 ` [PATCH 3/7] ARM: OMAP1: create read_reset_sources() function (for initial use by watchdog) Paul Walmsley
2012-10-16  1:32   ` Paul Walmsley
2012-10-16  1:32 ` [PATCH 4/7] ARM: OMAP2+: PRM: create PRM reset source API for the watchdog timer driver Paul Walmsley
2012-10-16  1:32   ` Paul Walmsley
2012-10-16  1:32 ` [PATCH 5/7] ARM: OMAP2+: WDT: move init; add read_reset_sources pdata function pointer Paul Walmsley
2012-10-16  1:32   ` Paul Walmsley
2012-10-23  8:02   ` Wim Van Sebroeck
2012-10-25 15:38   ` Aaro Koskinen
2012-10-25 15:38     ` Aaro Koskinen
2012-10-25 18:51     ` Paul Walmsley
2012-10-25 18:51       ` Paul Walmsley
2012-10-25 18:57       ` Tony Lindgren
2012-10-25 18:57         ` Tony Lindgren
2012-10-25 19:09         ` Paul Walmsley
2012-10-25 19:09           ` Paul Walmsley
2012-10-25 19:19           ` Tony Lindgren
2012-10-25 19:19             ` Tony Lindgren
2012-10-25 19:31             ` Paul Walmsley
2012-10-25 19:31               ` Paul Walmsley
2012-10-25 19:34               ` Tony Lindgren
2012-10-25 19:34                 ` Tony Lindgren
2012-10-25 19:42                 ` Tony Lindgren
2012-10-25 19:42                   ` Tony Lindgren
2012-10-25 19:57                 ` Paul Walmsley
2012-10-25 19:57                   ` Paul Walmsley
2012-10-25 20:08                   ` Aaro Koskinen
2012-10-25 20:08                     ` Aaro Koskinen
2012-10-25 20:09                     ` Paul Walmsley
2012-10-25 20:09                       ` Paul Walmsley
2012-11-08 19:26   ` Paul Walmsley
2012-11-08 19:26     ` Paul Walmsley
2012-10-16  1:32 ` [PATCH 6/7] watchdog: OMAP: use standard GETBOOTSTATUS interface; use platform_data fn ptr Paul Walmsley
2012-10-16  1:32   ` Paul Walmsley
2012-10-23  8:03   ` Wim Van Sebroeck
2012-10-25 20:14   ` Jon Hunter
2012-10-25 20:14     ` Jon Hunter
2012-10-25 20:14     ` Jon Hunter
2012-10-25 20:16     ` Paul Walmsley
2012-10-25 20:16       ` Paul Walmsley
2012-10-25 20:29       ` Paul Walmsley
2012-10-25 20:29         ` Paul Walmsley
2012-10-25 20:59         ` Felipe Balbi
2012-10-25 20:59           ` Felipe Balbi
2012-10-25 20:59           ` Felipe Balbi
2012-10-25 21:09           ` Paul Walmsley
2012-10-25 21:09             ` Paul Walmsley
2012-10-16  1:32 ` [PATCH 7/7] ARM: OMAP2+: PRCM: remove omap_prcm_get_reset_sources() Paul Walmsley
2012-10-16  1:32   ` Paul Walmsley
2012-10-22 12:14 ` Benoit Cousson [this message]
2012-10-22 12:14   ` [PATCH 0/7] ARM: OMAP: second set of PRM/CM/CGRM cleanup patches for 3.8 Benoit Cousson
2012-10-22 17:06   ` Paul Walmsley
2012-10-22 17:06     ` Paul Walmsley
2012-10-22 17:29     ` Benoit Cousson
2012-10-22 17:29       ` Benoit Cousson

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=5085389F.6070106@ti.com \
    --to=b-cousson@ti.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=paul@pwsan.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.