All of lore.kernel.org
 help / color / mirror / Atom feed
From: shawnguo@kernel.org (Shawn Guo)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 2/2 v2] ARM: i.MX25: globally disable supervisor protect
Date: Tue, 4 Apr 2017 21:14:52 +0800	[thread overview]
Message-ID: <20170404131450.GD3394@dragon> (raw)
In-Reply-To: <1491248825-23041-2-git-send-email-martin@kaiser.cx>

On Mon, Apr 03, 2017 at 09:47:05PM +0200, Martin Kaiser wrote:
> The problem described in 6befda9a272b98bfb1dc772efc3564644cbfb270

This causes the following checkpatch error.

ERROR: Please use git commit description style 'commit <12+ chars of
sha1> ("<title line>")' - ie: 'commit 6befda9a272b ("ARM: i.MX53:
globally disable supervisor protect")'

I fixed it and applied the patch.

Shawn

> for the i.MX53 platform applies to i.MX25 as well.
> 
> E.g. CSPI1+SDMA and SSI1+SDMA are not working with the default AIPS
> configuration. Modifiy the AIPS configuration to allow access to the bus
> by SDMA and peripherals.
> 
> Signed-off-by: Martin Kaiser <martin@kaiser.cx>
> ---
> v2:
>    rebased against latest linux-next
>    re-sending both patches, it seems they got lost along the way
> 
>  arch/arm/mach-imx/mach-imx25.c | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/arch/arm/mach-imx/mach-imx25.c b/arch/arm/mach-imx/mach-imx25.c
> index 32dcb5e..353b86e 100644
> --- a/arch/arm/mach-imx/mach-imx25.c
> +++ b/arch/arm/mach-imx/mach-imx25.c
> @@ -23,6 +23,11 @@ static void __init imx25_init_early(void)
>  	mxc_set_cpu_type(MXC_CPU_MX25);
>  }
>  
> +static void __init imx25_dt_init(void)
> +{
> +	imx_aips_allow_unprivileged_access("fsl,imx25-aips");
> +}
> +
>  static void __init mx25_init_irq(void)
>  {
>  	struct device_node *np;
> @@ -41,6 +46,7 @@ static const char * const imx25_dt_board_compat[] __initconst = {
>  
>  DT_MACHINE_START(IMX25_DT, "Freescale i.MX25 (Device Tree Support)")
>  	.init_early	= imx25_init_early,
> +	.init_machine	= imx25_dt_init,
>  	.init_late      = imx25_pm_init,
>  	.init_irq	= mx25_init_irq,
>  	.dt_compat	= imx25_dt_board_compat,
> -- 
> 2.1.4
> 
> 
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

WARNING: multiple messages have this Message-ID (diff)
From: Shawn Guo <shawnguo-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
To: Martin Kaiser <martin-XxZfDwE/svGeZLLa646FqQ@public.gmane.org>
Cc: devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
	Sascha Hauer <kernel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: Re: [PATCH 2/2 v2] ARM: i.MX25: globally disable supervisor protect
Date: Tue, 4 Apr 2017 21:14:52 +0800	[thread overview]
Message-ID: <20170404131450.GD3394@dragon> (raw)
In-Reply-To: <1491248825-23041-2-git-send-email-martin-XxZfDwE/svGeZLLa646FqQ@public.gmane.org>

On Mon, Apr 03, 2017 at 09:47:05PM +0200, Martin Kaiser wrote:
> The problem described in 6befda9a272b98bfb1dc772efc3564644cbfb270

This causes the following checkpatch error.

ERROR: Please use git commit description style 'commit <12+ chars of
sha1> ("<title line>")' - ie: 'commit 6befda9a272b ("ARM: i.MX53:
globally disable supervisor protect")'

I fixed it and applied the patch.

Shawn

> for the i.MX53 platform applies to i.MX25 as well.
> 
> E.g. CSPI1+SDMA and SSI1+SDMA are not working with the default AIPS
> configuration. Modifiy the AIPS configuration to allow access to the bus
> by SDMA and peripherals.
> 
> Signed-off-by: Martin Kaiser <martin-XxZfDwE/svGeZLLa646FqQ@public.gmane.org>
> ---
> v2:
>    rebased against latest linux-next
>    re-sending both patches, it seems they got lost along the way
> 
>  arch/arm/mach-imx/mach-imx25.c | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/arch/arm/mach-imx/mach-imx25.c b/arch/arm/mach-imx/mach-imx25.c
> index 32dcb5e..353b86e 100644
> --- a/arch/arm/mach-imx/mach-imx25.c
> +++ b/arch/arm/mach-imx/mach-imx25.c
> @@ -23,6 +23,11 @@ static void __init imx25_init_early(void)
>  	mxc_set_cpu_type(MXC_CPU_MX25);
>  }
>  
> +static void __init imx25_dt_init(void)
> +{
> +	imx_aips_allow_unprivileged_access("fsl,imx25-aips");
> +}
> +
>  static void __init mx25_init_irq(void)
>  {
>  	struct device_node *np;
> @@ -41,6 +46,7 @@ static const char * const imx25_dt_board_compat[] __initconst = {
>  
>  DT_MACHINE_START(IMX25_DT, "Freescale i.MX25 (Device Tree Support)")
>  	.init_early	= imx25_init_early,
> +	.init_machine	= imx25_dt_init,
>  	.init_late      = imx25_pm_init,
>  	.init_irq	= mx25_init_irq,
>  	.dt_compat	= imx25_dt_board_compat,
> -- 
> 2.1.4
> 
> 
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

WARNING: multiple messages have this Message-ID (diff)
From: Shawn Guo <shawnguo@kernel.org>
To: Martin Kaiser <martin@kaiser.cx>
Cc: devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
	Sascha Hauer <kernel@pengutronix.de>,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH 2/2 v2] ARM: i.MX25: globally disable supervisor protect
Date: Tue, 4 Apr 2017 21:14:52 +0800	[thread overview]
Message-ID: <20170404131450.GD3394@dragon> (raw)
In-Reply-To: <1491248825-23041-2-git-send-email-martin@kaiser.cx>

On Mon, Apr 03, 2017 at 09:47:05PM +0200, Martin Kaiser wrote:
> The problem described in 6befda9a272b98bfb1dc772efc3564644cbfb270

This causes the following checkpatch error.

ERROR: Please use git commit description style 'commit <12+ chars of
sha1> ("<title line>")' - ie: 'commit 6befda9a272b ("ARM: i.MX53:
globally disable supervisor protect")'

I fixed it and applied the patch.

Shawn

> for the i.MX53 platform applies to i.MX25 as well.
> 
> E.g. CSPI1+SDMA and SSI1+SDMA are not working with the default AIPS
> configuration. Modifiy the AIPS configuration to allow access to the bus
> by SDMA and peripherals.
> 
> Signed-off-by: Martin Kaiser <martin@kaiser.cx>
> ---
> v2:
>    rebased against latest linux-next
>    re-sending both patches, it seems they got lost along the way
> 
>  arch/arm/mach-imx/mach-imx25.c | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/arch/arm/mach-imx/mach-imx25.c b/arch/arm/mach-imx/mach-imx25.c
> index 32dcb5e..353b86e 100644
> --- a/arch/arm/mach-imx/mach-imx25.c
> +++ b/arch/arm/mach-imx/mach-imx25.c
> @@ -23,6 +23,11 @@ static void __init imx25_init_early(void)
>  	mxc_set_cpu_type(MXC_CPU_MX25);
>  }
>  
> +static void __init imx25_dt_init(void)
> +{
> +	imx_aips_allow_unprivileged_access("fsl,imx25-aips");
> +}
> +
>  static void __init mx25_init_irq(void)
>  {
>  	struct device_node *np;
> @@ -41,6 +46,7 @@ static const char * const imx25_dt_board_compat[] __initconst = {
>  
>  DT_MACHINE_START(IMX25_DT, "Freescale i.MX25 (Device Tree Support)")
>  	.init_early	= imx25_init_early,
> +	.init_machine	= imx25_dt_init,
>  	.init_late      = imx25_pm_init,
>  	.init_irq	= mx25_init_irq,
>  	.dt_compat	= imx25_dt_board_compat,
> -- 
> 2.1.4
> 
> 
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

  reply	other threads:[~2017-04-04 13:14 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-01-31 19:20 [PATCH 1/2] ARM: dts: i.MX25: add AIPS control registers Martin Kaiser
2017-01-31 19:20 ` Martin Kaiser
2017-01-31 19:20 ` Martin Kaiser
2017-01-31 19:20 ` [PATCH 2/2] ARM: i.MX25: globally disable supervisor protect Martin Kaiser
2017-01-31 19:20   ` Martin Kaiser
2017-02-01  7:09 ` [PATCH 1/2] ARM: dts: i.MX25: add AIPS control registers Sascha Hauer
2017-02-01  7:09   ` Sascha Hauer
2017-02-01  7:09   ` Sascha Hauer
2017-02-01 22:16 ` [PATCH v2] " Martin Kaiser
2017-02-01 22:16   ` Martin Kaiser
2017-02-01 22:16   ` Martin Kaiser
2017-02-02  7:09   ` Sascha Hauer
2017-02-02  7:09     ` Sascha Hauer
2017-02-02  7:09     ` Sascha Hauer
2017-02-03 12:59 ` [PATCH v3] " Martin Kaiser
2017-02-03 12:59   ` Martin Kaiser
2017-02-03 12:59   ` Martin Kaiser
2017-02-03 17:27   ` Sascha Hauer
2017-02-03 17:27     ` Sascha Hauer
2017-02-03 17:27     ` Sascha Hauer
2017-04-03 19:47 ` [PATCH 1/2 v4] " Martin Kaiser
2017-04-03 19:47   ` Martin Kaiser
2017-04-03 19:47   ` Martin Kaiser
2017-04-03 19:47   ` [PATCH 2/2 v2] ARM: i.MX25: globally disable supervisor protect Martin Kaiser
2017-04-03 19:47     ` Martin Kaiser
2017-04-04 13:14     ` Shawn Guo [this message]
2017-04-04 13:14       ` Shawn Guo
2017-04-04 13:14       ` Shawn Guo
2017-04-04 13:13   ` [PATCH 1/2 v4] ARM: dts: i.MX25: add AIPS control registers Shawn Guo
2017-04-04 13:13     ` Shawn Guo
2017-04-04 13:13     ` Shawn Guo

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=20170404131450.GD3394@dragon \
    --to=shawnguo@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    /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.