From: Tony Lindgren <tony@atomide.com>
To: Hiroshi DOYU <Hiroshi.DOYU@nokia.com>
Cc: linux-omap@vger.kernel.org
Subject: Re: [PATCH 02/10] omap mailbox: add initial omap3 support
Date: Wed, 26 Nov 2008 10:36:14 -0800 [thread overview]
Message-ID: <20081126183612.GV11997@atomide.com> (raw)
In-Reply-To: <20081126.021040.127479183.Hiroshi.DOYU@nokia.com>
* Hiroshi DOYU <Hiroshi.DOYU@nokia.com> [081125 16:10]:
> Hi Tony,
>
> From: "ext Tony Lindgren" <tony@atomide.com>
> Subject: Re: [PATCH 02/10] omap mailbox: add initial omap3 support
> Date: Tue, 25 Nov 2008 13:54:14 -0800
>
> > * Hiroshi DOYU <Hiroshi.DOYU@nokia.com> [081125 01:40]:
> > > Signed-off-by: Hiroshi DOYU <Hiroshi.DOYU@nokia.com>
> > > ---
> > > arch/arm/mach-omap2/devices.c | 24 +++++++++++---
> > > arch/arm/mach-omap2/mailbox.c | 46 ++++++++++++++++++----------
> > > arch/arm/plat-omap/Kconfig | 2 +-
> > > arch/arm/plat-omap/include/mach/irqs.h | 1 +
> > > arch/arm/plat-omap/include/mach/omap34xx.h | 2 +-
> > > 5 files changed, 52 insertions(+), 23 deletions(-)
> > >
> > > diff --git a/arch/arm/mach-omap2/devices.c b/arch/arm/mach-omap2/devices.c
> > > index 241e418..d385f0f 100644
> > > --- a/arch/arm/mach-omap2/devices.c
> > > +++ b/arch/arm/mach-omap2/devices.c
> > > @@ -84,13 +84,15 @@ static inline void omap_init_camera(void)
> > > }
> > > #endif
> > >
> > > -#if defined(CONFIG_OMAP_DSP) || defined(CONFIG_OMAP_DSP_MODULE)
> > > -#define OMAP2_MBOX_BASE IO_ADDRESS(OMAP24XX_MAILBOX_BASE)
> > > +#if defined(CONFIG_OMAP_MBOX_FWK) || defined(CONFIG_OMAP_MBOX_FWK_MODULE)
> > > +
> > > +#define MBOX_REG_SIZE 0x120
> > >
> > > static struct resource mbox_resources[] = {
> > > +#if defined(CONFIG_ARCH_OMAP2420)
> > > {
> > > - .start = OMAP2_MBOX_BASE,
> > > - .end = OMAP2_MBOX_BASE + 0x11f,
> > > + .start = OMAP24XX_MAILBOX_BASE,
> > > + .end = OMAP24XX_MAILBOX_BASE + MBOX_REG_SIZE - 1,
> > > .flags = IORESOURCE_MEM,
> > > },
> > > {> @@ -101,6 +103,18 @@ static struct resource mbox_resources[] = {
> > > .start = INT_24XX_MAIL_U3_MPU,
> > > .flags = IORESOURCE_IRQ,
> > > },
> > > +/* FIXME: if multiple architecture support is necessary */
> > > +#elif defined(CONFIG_ARCH_OMAP3)
> > > + {
> > > + .start = OMAP34XX_MAILBOX_BASE,
> > > + .end = OMAP34XX_MAILBOX_BASE + MBOX_REG_SIZE - 1,
> > > + .flags = IORESOURCE_MEM,
> > > + },
> > > + {
> > > + .start = INT_34XX_MAIL_U0_MPU,
> > > + .flags = IORESOURCE_IRQ,
> > > + },
> > > +#endif
> > > };
> > >
> > > static struct platform_device mbox_device = {
> >
> > How about setting up omap2_mbox_resources[] and omap3_mbox_resources[]
> > above instead? Then just select the right one to use during init.
> >
> > We should not need to use ifdefs in any of the code for detecting the
> > omap cpu type, please use cpu_is_omap24xx() and cpu_is_omap34xx()
> > instead.
>
> Agreed and updated as below.
Thanks, since this is a large series, can you please send your series for
RMK to integrate to LAKML? Make sure it also applies against mainline
first :) Please feel free to add my Ack to your patches.
> Also, the latest patchset is available in the git repository at:
>
> http://git.gitorious.org/lk/mainline.git mailbox
Thanks, I've added mirroring for this branch to my kernel.org tree.
Let's wait for ack from RMK before integrating it to l-o.
BTW, we should move dspgateway also into a separate branch like
tidspbridge as it's not in the mainline tree. Do you want to keep the
original for that on your git server too?
Regards,
Tony
> From 7b62c1b04ae50adf2827fb3ca1ec8a3e72349d14 Mon Sep 17 00:00:00 2001
> From: Hiroshi DOYU <Hiroshi.DOYU@nokia.com>
> Date: Sat, 22 Nov 2008 02:28:36 +0200
> Subject: [PATCH 02/10] omap mailbox: add initial omap3 support
>
> Signed-off-by: Hiroshi DOYU <Hiroshi.DOYU@nokia.com>
> ---
> arch/arm/mach-omap2/devices.c | 36 +++++++++++++++++-----
> arch/arm/mach-omap2/mailbox.c | 46 ++++++++++++++++++----------
> arch/arm/plat-omap/Kconfig | 2 +-
> arch/arm/plat-omap/include/mach/irqs.h | 1 +
> arch/arm/plat-omap/include/mach/omap34xx.h | 2 +-
> 5 files changed, 61 insertions(+), 26 deletions(-)
>
> diff --git a/arch/arm/mach-omap2/devices.c b/arch/arm/mach-omap2/devices.c
> index 241e418..ea37f37 100644
> --- a/arch/arm/mach-omap2/devices.c
> +++ b/arch/arm/mach-omap2/devices.c
> @@ -84,13 +84,14 @@ static inline void omap_init_camera(void)
> }
> #endif
>
> -#if defined(CONFIG_OMAP_DSP) || defined(CONFIG_OMAP_DSP_MODULE)
> -#define OMAP2_MBOX_BASE IO_ADDRESS(OMAP24XX_MAILBOX_BASE)
> +#if defined(CONFIG_OMAP_MBOX_FWK) || defined(CONFIG_OMAP_MBOX_FWK_MODULE)
>
> -static struct resource mbox_resources[] = {
> +#define MBOX_REG_SIZE 0x120
> +
> +static struct resource omap2_mbox_resources[] = {
> {
> - .start = OMAP2_MBOX_BASE,
> - .end = OMAP2_MBOX_BASE + 0x11f,
> + .start = OMAP24XX_MAILBOX_BASE,
> + .end = OMAP24XX_MAILBOX_BASE + MBOX_REG_SIZE - 1,
> .flags = IORESOURCE_MEM,
> },
> {
> @@ -103,20 +104,39 @@ static struct resource mbox_resources[] = {
> },
> };
>
> +static struct resource omap3_mbox_resources[] = {
> + {
> + .start = OMAP34XX_MAILBOX_BASE,
> + .end = OMAP34XX_MAILBOX_BASE + MBOX_REG_SIZE - 1,
> + .flags = IORESOURCE_MEM,
> + },
> + {
> + .start = INT_34XX_MAIL_U0_MPU,
> + .flags = IORESOURCE_IRQ,
> + },
> +};
> +
> static struct platform_device mbox_device = {
> .name = "mailbox",
> .id = -1,
> - .num_resources = ARRAY_SIZE(mbox_resources),
> - .resource = mbox_resources,
> };
>
> static inline void omap_init_mbox(void)
> {
> + if (cpu_is_omap2420()) {
> + mbox_device.num_resources = ARRAY_SIZE(omap2_mbox_resources);
> + mbox_device.resource = omap2_mbox_resources;
> + } else if (cpu_is_omap3430()) {
> + mbox_device.num_resources = ARRAY_SIZE(omap3_mbox_resources);
> + mbox_device.resource = omap3_mbox_resources;
> + } else {
> + return;
> + }
> platform_device_register(&mbox_device);
> }
> #else
> static inline void omap_init_mbox(void) { }
> -#endif
> +#endif /* CONFIG_OMAP_MBOX_FWK */
>
> #if defined(CONFIG_OMAP_STI)
>
> diff --git a/arch/arm/mach-omap2/mailbox.c b/arch/arm/mach-omap2/mailbox.c
> index 5ff2ed8..261cd79 100644
> --- a/arch/arm/mach-omap2/mailbox.c
> +++ b/arch/arm/mach-omap2/mailbox.c
> @@ -30,7 +30,7 @@
> #define MAILBOX_IRQ_NEWMSG(u) (1 << (2 * (u)))
> #define MAILBOX_IRQ_NOTFULL(u) (1 << (2 * (u) + 1))
>
> -static unsigned long mbox_base;
> +static void __iomem *mbox_base;
>
> struct omap_mbox2_fifo {
> unsigned long msg;
> @@ -52,14 +52,14 @@ static struct clk *mbox_ick_handle;
> static void omap2_mbox_enable_irq(struct omap_mbox *mbox,
> omap_mbox_type_t irq);
>
> -static inline unsigned int mbox_read_reg(unsigned int reg)
> +static inline unsigned int mbox_read_reg(size_t ofs)
> {
> - return __raw_readl((void __iomem *)(mbox_base + reg));
> + return __raw_readl(mbox_base + ofs);
> }
>
> -static inline void mbox_write_reg(unsigned int val, unsigned int reg)
> +static inline void mbox_write_reg(u32 val, size_t ofs)
> {
> - __raw_writel(val, (void __iomem *)(mbox_base + reg));
> + __raw_writel(val, mbox_base + ofs);
> }
>
> /* Mailbox H/W preparations */
> @@ -208,7 +208,7 @@ struct omap_mbox mbox_dsp_info = {
> };
> EXPORT_SYMBOL(mbox_dsp_info);
>
> -/* IVA */
> +#if defined(CONFIG_ARCH_OMAP2420) /* IVA */
> static struct omap_mbox2_priv omap2_mbox_iva_priv = {
> .tx_fifo = {
> .msg = MAILBOX_MESSAGE(2),
> @@ -229,52 +229,66 @@ static struct omap_mbox mbox_iva_info = {
> .ops = &omap2_mbox_ops,
> .priv = &omap2_mbox_iva_priv,
> };
> +#endif
>
> static int __init omap2_mbox_probe(struct platform_device *pdev)
> {
> struct resource *res;
> int ret = 0;
>
> - if (pdev->num_resources != 3) {
> - dev_err(&pdev->dev, "invalid number of resources: %d\n",
> - pdev->num_resources);
> - return -ENODEV;
> - }
> -
> /* MBOX base */
> res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
> if (unlikely(!res)) {
> dev_err(&pdev->dev, "invalid mem resource\n");
> return -ENODEV;
> }
> - mbox_base = res->start;
> + mbox_base = ioremap(res->start, res->end - res->start);
> + if (!mbox_base)
> + return -ENOMEM;
>
> - /* DSP IRQ */
> + /* DSP or IVA2 IRQ */
> res = platform_get_resource(pdev, IORESOURCE_IRQ, 0);
> if (unlikely(!res)) {
> dev_err(&pdev->dev, "invalid irq resource\n");
> - return -ENODEV;
> + ret = -ENODEV;
> + goto err_dsp;
> }
> mbox_dsp_info.irq = res->start;
>
> ret = omap_mbox_register(&mbox_dsp_info);
> + if (!ret)
> + goto err_dsp;
>
> +#if defined(CONFIG_ARCH_OMAP2420)
> /* IVA IRQ */
> res = platform_get_resource(pdev, IORESOURCE_IRQ, 1);
> if (unlikely(!res)) {
> dev_err(&pdev->dev, "invalid irq resource\n");
> - return -ENODEV;
> + ret = -ENODEV;
> + goto err_iva1;
> }
> mbox_iva_info.irq = res->start;
>
> ret = omap_mbox_register(&mbox_iva_info);
> + if (!ret)
> + goto err_iva1;
> +#endif
> + return ret;
>
> +err_iva1:
> + omap_mbox_unregister(&mbox_dsp_info);
> +err_dsp:
> + iounmap(mbox_base);
> return ret;
> }
>
> static int omap2_mbox_remove(struct platform_device *pdev)
> {
> +#if defined(CONFIG_ARCH_OMAP2420)
> + omap_mbox_unregister(&mbox_iva_info);
> +#endif
> omap_mbox_unregister(&mbox_dsp_info);
> + iounmap(mbox_base);
> return 0;
> }
>
> diff --git a/arch/arm/plat-omap/Kconfig b/arch/arm/plat-omap/Kconfig
> index 960c13f..2465aea 100644
> --- a/arch/arm/plat-omap/Kconfig
> +++ b/arch/arm/plat-omap/Kconfig
> @@ -180,7 +180,7 @@ config OMAP_MBOX_FWK
> default n
> help
> Say Y here if you want to use OMAP Mailbox framework support for
> - DSP and IVA1.0 in OMAP1/2.
> + DSP, IVA1.0 and IVA2 in OMAP1/2/3.
>
> choice
> prompt "System timer"
> diff --git a/arch/arm/plat-omap/include/mach/irqs.h b/arch/arm/plat-omap/include/mach/irqs.h
> index d12c39f..74f58c6 100644
> --- a/arch/arm/plat-omap/include/mach/irqs.h
> +++ b/arch/arm/plat-omap/include/mach/irqs.h
> @@ -315,6 +315,7 @@
> #define INT_34XX_MCBSP3_IRQ 22
> #define INT_34XX_MCBSP4_IRQ 23
> #define INT_34XX_CAM_IRQ 24
> +#define INT_34XX_MAIL_U0_MPU 26
> #define INT_34XX_MCBSP5_IRQ 27
> #define INT_34XX_GPIO_BANK1 29
> #define INT_34XX_GPIO_BANK2 30
> diff --git a/arch/arm/plat-omap/include/mach/omap34xx.h b/arch/arm/plat-omap/include/mach/omap34xx.h
> index 0a137c1..afd0f6c 100644
> --- a/arch/arm/plat-omap/include/mach/omap34xx.h
> +++ b/arch/arm/plat-omap/include/mach/omap34xx.h
> @@ -63,8 +63,8 @@
> #define OMAP2_CM_BASE OMAP3430_CM_BASE
> #define OMAP2_PRM_BASE OMAP3430_PRM_BASE
> #define OMAP2_VA_IC_BASE IO_ADDRESS(OMAP34XX_IC_BASE)
> +#define OMAP34XX_MAILBOX_BASE (L4_34XX_BASE + 0x94000)
> #define OMAP34XX_CAMERA_BASE (L4_34XX_BASE + 0xBC000)
> -
> #endif
>
> #define OMAP34XX_DSP_BASE 0x58000000
> --
> 1.6.0.3.613.g9f8f13
>
next prev parent reply other threads:[~2008-11-26 18:36 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-11-25 9:39 [PATCH 0/10] omap mailbox driver update with omap3 support Hiroshi DOYU
2008-11-25 9:39 ` [PATCH 01/10] omap mailbox: cleanup omap2 register definition with macro Hiroshi DOYU
2008-11-25 9:39 ` [PATCH 02/10] omap mailbox: add initial omap3 support Hiroshi DOYU
2008-11-25 9:39 ` [PATCH 03/10] omap mailbox: print hardware revision at startup Hiroshi DOYU
2008-11-25 9:39 ` [PATCH 04/10] omap mailbox: fix empty struct device for omap_mbox Hiroshi DOYU
2008-11-25 9:39 ` [PATCH 05/10] omap mailbox: fix empty struct device for omap1 Hiroshi DOYU
2008-11-25 9:39 ` [PATCH 06/10] omap mailbox: fix empty struct device for omap2 Hiroshi DOYU
2008-11-25 9:39 ` [PATCH 07/10] omap mailbox: add save_/restore_ctx() for PM Hiroshi DOYU
2008-11-25 9:39 ` [PATCH 08/10] omap mailbox: move mailbox.h into mailbox.c Hiroshi DOYU
2008-11-25 9:39 ` [PATCH 09/10] omap mailbox: convert sequence bit checking to module paramter Hiroshi DOYU
2008-11-25 9:39 ` [PATCH 10/10] omap mailbox: remove unnecessary header file inclusion Hiroshi DOYU
2008-11-25 9:44 ` [PATCH 05/10] omap mailbox: fix empty struct device for omap1 Felipe Balbi
2008-11-25 11:13 ` Hiroshi DOYU
2008-11-25 21:54 ` [PATCH 02/10] omap mailbox: add initial omap3 support Tony Lindgren
2008-11-26 0:10 ` Hiroshi DOYU
2008-11-26 18:36 ` Tony Lindgren [this message]
2008-11-27 14:39 ` Hiroshi DOYU
2009-01-21 20:02 ` Kanigeri, Hari
2009-01-21 22:45 ` Hiroshi DOYU
2009-01-22 5:12 ` Ramesh Gupta Guntha
2009-01-22 6:10 ` Hiroshi DOYU
-- strict thread matches above, loose matches on Subject: below --
2009-01-16 8:27 [PATCH 00/10] arm: omap mailbox: add omap3 support with some fixes Hiroshi DOYU
2009-01-16 8:27 ` [PATCH 02/10] omap mailbox: add initial omap3 support Hiroshi DOYU
2009-01-16 15:43 ` Felipe Balbi
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=20081126183612.GV11997@atomide.com \
--to=tony@atomide.com \
--cc=Hiroshi.DOYU@nokia.com \
--cc=linux-omap@vger.kernel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox