From: Jason Chen <weitway@gmail.com>
To: Sascha Hauer <s.hauer@pengutronix.de>
Cc: linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org, linux-fbdev@vger.kernel.org,
Paul Mundt <lethal@linux-sh.org>,
Samuel Ortiz <sameo@linux.intel.com>
Subject: Re: [PATCH 1/7] Add a mfd IPUv3 driver
Date: Mon, 21 Feb 2011 13:38:32 +0800 [thread overview]
Message-ID: <AANLkTinQrXUwyRjehLr4YFiR66CRxT8ocEQMLXOEzHJx@mail.gmail.com> (raw)
In-Reply-To: <1297865452-32181-2-git-send-email-s.hauer@pengutronix.de>
hi, Sascha,
I have few comments for this patch.
> +#define MX51_IPU_CHANNEL_CSI0 0
> +#define MX51_IPU_CHANNEL_CSI1 1
> +#define MX51_IPU_CHANNEL_CSI2 2
> +#define MX51_IPU_CHANNEL_CSI3 3
> +#define MX51_IPU_CHANNEL_MEM_BG_SYNC 23
> +#define MX51_IPU_CHANNEL_MEM_FG_SYNC 27
> +#define MX51_IPU_CHANNEL_MEM_DC_SYNC 28
> +#define MX51_IPU_CHANNEL_MEM_FG_SYNC_ALPHA 31
> +#define MX51_IPU_CHANNEL_MEM_DC_ASYNC 41
> +#define MX51_IPU_CHANNEL_ROT_ENC_MEM 45
> +#define MX51_IPU_CHANNEL_ROT_VF_MEM 46
> +#define MX51_IPU_CHANNEL_ROT_PP_MEM 47
> +#define MX51_IPU_CHANNEL_ROT_ENC_MEM_OUT 48
> +#define MX51_IPU_CHANNEL_ROT_VF_MEM_OUT 49
> +#define MX51_IPU_CHANNEL_ROT_PP_MEM_OUT 50
> +#define MX51_IPU_CHANNEL_MEM_BG_SYNC_ALPHA 51
Had better use MX5_IPU as prefix because it's same value in MX5X.
> +
> + ipu_idmac_reg = ioremap(ipu_base + IPU_IDMAC_REG_BASE, PAGE_SIZE);
> + if (!ipu_idmac_reg) {
> + ret = -ENOMEM;
> + goto failed_ioremap2;
> + }
> +
> + ret = ipu_mipi_setup();
> + if (ret)
> + goto failed_mipi_setup;
I dont know what's the best way to do it, but I think you had better
consider mx53 platform, so maybe this mipi_setup function and also
below ipu hw reset could be set in platform data.
> +
> + ipu_clk = clk_get(&pdev->dev, "ipu");
> + if (IS_ERR(ipu_clk)) {
> + ret = PTR_ERR(ipu_clk);
> + dev_err(&pdev->dev, "clk_get failed with %d", ret);
> + goto failed_clk_get;
> + }
> +
> + ipu_get();
> +
> + ret = request_irq(irq1, ipu_irq_handler, IRQF_DISABLED, pdev->name,
> + &pdev->dev);
> + if (ret) {
> + dev_err(&pdev->dev, "request irq %d failed with: %d\n", irq1, ret);
> + goto failed_request_irq1;
> + }
> +
> + ret = request_irq(irq2, ipu_irq_handler, IRQF_DISABLED, pdev->name,
> + &pdev->dev);
> + if (ret) {
> + dev_err(&pdev->dev, "request irq %d failed with: %d\n", irq2, ret);
> + goto failed_request_irq2;
> + }
> +
> + ipu_reset();
WARNING: multiple messages have this Message-ID (diff)
From: Jason Chen <weitway@gmail.com>
To: linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 1/7] Add a mfd IPUv3 driver
Date: Mon, 21 Feb 2011 05:38:32 +0000 [thread overview]
Message-ID: <AANLkTinQrXUwyRjehLr4YFiR66CRxT8ocEQMLXOEzHJx@mail.gmail.com> (raw)
In-Reply-To: <1297865452-32181-2-git-send-email-s.hauer@pengutronix.de>
hi, Sascha,
I have few comments for this patch.
> +#define MX51_IPU_CHANNEL_CSI0 0
> +#define MX51_IPU_CHANNEL_CSI1 1
> +#define MX51_IPU_CHANNEL_CSI2 2
> +#define MX51_IPU_CHANNEL_CSI3 3
> +#define MX51_IPU_CHANNEL_MEM_BG_SYNC 23
> +#define MX51_IPU_CHANNEL_MEM_FG_SYNC 27
> +#define MX51_IPU_CHANNEL_MEM_DC_SYNC 28
> +#define MX51_IPU_CHANNEL_MEM_FG_SYNC_ALPHA 31
> +#define MX51_IPU_CHANNEL_MEM_DC_ASYNC 41
> +#define MX51_IPU_CHANNEL_ROT_ENC_MEM 45
> +#define MX51_IPU_CHANNEL_ROT_VF_MEM 46
> +#define MX51_IPU_CHANNEL_ROT_PP_MEM 47
> +#define MX51_IPU_CHANNEL_ROT_ENC_MEM_OUT 48
> +#define MX51_IPU_CHANNEL_ROT_VF_MEM_OUT 49
> +#define MX51_IPU_CHANNEL_ROT_PP_MEM_OUT 50
> +#define MX51_IPU_CHANNEL_MEM_BG_SYNC_ALPHA 51
Had better use MX5_IPU as prefix because it's same value in MX5X.
> +
> + ipu_idmac_reg = ioremap(ipu_base + IPU_IDMAC_REG_BASE, PAGE_SIZE);
> + if (!ipu_idmac_reg) {
> + ret = -ENOMEM;
> + goto failed_ioremap2;
> + }
> +
> + ret = ipu_mipi_setup();
> + if (ret)
> + goto failed_mipi_setup;
I dont know what's the best way to do it, but I think you had better
consider mx53 platform, so maybe this mipi_setup function and also
below ipu hw reset could be set in platform data.
> +
> + ipu_clk = clk_get(&pdev->dev, "ipu");
> + if (IS_ERR(ipu_clk)) {
> + ret = PTR_ERR(ipu_clk);
> + dev_err(&pdev->dev, "clk_get failed with %d", ret);
> + goto failed_clk_get;
> + }
> +
> + ipu_get();
> +
> + ret = request_irq(irq1, ipu_irq_handler, IRQF_DISABLED, pdev->name,
> + &pdev->dev);
> + if (ret) {
> + dev_err(&pdev->dev, "request irq %d failed with: %d\n", irq1, ret);
> + goto failed_request_irq1;
> + }
> +
> + ret = request_irq(irq2, ipu_irq_handler, IRQF_DISABLED, pdev->name,
> + &pdev->dev);
> + if (ret) {
> + dev_err(&pdev->dev, "request irq %d failed with: %d\n", irq2, ret);
> + goto failed_request_irq2;
> + }
> +
> + ipu_reset();
WARNING: multiple messages have this Message-ID (diff)
From: weitway@gmail.com (Jason Chen)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 1/7] Add a mfd IPUv3 driver
Date: Mon, 21 Feb 2011 13:38:32 +0800 [thread overview]
Message-ID: <AANLkTinQrXUwyRjehLr4YFiR66CRxT8ocEQMLXOEzHJx@mail.gmail.com> (raw)
In-Reply-To: <1297865452-32181-2-git-send-email-s.hauer@pengutronix.de>
hi, Sascha,
I have few comments for this patch.
> +#define MX51_IPU_CHANNEL_CSI0 ? ? ? ? ? ? ? ? ? 0
> +#define MX51_IPU_CHANNEL_CSI1 ? ? ? ? ? ? ? ? ? 1
> +#define MX51_IPU_CHANNEL_CSI2 ? ? ? ? ? ? ? ? ? 2
> +#define MX51_IPU_CHANNEL_CSI3 ? ? ? ? ? ? ? ? ? 3
> +#define MX51_IPU_CHANNEL_MEM_BG_SYNC ? ? ? ? ? 23
> +#define MX51_IPU_CHANNEL_MEM_FG_SYNC ? ? ? ? ? 27
> +#define MX51_IPU_CHANNEL_MEM_DC_SYNC ? ? ? ? ? 28
> +#define MX51_IPU_CHANNEL_MEM_FG_SYNC_ALPHA ? ? 31
> +#define MX51_IPU_CHANNEL_MEM_DC_ASYNC ? ? ? ? ?41
> +#define MX51_IPU_CHANNEL_ROT_ENC_MEM ? ? ? ? ? 45
> +#define MX51_IPU_CHANNEL_ROT_VF_MEM ? ? ? ? ? ?46
> +#define MX51_IPU_CHANNEL_ROT_PP_MEM ? ? ? ? ? ?47
> +#define MX51_IPU_CHANNEL_ROT_ENC_MEM_OUT ? ? ? 48
> +#define MX51_IPU_CHANNEL_ROT_VF_MEM_OUT ? ? ? ? ? ? ? ?49
> +#define MX51_IPU_CHANNEL_ROT_PP_MEM_OUT ? ? ? ? ? ? ? ?50
> +#define MX51_IPU_CHANNEL_MEM_BG_SYNC_ALPHA ? ? 51
Had better use MX5_IPU as prefix because it's same value in MX5X.
> +
> + ? ? ? ipu_idmac_reg = ioremap(ipu_base + IPU_IDMAC_REG_BASE, PAGE_SIZE);
> + ? ? ? if (!ipu_idmac_reg) {
> + ? ? ? ? ? ? ? ret = -ENOMEM;
> + ? ? ? ? ? ? ? goto failed_ioremap2;
> + ? ? ? }
> +
> + ? ? ? ret = ipu_mipi_setup();
> + ? ? ? if (ret)
> + ? ? ? ? ? ? ? goto failed_mipi_setup;
I dont know what's the best way to do it, but I think you had better
consider mx53 platform, so maybe this mipi_setup function and also
below ipu hw reset could be set in platform data.
> +
> + ? ? ? ipu_clk = clk_get(&pdev->dev, "ipu");
> + ? ? ? if (IS_ERR(ipu_clk)) {
> + ? ? ? ? ? ? ? ret = PTR_ERR(ipu_clk);
> + ? ? ? ? ? ? ? dev_err(&pdev->dev, "clk_get failed with %d", ret);
> + ? ? ? ? ? ? ? goto failed_clk_get;
> + ? ? ? }
> +
> + ? ? ? ipu_get();
> +
> + ? ? ? ret = request_irq(irq1, ipu_irq_handler, IRQF_DISABLED, pdev->name,
> + ? ? ? ? ? ? ? ? ? ? ? &pdev->dev);
> + ? ? ? if (ret) {
> + ? ? ? ? ? ? ? dev_err(&pdev->dev, "request irq %d failed with: %d\n", irq1, ret);
> + ? ? ? ? ? ? ? goto failed_request_irq1;
> + ? ? ? }
> +
> + ? ? ? ret = request_irq(irq2, ipu_irq_handler, IRQF_DISABLED, pdev->name,
> + ? ? ? ? ? ? ? ? ? ? ? &pdev->dev);
> + ? ? ? if (ret) {
> + ? ? ? ? ? ? ? dev_err(&pdev->dev, "request irq %d failed with: %d\n", irq2, ret);
> + ? ? ? ? ? ? ? goto failed_request_irq2;
> + ? ? ? }
> +
> + ? ? ? ipu_reset();
next prev parent reply other threads:[~2011-02-21 5:38 UTC|newest]
Thread overview: 42+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-02-16 14:10 [PATCH v3] Add i.MX51/53 IPU framebuffer support Sascha Hauer
2011-02-16 14:10 ` [PATCH 1/7] Add a mfd IPUv3 driver Sascha Hauer
2011-02-16 14:10 ` Sascha Hauer
2011-02-17 18:10 ` Arnaud Patard
2011-02-17 18:10 ` Arnaud Patard (Rtp)
2011-02-17 18:10 ` Arnaud Patard
2011-02-18 9:49 ` Sascha Hauer
2011-02-18 9:49 ` Sascha Hauer
2011-02-18 9:49 ` Sascha Hauer
2011-02-18 12:07 ` Samuel Ortiz
2011-02-18 12:07 ` Samuel Ortiz
2011-02-18 12:07 ` Samuel Ortiz
2011-02-21 5:38 ` Jason Chen [this message]
2011-02-21 5:38 ` Jason Chen
2011-02-21 5:38 ` Jason Chen
2011-02-16 14:10 ` [PATCH 2/7] fb: export fb mode db table Sascha Hauer
2011-02-16 14:10 ` Sascha Hauer
2011-02-16 14:10 ` Sascha Hauer
2011-02-16 14:10 ` [PATCH 3/7] Add i.MX5 framebuffer driver Sascha Hauer
2011-02-16 14:10 ` Sascha Hauer
2011-02-16 14:10 ` Sascha Hauer
2011-02-18 9:22 ` Jason Chen
2011-02-18 9:22 ` Jason Chen
2011-02-18 9:22 ` Jason Chen
2011-02-18 9:52 ` Sascha Hauer
2011-02-18 9:52 ` Sascha Hauer
2011-02-18 9:52 ` Sascha Hauer
2011-02-19 2:22 ` Jason Chen
2011-02-19 2:22 ` Jason Chen
2011-02-19 2:22 ` Jason Chen
2011-02-16 14:10 ` [PATCH 4/7] ARM i.MX51: Add IPU device support Sascha Hauer
2011-02-16 14:10 ` [PATCH 5/7] ARM i.MX5: Allow to increase max zone order Sascha Hauer
2011-02-16 14:10 ` [PATCH 6/7] ARM i.MX5: increase dma consistent size for IPU support Sascha Hauer
2011-02-16 14:10 ` [PATCH 7/7] ARM i.MX51 babbage: Add framebuffer support Sascha Hauer
2011-02-17 18:11 ` Arnaud Patard (Rtp)
2011-02-18 9:47 ` Sascha Hauer
2011-02-23 16:48 ` [PATCH v3] Add i.MX51/53 IPU " Eric Benard
2011-04-13 15:53 [PATCH 1/7] Add a mfd IPUv3 driver weitway at gmail.com
2011-04-14 9:08 ` Sascha Hauer
2011-04-14 2:04 jason.chen at freescale.com
2011-04-14 2:27 Chen Jie-B02280
2011-04-25 13:37 ` Jason Chen
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=AANLkTinQrXUwyRjehLr4YFiR66CRxT8ocEQMLXOEzHJx@mail.gmail.com \
--to=weitway@gmail.com \
--cc=lethal@linux-sh.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-fbdev@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=s.hauer@pengutronix.de \
--cc=sameo@linux.intel.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.