public inbox for linux-arm-kernel@lists.infradead.org 
 help / color / mirror / Atom feed
From: kgene.kim@samsung•com (Kukjin Kim)
To: linux-arm-kernel@lists•infradead.org
Subject: [PATCH 4/4] ARM: Exynos5: Add PDMA and MDMA physical base address defines
Date: Tue, 10 Apr 2012 09:56:29 -0700	[thread overview]
Message-ID: <4F84663D.5070501@samsung.com> (raw)
In-Reply-To: <CAOesGMisDq_Z3zNFNmOeyDPEj-VXDsy9a8ABhJie4fsjtRkeJw@mail.gmail.com>

Olof Johansson wrote:
> Hi,
>
> 2012/3/28 Kukjin Kim<kgene.kim@samsung•com>:
>> Thomas Abraham wrote:
>>> Add PDMA and MDMA physical base address macros which is require for Exynos5
>>> of_dev_auxdata setup.
>>>
>>> Signed-off-by: Thomas Abraham<thomas.ab@samsung•com>
>>> ---
>>>    arch/arm/mach-exynos/include/mach/map.h |    5 +++++
>>>    arch/arm/mach-exynos/mach-exynos5-dt.c  |    3 ++-
>>>    2 files changed, 7 insertions(+), 1 deletions(-)
>>>
>>> diff --git a/arch/arm/mach-exynos/include/mach/map.h b/arch/arm/mach-exynos/include/mach/map.h
>>> index 024d38f..37d93fd 100644
>>> --- a/arch/arm/mach-exynos/include/mach/map.h
>>> +++ b/arch/arm/mach-exynos/include/mach/map.h
>>> @@ -90,6 +90,11 @@
>>>    #define EXYNOS4_PA_PDMA0            0x12680000
>>>    #define EXYNOS4_PA_PDMA1            0x12690000
>>>
>>> +#define EXYNOS5_PA_PDMA0             0x121A0000
>>> +#define EXYNOS5_PA_PDMA1             0x121B0000
>>> +#define EXYNOS5_PA_MDMA0             0x10800000
>>> +#define EXYNOS5_PA_MDMA1             0x11C10000
>>> +
>>
>> See, "ARM: EXYNOS: Support DMA for Exynos5250 SoC"
>> http://lists.infradead.org/pipermail/linux-arm-kernel/2012-February/085434.html
>>
>> But this merge window is not including it :(
>
> Compilation of CONFIG_MACH_EXYNOS5_DT is broken without the above PDMA
> defines. Either add the defines, or remove the AUXDATA as part of the
> fixes for this release, please.
>
Yeah, right.

I fixed it with this but changed dma channel for mdma1.

@@ -45,7 +45,7 @@ static const struct of_dev_auxdata 
exynos5250_auxdata_lookup[]
                                 "exynos4210-uart.3", NULL),
         OF_DEV_AUXDATA("arm,pl330", EXYNOS5_PA_PDMA0, "dma-pl330.0", NULL),
         OF_DEV_AUXDATA("arm,pl330", EXYNOS5_PA_PDMA1, "dma-pl330.1", NULL),
-       OF_DEV_AUXDATA("arm,pl330", EXYNOS5_PA_PDMA1, "dma-pl330.2", NULL),
+       OF_DEV_AUXDATA("arm,pl330", EXYNOS5_PA_MDMA1, "dma-pl330.2", NULL),
         {},
  };


Thanks.

Best regards,
Kgene.
--
Kukjin Kim <kgene.kim@samsung•com>, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.

      reply	other threads:[~2012-04-10 16:56 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-03-27  9:26 [PATCH 0/4] ARM: Exynos: Fixes for Exynos5 device tree support Thomas Abraham
2012-03-27  9:26 ` [PATCH 1/4] ARM: Exynos: Use CHIP_ID to detect cpu type during decompression setup Thomas Abraham
2012-03-29  1:34   ` Kukjin Kim
2012-03-27  9:26 ` [PATCH 2/4] ARM: Exynos: Remove a new bus_type instance for Exynos5 Thomas Abraham
2012-03-27  9:26 ` [PATCH 3/4] ARM: Exynos5: Fix incorrect initialization of GIC Thomas Abraham
2012-03-29  1:46   ` Kukjin Kim
2012-03-27  9:26 ` [PATCH 4/4] ARM: Exynos5: Add PDMA and MDMA physical base address defines Thomas Abraham
2012-03-29  1:54   ` Kukjin Kim
2012-04-09  3:48     ` Olof Johansson
2012-04-10 16:56       ` Kukjin Kim [this message]

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=4F84663D.5070501@samsung.com \
    --to=kgene.kim@samsung$(echo .)com \
    --cc=linux-arm-kernel@lists$(echo .)infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html
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