* [PATCH] ARM: omap2: add select of TI_PRIV_EDMA
@ 2013-07-09 8:37 Olof Johansson
2013-07-10 7:50 ` Tony Lindgren
0 siblings, 1 reply; 5+ messages in thread
From: Olof Johansson @ 2013-07-09 8:37 UTC (permalink / raw)
To: linux-arm-kernel
"ARM: OMAP: build mach-omap code only if needed" moved around the
ARCH_OMAP2PLUS stanza, but accidentally dropped the seleciton of
TI_PRIV_EDMA in the process. Add it back.
Cc: Arnd Bergmann <arnd@arndb.de>
Cc: Tony Lindgren <tony@atomide.com>
Signed-off-by: Olof Johansson <olof@lixom.net>
---
I noticed the original patch was a bit down in the fixes branch, so
instead of rebasing I checked this in on top.
-Olof
arch/arm/mach-omap2/Kconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/mach-omap2/Kconfig b/arch/arm/mach-omap2/Kconfig
index 5c14051..58152b1 100644
--- a/arch/arm/mach-omap2/Kconfig
+++ b/arch/arm/mach-omap2/Kconfig
@@ -99,6 +99,7 @@ config ARCH_OMAP2PLUS
select PROC_DEVICETREE if PROC_FS
select SOC_BUS
select SPARSE_IRQ
+ select TI_PRIV_EDMA
select USE_OF
help
Systems based on OMAP2, OMAP3, OMAP4 or OMAP5
--
1.8.1.192.gc4361b8
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH] ARM: omap2: add select of TI_PRIV_EDMA
2013-07-09 8:37 [PATCH] ARM: omap2: add select of TI_PRIV_EDMA Olof Johansson
@ 2013-07-10 7:50 ` Tony Lindgren
2013-07-10 14:28 ` Sekhar Nori
0 siblings, 1 reply; 5+ messages in thread
From: Tony Lindgren @ 2013-07-10 7:50 UTC (permalink / raw)
To: linux-arm-kernel
* Olof Johansson <olof@lixom.net> [130709 01:44]:
> "ARM: OMAP: build mach-omap code only if needed" moved around the
> ARCH_OMAP2PLUS stanza, but accidentally dropped the seleciton of
> TI_PRIV_EDMA in the process. Add it back.
>
> Cc: Arnd Bergmann <arnd@arndb.de>
Acked-by: Tony Lindgren <tony@atomide.com>
> Signed-off-by: Olof Johansson <olof@lixom.net>
>
> ---
>
> I noticed the original patch was a bit down in the fixes branch, so
> instead of rebasing I checked this in on top.
>
> -Olof
>
> arch/arm/mach-omap2/Kconfig | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/arch/arm/mach-omap2/Kconfig b/arch/arm/mach-omap2/Kconfig
> index 5c14051..58152b1 100644
> --- a/arch/arm/mach-omap2/Kconfig
> +++ b/arch/arm/mach-omap2/Kconfig
> @@ -99,6 +99,7 @@ config ARCH_OMAP2PLUS
> select PROC_DEVICETREE if PROC_FS
> select SOC_BUS
> select SPARSE_IRQ
> + select TI_PRIV_EDMA
> select USE_OF
> help
> Systems based on OMAP2, OMAP3, OMAP4 or OMAP5
> --
> 1.8.1.192.gc4361b8
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH] ARM: omap2: add select of TI_PRIV_EDMA
2013-07-10 7:50 ` Tony Lindgren
@ 2013-07-10 14:28 ` Sekhar Nori
2013-07-10 14:36 ` Arnd Bergmann
0 siblings, 1 reply; 5+ messages in thread
From: Sekhar Nori @ 2013-07-10 14:28 UTC (permalink / raw)
To: linux-arm-kernel
On 7/10/2013 1:20 PM, Tony Lindgren wrote:
> * Olof Johansson <olof@lixom.net> [130709 01:44]:
>> "ARM: OMAP: build mach-omap code only if needed" moved around the
>> ARCH_OMAP2PLUS stanza, but accidentally dropped the seleciton of
>> TI_PRIV_EDMA in the process. Add it back.
>>
>> Cc: Arnd Bergmann <arnd@arndb.de>
>
> Acked-by: Tony Lindgren <tony@atomide.com>
>
>> Signed-off-by: Olof Johansson <olof@lixom.net>
Thinking more, I do not think we should be selecting TI_PRIV_EDMA for
OMAP2PLUS. There is no part of mach-omap2 that depends directly on
TI_PRIV_EDMA being enabled.
Instead, TI_PRIV_EDMA should be selected by the DMA engine driver as
done in the patch here:
http://www.spinics.net/lists/kernel/msg1563777.html
Its the DMA engine driver which makes the private DMA calls so this
patch is better.
Thanks,
Sekhar
>>
>> ---
>>
>> I noticed the original patch was a bit down in the fixes branch, so
>> instead of rebasing I checked this in on top.
>>
>> -Olof
>>
>> arch/arm/mach-omap2/Kconfig | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/arch/arm/mach-omap2/Kconfig b/arch/arm/mach-omap2/Kconfig
>> index 5c14051..58152b1 100644
>> --- a/arch/arm/mach-omap2/Kconfig
>> +++ b/arch/arm/mach-omap2/Kconfig
>> @@ -99,6 +99,7 @@ config ARCH_OMAP2PLUS
>> select PROC_DEVICETREE if PROC_FS
>> select SOC_BUS
>> select SPARSE_IRQ
>> + select TI_PRIV_EDMA
>> select USE_OF
>> help
>> Systems based on OMAP2, OMAP3, OMAP4 or OMAP5
>> --
>> 1.8.1.192.gc4361b8
>>
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH] ARM: omap2: add select of TI_PRIV_EDMA
2013-07-10 14:28 ` Sekhar Nori
@ 2013-07-10 14:36 ` Arnd Bergmann
2013-07-10 14:41 ` Sekhar Nori
0 siblings, 1 reply; 5+ messages in thread
From: Arnd Bergmann @ 2013-07-10 14:36 UTC (permalink / raw)
To: linux-arm-kernel
On Wednesday 10 July 2013, Sekhar Nori wrote:
> On 7/10/2013 1:20 PM, Tony Lindgren wrote:
> > * Olof Johansson <olof@lixom.net> [130709 01:44]:
> >> "ARM: OMAP: build mach-omap code only if needed" moved around the
> >> ARCH_OMAP2PLUS stanza, but accidentally dropped the seleciton of
> >> TI_PRIV_EDMA in the process. Add it back.
> >>
> >> Cc: Arnd Bergmann <arnd@arndb.de>
> >
> > Acked-by: Tony Lindgren <tony@atomide.com>
> >
> >> Signed-off-by: Olof Johansson <olof@lixom.net>
>
> Thinking more, I do not think we should be selecting TI_PRIV_EDMA for
> OMAP2PLUS. There is no part of mach-omap2 that depends directly on
> TI_PRIV_EDMA being enabled.
>
> Instead, TI_PRIV_EDMA should be selected by the DMA engine driver as
> done in the patch here:
>
> http://www.spinics.net/lists/kernel/msg1563777.html
>
> Its the DMA engine driver which makes the private DMA calls so this
> patch is better.
I would have done 'depends on TI_PRIV_EDMA' instead, but any of the three
solutions is fine with me.
Arnd
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH] ARM: omap2: add select of TI_PRIV_EDMA
2013-07-10 14:36 ` Arnd Bergmann
@ 2013-07-10 14:41 ` Sekhar Nori
0 siblings, 0 replies; 5+ messages in thread
From: Sekhar Nori @ 2013-07-10 14:41 UTC (permalink / raw)
To: linux-arm-kernel
On 7/10/2013 8:06 PM, Arnd Bergmann wrote:
> On Wednesday 10 July 2013, Sekhar Nori wrote:
>> On 7/10/2013 1:20 PM, Tony Lindgren wrote:
>>> * Olof Johansson <olof@lixom.net> [130709 01:44]:
>>>> "ARM: OMAP: build mach-omap code only if needed" moved around the
>>>> ARCH_OMAP2PLUS stanza, but accidentally dropped the seleciton of
>>>> TI_PRIV_EDMA in the process. Add it back.
>>>>
>>>> Cc: Arnd Bergmann <arnd@arndb.de>
>>>
>>> Acked-by: Tony Lindgren <tony@atomide.com>
>>>
>>>> Signed-off-by: Olof Johansson <olof@lixom.net>
>>
>> Thinking more, I do not think we should be selecting TI_PRIV_EDMA for
>> OMAP2PLUS. There is no part of mach-omap2 that depends directly on
>> TI_PRIV_EDMA being enabled.
>>
>> Instead, TI_PRIV_EDMA should be selected by the DMA engine driver as
>> done in the patch here:
>>
>> http://www.spinics.net/lists/kernel/msg1563777.html
>>
>> Its the DMA engine driver which makes the private DMA calls so this
>> patch is better.
>
> I would have done 'depends on TI_PRIV_EDMA' instead, but any of the three
> solutions is fine with me.
TI_PRIV_EDMA is not a user selectable option so if 'depends on' is used,
then the select in mach-omap2 is needed as well.
Thanks,
Sekhar
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2013-07-10 14:41 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-07-09 8:37 [PATCH] ARM: omap2: add select of TI_PRIV_EDMA Olof Johansson
2013-07-10 7:50 ` Tony Lindgren
2013-07-10 14:28 ` Sekhar Nori
2013-07-10 14:36 ` Arnd Bergmann
2013-07-10 14:41 ` Sekhar Nori
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.