diff mbox

[1/1] ARM: ux500: Describe UART platform registering issues more accurately

Message ID 1352968401-6092-1-git-send-email-lee.jones@linaro.org (mailing list archive)
State New, archived
Headers show

Commit Message

Lee Jones Nov. 15, 2012, 8:33 a.m. UTC
UARTs no longer require call-back information, since the reset
call-back was removed in 43b5f0d69291374f602ad8e1817f329573a59010.
The only AUXDATA dependencies remaining for UARTs are DMA settings.

Signed-off-by: Lee Jones <lee.jones@linaro.org>
---
 arch/arm/mach-ux500/cpu-db8500.c |    3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

Comments

Arnd Bergmann Nov. 15, 2012, 12:23 p.m. UTC | #1
On Thursday 15 November 2012, Lee Jones wrote:
> UARTs no longer require call-back information, since the reset
> call-back was removed in 43b5f0d69291374f602ad8e1817f329573a59010.
> The only AUXDATA dependencies remaining for UARTs are DMA settings.
> 
> Signed-off-by: Lee Jones <lee.jones@linaro.org>

Acked-by: Arnd Bergmann <arnd@arndb.de>

What is the state of the DMA binding now? We originally wanted to merge
it for 3.7, but that didn't work out, so I hope we can get it done for
3.8.

	Arnd
Lee Jones Nov. 15, 2012, 12:33 p.m. UTC | #2
On Thu, 15 Nov 2012, Arnd Bergmann wrote:

> On Thursday 15 November 2012, Lee Jones wrote:
> > UARTs no longer require call-back information, since the reset
> > call-back was removed in 43b5f0d69291374f602ad8e1817f329573a59010.
> > The only AUXDATA dependencies remaining for UARTs are DMA settings.
> > 
> > Signed-off-by: Lee Jones <lee.jones@linaro.org>
> 
> Acked-by: Arnd Bergmann <arnd@arndb.de>
> 
> What is the state of the DMA binding now? We originally wanted to merge
> it for 3.7, but that didn't work out, so I hope we can get it done for
> 3.8.

It's dead. The conversation has been stale for weeks.

Perhaps you'd like to go and poke it? :)
Linus Walleij Nov. 15, 2012, 5:28 p.m. UTC | #3
On Thu, Nov 15, 2012 at 9:33 AM, Lee Jones <lee.jones@linaro.org> wrote:

> UARTs no longer require call-back information, since the reset
> call-back was removed in 43b5f0d69291374f602ad8e1817f329573a59010.
> The only AUXDATA dependencies remaining for UARTs are DMA settings.
>
> Signed-off-by: Lee Jones <lee.jones@linaro.org>

Acked-by: Linus Walleij <linus.walleij@linaro.org>

Yours,
Linus Walleij
Arnd Bergmann Nov. 16, 2012, 9:36 a.m. UTC | #4
On Thursday 15 November 2012, Lee Jones wrote:
> On Thu, 15 Nov 2012, Arnd Bergmann wrote:
> 
> > On Thursday 15 November 2012, Lee Jones wrote:
> > > UARTs no longer require call-back information, since the reset
> > > call-back was removed in 43b5f0d69291374f602ad8e1817f329573a59010.
> > > The only AUXDATA dependencies remaining for UARTs are DMA settings.
> > > 
> > > Signed-off-by: Lee Jones <lee.jones@linaro.org>
> > 
> > Acked-by: Arnd Bergmann <arnd@arndb.de>
> > 
> > What is the state of the DMA binding now? We originally wanted to merge
> > it for 3.7, but that didn't work out, so I hope we can get it done for
> > 3.8.
> 
> It's dead. The conversation has been stale for weeks.
> 
> Perhaps you'd like to go and poke it? :)
> 

Vinod,

is there anything you are waiting for still? Should Jon resend
his latest patches to make sure we get them merged this time?

I have multiple people that want to send me patches for 3.8 based
on that work, so we are running out of time now.

	Arnd
Hunter, Jon Nov. 16, 2012, 3:27 p.m. UTC | #5
On 11/16/2012 03:36 AM, Arnd Bergmann wrote:
> On Thursday 15 November 2012, Lee Jones wrote:
>> On Thu, 15 Nov 2012, Arnd Bergmann wrote:
>>
>>> On Thursday 15 November 2012, Lee Jones wrote:
>>>> UARTs no longer require call-back information, since the reset
>>>> call-back was removed in 43b5f0d69291374f602ad8e1817f329573a59010.
>>>> The only AUXDATA dependencies remaining for UARTs are DMA settings.
>>>>
>>>> Signed-off-by: Lee Jones <lee.jones@linaro.org>
>>>
>>> Acked-by: Arnd Bergmann <arnd@arndb.de>
>>>
>>> What is the state of the DMA binding now? We originally wanted to merge
>>> it for 3.7, but that didn't work out, so I hope we can get it done for
>>> 3.8.
>>
>> It's dead. The conversation has been stale for weeks.
>>
>> Perhaps you'd like to go and poke it? :)
>>
> 
> Vinod,
> 
> is there anything you are waiting for still? Should Jon resend
> his latest patches to make sure we get them merged this time?
> 
> I have multiple people that want to send me patches for 3.8 based
> on that work, so we are running out of time now.

Vinod responded today saying it will be in for v3.8 [1].

Jon

[1] http://article.gmane.org/gmane.linux.ports.arm.omap/89502
Arnd Bergmann Nov. 16, 2012, 3:34 p.m. UTC | #6
On Friday 16 November 2012, Jon Hunter wrote:
> > is there anything you are waiting for still? Should Jon resend
> > his latest patches to make sure we get them merged this time?
> > 
> > I have multiple people that want to send me patches for 3.8 based
> > on that work, so we are running out of time now.
> 
> Vinod responded today saying it will be in for v3.8 [1].
> 

Ok, very good. Thanks everyone!

	Arnd
Vinod Koul Nov. 16, 2012, 3:39 p.m. UTC | #7
> > > is there anything you are waiting for still? Should Jon resend his
> > > latest patches to make sure we get them merged this time?
> > >
> > > I have multiple people that want to send me patches for 3.8 based on
> > > that work, so we are running out of time now.
> >
> > Vinod responded today saying it will be in for v3.8 [1].
> >
> 
> Ok, very good. Thanks everyone!
If you have any patches for this, please send me. Your work should not stall,
that is why I had the topic branch.

I plan to merge this to my next mid next week,....
....based on how early my travel permits :)

--
~Vinod
diff mbox

Patch

diff --git a/arch/arm/mach-ux500/cpu-db8500.c b/arch/arm/mach-ux500/cpu-db8500.c
index 4a0c40a..d2076ce 100644
--- a/arch/arm/mach-ux500/cpu-db8500.c
+++ b/arch/arm/mach-ux500/cpu-db8500.c
@@ -262,11 +262,10 @@  static struct device * __init u8500_of_init_devices(void)
 static struct of_dev_auxdata u8500_auxdata_lookup[] __initdata = {
 	/* Requires call-back bindings. */
 	OF_DEV_AUXDATA("arm,cortex-a9-pmu", 0, "arm-pmu", &db8500_pmu_platdata),
-	/* Requires DMA and call-back bindings. */
+	/* Requires DMA bindings. */
 	OF_DEV_AUXDATA("arm,pl011", 0x80120000, "uart0", &uart0_plat),
 	OF_DEV_AUXDATA("arm,pl011", 0x80121000, "uart1", &uart1_plat),
 	OF_DEV_AUXDATA("arm,pl011", 0x80007000, "uart2", &uart2_plat),
-	/* Requires DMA bindings. */
 	OF_DEV_AUXDATA("arm,pl022", 0x80002000, "ssp0",  &ssp0_plat),
 	OF_DEV_AUXDATA("arm,pl18x", 0x80126000, "sdi0",  &mop500_sdi0_data),
 	OF_DEV_AUXDATA("arm,pl18x", 0x80118000, "sdi1",  &mop500_sdi1_data),