Message ID | 1352131185-12079-1-git-send-email-javier.martin@vista-silicon.com (mailing list archive) |
---|---|
State | Changes Requested, archived |
Headers |
Received: from mail.tu-berlin.de ([130.149.7.33]) by www.linuxtv.org with esmtp (Exim 4.72) (envelope-from <linux-media-owner@vger.kernel.org>) id 1TVP61-0004Nj-LU for patchwork@linuxtv.org; Mon, 05 Nov 2012 17:00:17 +0100 X-tubIT-Incoming-IP: 209.132.180.67 Received: from vger.kernel.org ([209.132.180.67]) by mail.tu-berlin.de (exim-4.75/mailfrontend-4) with esmtp for <patchwork@linuxtv.org> id 1TVP60-0007Uv-CU; Mon, 05 Nov 2012 17:00:17 +0100 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751569Ab2KEQAO (ORCPT <rfc822;patchwork@linuxtv.org>); Mon, 5 Nov 2012 11:00:14 -0500 Received: from mail-wg0-f44.google.com ([74.125.82.44]:35046 "EHLO mail-wg0-f44.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750785Ab2KEQAN (ORCPT <rfc822; linux-media@vger.kernel.org>); Mon, 5 Nov 2012 11:00:13 -0500 Received: by mail-wg0-f44.google.com with SMTP id dr13so4061968wgb.1 for <linux-media@vger.kernel.org>; Mon, 05 Nov 2012 08:00:12 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20120113; h=from:to:cc:subject:date:message-id:x-mailer:x-gm-message-state; bh=Ry6jxngf7/zVac5jg7hCXSMzYtlyOJ/HEwXLxkibRLg=; b=IXPkQiJi6qz1TVYov6SUat1oi/K1U0AHRTzKm+72OWd4Ptap/rH23rj74H+GjhTlPf GgT4XYd6dQQlhIyairdZD7OEy4e6qp9/MdLt6RXDUSZ5+gbRMiIZd6wKGo+zMZnFhBIa YzCn4AC6hDsiR/OSWDGXDqn3owkA3rpNRGkCbWpekrj4EWyA63gvmTvefjrg2h6R1mX0 B/T7OfStjOlwuhaUPcmObTPxiXnT3dc8EmcSAdaSfC13E5OEqowAoiQcBrIy2NDo2h7d MZFLzDm8exXIYuKoIEZaIql/sEeckLUErGH7a6YBR38j02kPk1NCCrMwiR9DOEFCbFQm yqbw== Received: by 10.180.104.97 with SMTP id gd1mr13955860wib.4.1352131211972; Mon, 05 Nov 2012 08:00:11 -0800 (PST) Received: from piscis.vsilicon.net (149.93.18.95.dynamic.jazztel.es. [95.18.93.149]) by mx.google.com with ESMTPS id j8sm10915175wiy.9.2012.11.05.08.00.09 (version=TLSv1/SSLv3 cipher=OTHER); Mon, 05 Nov 2012 08:00:11 -0800 (PST) From: Javier Martin <javier.martin@vista-silicon.com> To: linux-media@vger.kernel.org Cc: linux-arm-kernel@lists.infradead.org, p.zabel@pengutronix.de, s.nawrocki@samsung.com, mchehab@infradead.org, kernel@pengutronix.de, Javier Martin <javier.martin@vista-silicon.com> Subject: [PATCH 1/2] ARM: i.MX27: Add platform support for IRAM. Date: Mon, 5 Nov 2012 16:59:44 +0100 Message-Id: <1352131185-12079-1-git-send-email-javier.martin@vista-silicon.com> X-Mailer: git-send-email 1.7.9.5 X-Gm-Message-State: ALoCoQljyqAdPcfJnSQWaaJp0d6dY+IYeBPOX62/LvqRm4htPxAFm9ZsovFxy3Lbbcmwz2NroxAU Sender: linux-media-owner@vger.kernel.org Precedence: bulk List-ID: <linux-media.vger.kernel.org> X-Mailing-List: linux-media@vger.kernel.org X-PMX-Version: 5.6.1.2065439, Antispam-Engine: 2.7.2.376379, Antispam-Data: 2012.11.5.155118 X-PMX-Spam: Gauge=IIIIIIII, Probability=8%, Report=' MULTIPLE_RCPTS 0.1, HTML_00_01 0.05, HTML_00_10 0.05, BODYTEXTP_SIZE_3000_LESS 0, BODY_SIZE_1100_1199 0, BODY_SIZE_2000_LESS 0, BODY_SIZE_5000_LESS 0, BODY_SIZE_7000_LESS 0, URI_ENDS_IN_HTML 0, __ANY_URI 0, __CP_URI_IN_BODY 0, __HAS_FROM 0, __HAS_MSGID 0, __HAS_X_MAILER 0, __HAS_X_MAILING_LIST 0, __MIME_TEXT_ONLY 0, __MULTIPLE_RCPTS_CC_X2 0, __SANE_MSGID 0, __TO_MALFORMED_2 0, __TO_NO_NAME 0, __URI_NO_WWW 0, __URI_NS , __YOUTUBE_RCVD 0' |
Commit Message
Javier Martin
Nov. 5, 2012, 3:59 p.m. UTC
Add support for IRAM to i.MX27 non-DT platforms using
iram_init() function.
Signed-off-by: Javier Martin <javier.martin@vista-silicon.com>
---
arch/arm/mach-imx/mm-imx27.c | 3 +++
1 file changed, 3 insertions(+)
Comments
Hi Javier On Mon, 5 Nov 2012, Javier Martin wrote: > Add support for IRAM to i.MX27 non-DT platforms using > iram_init() function. I'm not sure this belongs in a camera driver. Can IRAM not be used for anything else? I'll check the i.MX27 datasheet when I'm back home after the conference, so far this seems a bit odd. Thanks Guennadi > > Signed-off-by: Javier Martin <javier.martin@vista-silicon.com> > --- > arch/arm/mach-imx/mm-imx27.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/arch/arm/mach-imx/mm-imx27.c b/arch/arm/mach-imx/mm-imx27.c > index e7e24af..fd2416d 100644 > --- a/arch/arm/mach-imx/mm-imx27.c > +++ b/arch/arm/mach-imx/mm-imx27.c > @@ -27,6 +27,7 @@ > #include <asm/pgtable.h> > #include <asm/mach/map.h> > #include <mach/iomux-v1.h> > +#include <mach/iram.h> > > /* MX27 memory map definition */ > static struct map_desc imx27_io_desc[] __initdata = { > @@ -94,4 +95,6 @@ void __init imx27_soc_init(void) > /* imx27 has the imx21 type audmux */ > platform_device_register_simple("imx21-audmux", 0, imx27_audmux_res, > ARRAY_SIZE(imx27_audmux_res)); > + /* imx27 has an iram of 46080 bytes size */ > + iram_init(MX27_IRAM_BASE_ADDR, 46080); > } > -- > 1.7.9.5 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-media" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > --- Guennadi Liakhovetski, Ph.D. Freelance Open-Source Software Developer http://www.open-technology.de/ -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Tue, Nov 06, 2012 at 12:37:35PM +0100, Guennadi Liakhovetski wrote: > Hi Javier > > On Mon, 5 Nov 2012, Javier Martin wrote: > > > Add support for IRAM to i.MX27 non-DT platforms using > > iram_init() function. > > I'm not sure this belongs in a camera driver. Can IRAM not be used for > anything else? I'll check the i.MX27 datasheet when I'm back home after > the conference, so far this seems a bit odd. This patch just adds the sram pool to the system in i.MX27 code, the patch is not camera specific. Sascha
On Mon, Nov 05, 2012 at 04:59:44PM +0100, Javier Martin wrote: > Add support for IRAM to i.MX27 non-DT platforms using > iram_init() function. > > Signed-off-by: Javier Martin <javier.martin@vista-silicon.com> > --- > arch/arm/mach-imx/mm-imx27.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/arch/arm/mach-imx/mm-imx27.c b/arch/arm/mach-imx/mm-imx27.c > index e7e24af..fd2416d 100644 > --- a/arch/arm/mach-imx/mm-imx27.c > +++ b/arch/arm/mach-imx/mm-imx27.c > @@ -27,6 +27,7 @@ > #include <asm/pgtable.h> > #include <asm/mach/map.h> > #include <mach/iomux-v1.h> > +#include <mach/iram.h> > > /* MX27 memory map definition */ > static struct map_desc imx27_io_desc[] __initdata = { > @@ -94,4 +95,6 @@ void __init imx27_soc_init(void) > /* imx27 has the imx21 type audmux */ > platform_device_register_simple("imx21-audmux", 0, imx27_audmux_res, > ARRAY_SIZE(imx27_audmux_res)); > + /* imx27 has an iram of 46080 bytes size */ > + iram_init(MX27_IRAM_BASE_ADDR, 46080); For this rather Philipps sram allocater patches should be used. This would also solve the problem that mach/iram.h cannot be accessed anymore in current -next. Fabio already sent a patch addressing this, but I think we should go for a proper fix rather than just moving iram.h to include/linux/ Sascha
On 16 November 2012 13:48, Sascha Hauer <s.hauer@pengutronix.de> wrote: > On Mon, Nov 05, 2012 at 04:59:44PM +0100, Javier Martin wrote: >> Add support for IRAM to i.MX27 non-DT platforms using >> iram_init() function. >> >> Signed-off-by: Javier Martin <javier.martin@vista-silicon.com> >> --- >> arch/arm/mach-imx/mm-imx27.c | 3 +++ >> 1 file changed, 3 insertions(+) >> >> diff --git a/arch/arm/mach-imx/mm-imx27.c b/arch/arm/mach-imx/mm-imx27.c >> index e7e24af..fd2416d 100644 >> --- a/arch/arm/mach-imx/mm-imx27.c >> +++ b/arch/arm/mach-imx/mm-imx27.c >> @@ -27,6 +27,7 @@ >> #include <asm/pgtable.h> >> #include <asm/mach/map.h> >> #include <mach/iomux-v1.h> >> +#include <mach/iram.h> >> >> /* MX27 memory map definition */ >> static struct map_desc imx27_io_desc[] __initdata = { >> @@ -94,4 +95,6 @@ void __init imx27_soc_init(void) >> /* imx27 has the imx21 type audmux */ >> platform_device_register_simple("imx21-audmux", 0, imx27_audmux_res, >> ARRAY_SIZE(imx27_audmux_res)); >> + /* imx27 has an iram of 46080 bytes size */ >> + iram_init(MX27_IRAM_BASE_ADDR, 46080); > > For this rather Philipps sram allocater patches should be used. This > would also solve the problem that mach/iram.h cannot be accessed anymore > in current -next. Fabio already sent a patch addressing this, but I > think we should go for a proper fix rather than just moving iram.h > to include/linux/ Fine, I'll take a look at Philipps' patches. Regards.
diff --git a/arch/arm/mach-imx/mm-imx27.c b/arch/arm/mach-imx/mm-imx27.c index e7e24af..fd2416d 100644 --- a/arch/arm/mach-imx/mm-imx27.c +++ b/arch/arm/mach-imx/mm-imx27.c @@ -27,6 +27,7 @@ #include <asm/pgtable.h> #include <asm/mach/map.h> #include <mach/iomux-v1.h> +#include <mach/iram.h> /* MX27 memory map definition */ static struct map_desc imx27_io_desc[] __initdata = { @@ -94,4 +95,6 @@ void __init imx27_soc_init(void) /* imx27 has the imx21 type audmux */ platform_device_register_simple("imx21-audmux", 0, imx27_audmux_res, ARRAY_SIZE(imx27_audmux_res)); + /* imx27 has an iram of 46080 bytes size */ + iram_init(MX27_IRAM_BASE_ADDR, 46080); }