All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sebastien Guiriec <s-guiriec@ti.com>
To: "Sebastien Guiriec" <s-guiriec@ti.com>,
	"Benoît Cousson" <b-cousson@ti.com>,
	"Paul Walmsley" <paul@pwsan.com>
Cc: Tony Lindgren <tony@atomide.com>,
	linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
	Peter Ujfalusi <peter.ujfalusi@ti.com>,
	Tero Kristo <t-kristo@ti.com>, Jon Hunter <jon-hunter@ti.com>
Subject: Re: [PATCH v2 0/5] ARM: OMAP4: Enable AESS IP.
Date: Fri, 18 Jan 2013 15:47:38 +0100	[thread overview]
Message-ID: <50F9608A.8060102@ti.com> (raw)
In-Reply-To: <1357743792-13917-1-git-send-email-s-guiriec@ti.com>

Paul, Benoit,

Any comments before I resend the serie with the minor comment for Felipe?

Sebastien

On 01/09/2013 04:03 PM, Sebastien Guiriec wrote:
> v2:
> - Add missing AESS memory banks.
> - Update the serie base on comments related earlier serie:
> http://www.mail-archive.com/linux-omap@vger.kernel.org/msg69853.html
>
> The location of the callback has been updated in order to fit with AESS.
> Some call of PM runtime can be done by ASoC core when AESS is in RET so
> we need to ensure that the additional callback is call on enable
> also in order to have clock correctly stop.
>
> v1:
> ARM: OMAP4: Enable AESS IP.
>
> This patch serie extends the the hwmod HWMOD_EXT_OPT_MAIN_CLK flag for the
> AESS IP. This IP has additional register for Auto Gatting configuration. As
> it is used only for Audio the driver is not always loaded. We can reuse the
> same flag as McPDM to work around the HW problem due to bad reset value of
> AESS Auto gatting configuration register.
>
> If we try to setup and reset AESS during boot time without this serie the
> next clocks will still remain enable.
>
> omapconf abe cfg:
> |--------------------------------------------------------------|
> | ABE        Clock Domain Configuration                        |
> |--------------------------------|-----------------------------|
> | Clock State Transition control | HW-Auto                     |
> | Clock State                    |                             |
> |   ABE_24M_FCLK                 | GATED                       |
> |   ABE_ALWON_32K_CLK            | GATED                       |
> |   ABE_SYSCLK                   | GATED                       |
> |   24M_FCLK                     | GATED                       |
> |   ABE_ICLK2                    | RUNNING                     |
> |   DPLL_ABE_X2_CLK              | RUNNING                     |
> |   PAD_CLKS                     | GATED                       |
> |   SLIMBUS_CLK                  | GATED                       |
> | OPP Divider                    | ABE_CLK = DPLL_ABE_X2_CLK/1 |
> |--------------------------------------------------------------|
>
> Paul Walmsley (3):
>    ARM: OMAP2+: hwmod: add enable_preprogram hook
>    ASoC: TI AESS: add autogating-enable function, callable from
>      architecture code
>    ARM: OMAP4+: AESS: enable internal auto-gating during initial setup
>
> Sebastien Guiriec (2):
>    OMAP4: hwmod data: Enable AESS hwmod device
>    OMAP4: hwmod data: Update AESS data with memory bank area
>
>   arch/arm/mach-omap2/Makefile               |    2 +-
>   arch/arm/mach-omap2/omap_hwmod.c           |   18 ++++++++++
>   arch/arm/mach-omap2/omap_hwmod.h           |    8 +++++
>   arch/arm/mach-omap2/omap_hwmod_44xx_data.c |   49 +++++++++++++++++++++++--
>   arch/arm/mach-omap2/omap_hwmod_reset.c     |   52 +++++++++++++++++++++++++++
>   include/sound/aess.h                       |   53 ++++++++++++++++++++++++++++
>   6 files changed, 178 insertions(+), 4 deletions(-)
>   create mode 100644 arch/arm/mach-omap2/omap_hwmod_reset.c
>   create mode 100644 include/sound/aess.h
>


WARNING: multiple messages have this Message-ID (diff)
From: s-guiriec@ti.com (Sebastien Guiriec)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v2 0/5] ARM: OMAP4: Enable AESS IP.
Date: Fri, 18 Jan 2013 15:47:38 +0100	[thread overview]
Message-ID: <50F9608A.8060102@ti.com> (raw)
In-Reply-To: <1357743792-13917-1-git-send-email-s-guiriec@ti.com>

Paul, Benoit,

Any comments before I resend the serie with the minor comment for Felipe?

Sebastien

On 01/09/2013 04:03 PM, Sebastien Guiriec wrote:
> v2:
> - Add missing AESS memory banks.
> - Update the serie base on comments related earlier serie:
> http://www.mail-archive.com/linux-omap at vger.kernel.org/msg69853.html
>
> The location of the callback has been updated in order to fit with AESS.
> Some call of PM runtime can be done by ASoC core when AESS is in RET so
> we need to ensure that the additional callback is call on enable
> also in order to have clock correctly stop.
>
> v1:
> ARM: OMAP4: Enable AESS IP.
>
> This patch serie extends the the hwmod HWMOD_EXT_OPT_MAIN_CLK flag for the
> AESS IP. This IP has additional register for Auto Gatting configuration. As
> it is used only for Audio the driver is not always loaded. We can reuse the
> same flag as McPDM to work around the HW problem due to bad reset value of
> AESS Auto gatting configuration register.
>
> If we try to setup and reset AESS during boot time without this serie the
> next clocks will still remain enable.
>
> omapconf abe cfg:
> |--------------------------------------------------------------|
> | ABE        Clock Domain Configuration                        |
> |--------------------------------|-----------------------------|
> | Clock State Transition control | HW-Auto                     |
> | Clock State                    |                             |
> |   ABE_24M_FCLK                 | GATED                       |
> |   ABE_ALWON_32K_CLK            | GATED                       |
> |   ABE_SYSCLK                   | GATED                       |
> |   24M_FCLK                     | GATED                       |
> |   ABE_ICLK2                    | RUNNING                     |
> |   DPLL_ABE_X2_CLK              | RUNNING                     |
> |   PAD_CLKS                     | GATED                       |
> |   SLIMBUS_CLK                  | GATED                       |
> | OPP Divider                    | ABE_CLK = DPLL_ABE_X2_CLK/1 |
> |--------------------------------------------------------------|
>
> Paul Walmsley (3):
>    ARM: OMAP2+: hwmod: add enable_preprogram hook
>    ASoC: TI AESS: add autogating-enable function, callable from
>      architecture code
>    ARM: OMAP4+: AESS: enable internal auto-gating during initial setup
>
> Sebastien Guiriec (2):
>    OMAP4: hwmod data: Enable AESS hwmod device
>    OMAP4: hwmod data: Update AESS data with memory bank area
>
>   arch/arm/mach-omap2/Makefile               |    2 +-
>   arch/arm/mach-omap2/omap_hwmod.c           |   18 ++++++++++
>   arch/arm/mach-omap2/omap_hwmod.h           |    8 +++++
>   arch/arm/mach-omap2/omap_hwmod_44xx_data.c |   49 +++++++++++++++++++++++--
>   arch/arm/mach-omap2/omap_hwmod_reset.c     |   52 +++++++++++++++++++++++++++
>   include/sound/aess.h                       |   53 ++++++++++++++++++++++++++++
>   6 files changed, 178 insertions(+), 4 deletions(-)
>   create mode 100644 arch/arm/mach-omap2/omap_hwmod_reset.c
>   create mode 100644 include/sound/aess.h
>

  parent reply	other threads:[~2013-01-18 14:47 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-01-09 15:03 [PATCH v2 0/5] ARM: OMAP4: Enable AESS IP Sebastien Guiriec
2013-01-09 15:03 ` Sebastien Guiriec
2013-01-09 15:03 ` [PATCH v2 1/5] ARM: OMAP2+: hwmod: add enable_preprogram hook Sebastien Guiriec
2013-01-09 15:03   ` Sebastien Guiriec
2013-01-09 15:07   ` Felipe Balbi
2013-01-09 15:07     ` Felipe Balbi
2013-01-09 15:03 ` [PATCH v2 2/5] ASoC: TI AESS: add autogating-enable function, callable from architecture code Sebastien Guiriec
2013-01-09 15:03   ` Sebastien Guiriec
2013-01-09 15:41   ` Mark Brown
2013-01-09 15:41     ` Mark Brown
2013-01-09 15:03 ` [PATCH v2 3/5] ARM: OMAP4+: AESS: enable internal auto-gating during initial setup Sebastien Guiriec
2013-01-09 15:03   ` Sebastien Guiriec
2013-01-09 15:03 ` [PATCH v2 4/5] OMAP4: hwmod data: Enable AESS hwmod device Sebastien Guiriec
2013-01-09 15:03   ` Sebastien Guiriec
2013-01-09 15:03 ` [PATCH v2 5/5] OMAP4: hwmod data: Update AESS data with memory bank area Sebastien Guiriec
2013-01-09 15:03   ` Sebastien Guiriec
2013-02-10 19:19   ` Paul Walmsley
2013-02-10 19:19     ` Paul Walmsley
2013-01-18 14:47 ` Sebastien Guiriec [this message]
2013-01-18 14:47   ` [PATCH v2 0/5] ARM: OMAP4: Enable AESS IP Sebastien Guiriec
2013-01-21  1:54   ` Paul Walmsley
2013-01-21  1:54     ` Paul Walmsley

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=50F9608A.8060102@ti.com \
    --to=s-guiriec@ti.com \
    --cc=b-cousson@ti.com \
    --cc=jon-hunter@ti.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=paul@pwsan.com \
    --cc=peter.ujfalusi@ti.com \
    --cc=t-kristo@ti.com \
    --cc=tony@atomide.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.