All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2] ARM: davinci: Add the clock for the CPPI 4.1 DMA engine
@ 2017-04-05 17:17 ` Alexandre Bailon
  0 siblings, 0 replies; 10+ messages in thread
From: Alexandre Bailon @ 2017-04-05 17:17 UTC (permalink / raw)
  To: nsekhar-l0cyMroinI0
  Cc: khilman-rdvid1DuHRBWk0Htik3J/w, ptitiano-rdvid1DuHRBWk0Htik3J/w,
	tony-4v6yS6AI5VpBDgjK7y7TUQ, linux-omap-u79uwXL29TY76Z2rM5mHXA,
	b-liu-l0cyMroinI0,
	sergei.shtylyov-M4DtvfQ/ZS1MRgGoP+s0PdBPR1lH4CV8,
	linux-usb-u79uwXL29TY76Z2rM5mHXA,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	grygorii.strashko-l0cyMroinI0, Alexandre Bailon

The CPPI 4.1 DMA is sharing its clock with the USB OTG,
and most of the time, the clock will be enabled by USB.
But during the init of the DMA, USB is not enabled (waiting for DMA),
and then we must enable the clock before doing anything.
Add the clock for the CPPI 4.1 DMA engine.

Signed-off-by: Alexandre Bailon <abailon-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>
---
 arch/arm/mach-davinci/da830.c | 6 ++++++
 arch/arm/mach-davinci/da850.c | 6 ++++++
 2 files changed, 12 insertions(+)

diff --git a/arch/arm/mach-davinci/da830.c b/arch/arm/mach-davinci/da830.c
index 073c458..bd88470 100644
--- a/arch/arm/mach-davinci/da830.c
+++ b/arch/arm/mach-davinci/da830.c
@@ -304,6 +304,11 @@ static struct clk usb20_clk = {
 	.gpsc		= 1,
 };
 
+static struct clk cppi41_clk = {
+	.name		= "cppi41",
+	.parent		= &usb20_clk,
+};
+
 static struct clk aemif_clk = {
 	.name		= "aemif",
 	.parent		= &pll0_sysclk3,
@@ -413,6 +418,7 @@ static struct clk_lookup da830_clks[] = {
 	CLK("davinci-mcasp.1",	NULL,		&mcasp1_clk),
 	CLK("davinci-mcasp.2",	NULL,		&mcasp2_clk),
 	CLK("musb-da8xx",	"usb20",	&usb20_clk),
+	CLK("cppi41-dmaengine",	NULL,		&cppi41_clk),
 	CLK(NULL,		"aemif",	&aemif_clk),
 	CLK(NULL,		"aintc",	&aintc_clk),
 	CLK(NULL,		"secu_mgr",	&secu_mgr_clk),
diff --git a/arch/arm/mach-davinci/da850.c b/arch/arm/mach-davinci/da850.c
index ccad2f9..07d6f0e 100644
--- a/arch/arm/mach-davinci/da850.c
+++ b/arch/arm/mach-davinci/da850.c
@@ -401,6 +401,11 @@ static struct clk usb20_clk = {
 	.gpsc		= 1,
 };
 
+static struct clk cppi41_clk = {
+	.name		= "cppi41",
+	.parent		= &usb20_clk,
+};
+
 static struct clk spi0_clk = {
 	.name		= "spi0",
 	.parent		= &pll0_sysclk2,
@@ -560,6 +565,7 @@ static struct clk_lookup da850_clks[] = {
 	CLK("davinci-nand.0",	"aemif",	&aemif_nand_clk),
 	CLK("ohci-da8xx",	"usb11",	&usb11_clk),
 	CLK("musb-da8xx",	"usb20",	&usb20_clk),
+	CLK("cppi41-dmaengine",	NULL,		&cppi41_clk),
 	CLK("spi_davinci.0",	NULL,		&spi0_clk),
 	CLK("spi_davinci.1",	NULL,		&spi1_clk),
 	CLK("vpif",		NULL,		&vpif_clk),
-- 
2.10.2

--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply related	[flat|nested] 10+ messages in thread

* [PATCH v2] ARM: davinci: Add the clock for the CPPI 4.1 DMA engine
@ 2017-04-05 17:17 ` Alexandre Bailon
  0 siblings, 0 replies; 10+ messages in thread
From: Alexandre Bailon @ 2017-04-05 17:17 UTC (permalink / raw)
  To: linux-arm-kernel

The CPPI 4.1 DMA is sharing its clock with the USB OTG,
and most of the time, the clock will be enabled by USB.
But during the init of the DMA, USB is not enabled (waiting for DMA),
and then we must enable the clock before doing anything.
Add the clock for the CPPI 4.1 DMA engine.

Signed-off-by: Alexandre Bailon <abailon@baylibre.com>
---
 arch/arm/mach-davinci/da830.c | 6 ++++++
 arch/arm/mach-davinci/da850.c | 6 ++++++
 2 files changed, 12 insertions(+)

diff --git a/arch/arm/mach-davinci/da830.c b/arch/arm/mach-davinci/da830.c
index 073c458..bd88470 100644
--- a/arch/arm/mach-davinci/da830.c
+++ b/arch/arm/mach-davinci/da830.c
@@ -304,6 +304,11 @@ static struct clk usb20_clk = {
 	.gpsc		= 1,
 };
 
+static struct clk cppi41_clk = {
+	.name		= "cppi41",
+	.parent		= &usb20_clk,
+};
+
 static struct clk aemif_clk = {
 	.name		= "aemif",
 	.parent		= &pll0_sysclk3,
@@ -413,6 +418,7 @@ static struct clk_lookup da830_clks[] = {
 	CLK("davinci-mcasp.1",	NULL,		&mcasp1_clk),
 	CLK("davinci-mcasp.2",	NULL,		&mcasp2_clk),
 	CLK("musb-da8xx",	"usb20",	&usb20_clk),
+	CLK("cppi41-dmaengine",	NULL,		&cppi41_clk),
 	CLK(NULL,		"aemif",	&aemif_clk),
 	CLK(NULL,		"aintc",	&aintc_clk),
 	CLK(NULL,		"secu_mgr",	&secu_mgr_clk),
diff --git a/arch/arm/mach-davinci/da850.c b/arch/arm/mach-davinci/da850.c
index ccad2f9..07d6f0e 100644
--- a/arch/arm/mach-davinci/da850.c
+++ b/arch/arm/mach-davinci/da850.c
@@ -401,6 +401,11 @@ static struct clk usb20_clk = {
 	.gpsc		= 1,
 };
 
+static struct clk cppi41_clk = {
+	.name		= "cppi41",
+	.parent		= &usb20_clk,
+};
+
 static struct clk spi0_clk = {
 	.name		= "spi0",
 	.parent		= &pll0_sysclk2,
@@ -560,6 +565,7 @@ static struct clk_lookup da850_clks[] = {
 	CLK("davinci-nand.0",	"aemif",	&aemif_nand_clk),
 	CLK("ohci-da8xx",	"usb11",	&usb11_clk),
 	CLK("musb-da8xx",	"usb20",	&usb20_clk),
+	CLK("cppi41-dmaengine",	NULL,		&cppi41_clk),
 	CLK("spi_davinci.0",	NULL,		&spi0_clk),
 	CLK("spi_davinci.1",	NULL,		&spi1_clk),
 	CLK("vpif",		NULL,		&vpif_clk),
-- 
2.10.2

^ permalink raw reply related	[flat|nested] 10+ messages in thread

* Re: [PATCH v2] ARM: davinci: Add the clock for the CPPI 4.1 DMA engine
  2017-04-05 17:17 ` Alexandre Bailon
@ 2017-04-07 14:36   ` Sekhar Nori
  -1 siblings, 0 replies; 10+ messages in thread
From: Sekhar Nori @ 2017-04-07 14:36 UTC (permalink / raw)
  To: Alexandre Bailon
  Cc: grygorii.strashko, sergei.shtylyov, tony, khilman, linux-usb,
	ptitiano, linux-omap, b-liu, linux-arm-kernel

On Wednesday 05 April 2017 10:47 PM, Alexandre Bailon wrote:
> The CPPI 4.1 DMA is sharing its clock with the USB OTG,
> and most of the time, the clock will be enabled by USB.
> But during the init of the DMA, USB is not enabled (waiting for DMA),
> and then we must enable the clock before doing anything.
> Add the clock for the CPPI 4.1 DMA engine.
> 
> Signed-off-by: Alexandre Bailon <abailon@baylibre.com>
> ---
>  arch/arm/mach-davinci/da830.c | 6 ++++++
>  arch/arm/mach-davinci/da850.c | 6 ++++++
>  2 files changed, 12 insertions(+)
> 
> diff --git a/arch/arm/mach-davinci/da830.c b/arch/arm/mach-davinci/da830.c
> index 073c458..bd88470 100644
> --- a/arch/arm/mach-davinci/da830.c
> +++ b/arch/arm/mach-davinci/da830.c
> @@ -304,6 +304,11 @@ static struct clk usb20_clk = {
>  	.gpsc		= 1,
>  };
>  
> +static struct clk cppi41_clk = {
> +	.name		= "cppi41",
> +	.parent		= &usb20_clk,
> +};
> +
>  static struct clk aemif_clk = {
>  	.name		= "aemif",
>  	.parent		= &pll0_sysclk3,
> @@ -413,6 +418,7 @@ static struct clk_lookup da830_clks[] = {
>  	CLK("davinci-mcasp.1",	NULL,		&mcasp1_clk),
>  	CLK("davinci-mcasp.2",	NULL,		&mcasp2_clk),
>  	CLK("musb-da8xx",	"usb20",	&usb20_clk),
> +	CLK("cppi41-dmaengine",	NULL,		&cppi41_clk),

I dont see this device name being used in current linux-next. Is this
name accepted ?

The patch otherwise looks okay.

Thanks,
Sekhar

^ permalink raw reply	[flat|nested] 10+ messages in thread

* [PATCH v2] ARM: davinci: Add the clock for the CPPI 4.1 DMA engine
@ 2017-04-07 14:36   ` Sekhar Nori
  0 siblings, 0 replies; 10+ messages in thread
From: Sekhar Nori @ 2017-04-07 14:36 UTC (permalink / raw)
  To: linux-arm-kernel

On Wednesday 05 April 2017 10:47 PM, Alexandre Bailon wrote:
> The CPPI 4.1 DMA is sharing its clock with the USB OTG,
> and most of the time, the clock will be enabled by USB.
> But during the init of the DMA, USB is not enabled (waiting for DMA),
> and then we must enable the clock before doing anything.
> Add the clock for the CPPI 4.1 DMA engine.
> 
> Signed-off-by: Alexandre Bailon <abailon@baylibre.com>
> ---
>  arch/arm/mach-davinci/da830.c | 6 ++++++
>  arch/arm/mach-davinci/da850.c | 6 ++++++
>  2 files changed, 12 insertions(+)
> 
> diff --git a/arch/arm/mach-davinci/da830.c b/arch/arm/mach-davinci/da830.c
> index 073c458..bd88470 100644
> --- a/arch/arm/mach-davinci/da830.c
> +++ b/arch/arm/mach-davinci/da830.c
> @@ -304,6 +304,11 @@ static struct clk usb20_clk = {
>  	.gpsc		= 1,
>  };
>  
> +static struct clk cppi41_clk = {
> +	.name		= "cppi41",
> +	.parent		= &usb20_clk,
> +};
> +
>  static struct clk aemif_clk = {
>  	.name		= "aemif",
>  	.parent		= &pll0_sysclk3,
> @@ -413,6 +418,7 @@ static struct clk_lookup da830_clks[] = {
>  	CLK("davinci-mcasp.1",	NULL,		&mcasp1_clk),
>  	CLK("davinci-mcasp.2",	NULL,		&mcasp2_clk),
>  	CLK("musb-da8xx",	"usb20",	&usb20_clk),
> +	CLK("cppi41-dmaengine",	NULL,		&cppi41_clk),

I dont see this device name being used in current linux-next. Is this
name accepted ?

The patch otherwise looks okay.

Thanks,
Sekhar

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH v2] ARM: davinci: Add the clock for the CPPI 4.1 DMA engine
  2017-04-07 14:36   ` Sekhar Nori
@ 2017-04-07 16:15       ` Alexandre Bailon
  -1 siblings, 0 replies; 10+ messages in thread
From: Alexandre Bailon @ 2017-04-07 16:15 UTC (permalink / raw)
  To: Sekhar Nori
  Cc: khilman-rdvid1DuHRBWk0Htik3J/w, ptitiano-rdvid1DuHRBWk0Htik3J/w,
	tony-4v6yS6AI5VpBDgjK7y7TUQ, linux-omap-u79uwXL29TY76Z2rM5mHXA,
	b-liu-l0cyMroinI0,
	sergei.shtylyov-M4DtvfQ/ZS1MRgGoP+s0PdBPR1lH4CV8,
	linux-usb-u79uwXL29TY76Z2rM5mHXA,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	grygorii.strashko-l0cyMroinI0



On 04/07/2017 04:36 PM, Sekhar Nori wrote:
> On Wednesday 05 April 2017 10:47 PM, Alexandre Bailon wrote:
>> The CPPI 4.1 DMA is sharing its clock with the USB OTG,
>> and most of the time, the clock will be enabled by USB.
>> But during the init of the DMA, USB is not enabled (waiting for DMA),
>> and then we must enable the clock before doing anything.
>> Add the clock for the CPPI 4.1 DMA engine.
>>
>> Signed-off-by: Alexandre Bailon <abailon-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>
>> ---
>>   arch/arm/mach-davinci/da830.c | 6 ++++++
>>   arch/arm/mach-davinci/da850.c | 6 ++++++
>>   2 files changed, 12 insertions(+)
>>
>> diff --git a/arch/arm/mach-davinci/da830.c b/arch/arm/mach-davinci/da830.c
>> index 073c458..bd88470 100644
>> --- a/arch/arm/mach-davinci/da830.c
>> +++ b/arch/arm/mach-davinci/da830.c
>> @@ -304,6 +304,11 @@ static struct clk usb20_clk = {
>>   	.gpsc		= 1,
>>   };
>>   
>> +static struct clk cppi41_clk = {
>> +	.name		= "cppi41",
>> +	.parent		= &usb20_clk,
>> +};
>> +
>>   static struct clk aemif_clk = {
>>   	.name		= "aemif",
>>   	.parent		= &pll0_sysclk3,
>> @@ -413,6 +418,7 @@ static struct clk_lookup da830_clks[] = {
>>   	CLK("davinci-mcasp.1",	NULL,		&mcasp1_clk),
>>   	CLK("davinci-mcasp.2",	NULL,		&mcasp2_clk),
>>   	CLK("musb-da8xx",	"usb20",	&usb20_clk),
>> +	CLK("cppi41-dmaengine",	NULL,		&cppi41_clk),
> I dont see this device name being used in current linux-next. Is this
> name accepted ?
There is here a typo. The name should be cppi41-dma-engine.
I will fix it.
>
> The patch otherwise looks okay.
>
> Thanks,
> Sekhar
Thanks,
Alexandre
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 10+ messages in thread

* [PATCH v2] ARM: davinci: Add the clock for the CPPI 4.1 DMA engine
@ 2017-04-07 16:15       ` Alexandre Bailon
  0 siblings, 0 replies; 10+ messages in thread
From: Alexandre Bailon @ 2017-04-07 16:15 UTC (permalink / raw)
  To: linux-arm-kernel



On 04/07/2017 04:36 PM, Sekhar Nori wrote:
> On Wednesday 05 April 2017 10:47 PM, Alexandre Bailon wrote:
>> The CPPI 4.1 DMA is sharing its clock with the USB OTG,
>> and most of the time, the clock will be enabled by USB.
>> But during the init of the DMA, USB is not enabled (waiting for DMA),
>> and then we must enable the clock before doing anything.
>> Add the clock for the CPPI 4.1 DMA engine.
>>
>> Signed-off-by: Alexandre Bailon <abailon@baylibre.com>
>> ---
>>   arch/arm/mach-davinci/da830.c | 6 ++++++
>>   arch/arm/mach-davinci/da850.c | 6 ++++++
>>   2 files changed, 12 insertions(+)
>>
>> diff --git a/arch/arm/mach-davinci/da830.c b/arch/arm/mach-davinci/da830.c
>> index 073c458..bd88470 100644
>> --- a/arch/arm/mach-davinci/da830.c
>> +++ b/arch/arm/mach-davinci/da830.c
>> @@ -304,6 +304,11 @@ static struct clk usb20_clk = {
>>   	.gpsc		= 1,
>>   };
>>   
>> +static struct clk cppi41_clk = {
>> +	.name		= "cppi41",
>> +	.parent		= &usb20_clk,
>> +};
>> +
>>   static struct clk aemif_clk = {
>>   	.name		= "aemif",
>>   	.parent		= &pll0_sysclk3,
>> @@ -413,6 +418,7 @@ static struct clk_lookup da830_clks[] = {
>>   	CLK("davinci-mcasp.1",	NULL,		&mcasp1_clk),
>>   	CLK("davinci-mcasp.2",	NULL,		&mcasp2_clk),
>>   	CLK("musb-da8xx",	"usb20",	&usb20_clk),
>> +	CLK("cppi41-dmaengine",	NULL,		&cppi41_clk),
> I dont see this device name being used in current linux-next. Is this
> name accepted ?
There is here a typo. The name should be cppi41-dma-engine.
I will fix it.
>
> The patch otherwise looks okay.
>
> Thanks,
> Sekhar
Thanks,
Alexandre

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH v2] ARM: davinci: Add the clock for the CPPI 4.1 DMA engine
  2017-04-07 16:15       ` Alexandre Bailon
@ 2017-04-07 17:31           ` Alexandre Bailon
  -1 siblings, 0 replies; 10+ messages in thread
From: Alexandre Bailon @ 2017-04-07 17:31 UTC (permalink / raw)
  To: Sekhar Nori
  Cc: khilman-rdvid1DuHRBWk0Htik3J/w, ptitiano-rdvid1DuHRBWk0Htik3J/w,
	tony-4v6yS6AI5VpBDgjK7y7TUQ, linux-omap-u79uwXL29TY76Z2rM5mHXA,
	b-liu-l0cyMroinI0,
	sergei.shtylyov-M4DtvfQ/ZS1MRgGoP+s0PdBPR1lH4CV8,
	linux-usb-u79uwXL29TY76Z2rM5mHXA,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	grygorii.strashko-l0cyMroinI0



On 04/07/2017 06:15 PM, Alexandre Bailon wrote:
>
>
> On 04/07/2017 04:36 PM, Sekhar Nori wrote:
>> On Wednesday 05 April 2017 10:47 PM, Alexandre Bailon wrote:
>>> The CPPI 4.1 DMA is sharing its clock with the USB OTG,
>>> and most of the time, the clock will be enabled by USB.
>>> But during the init of the DMA, USB is not enabled (waiting for DMA),
>>> and then we must enable the clock before doing anything.
>>> Add the clock for the CPPI 4.1 DMA engine.
>>>
>>> Signed-off-by: Alexandre Bailon <abailon-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>
>>> ---
>>>   arch/arm/mach-davinci/da830.c | 6 ++++++
>>>   arch/arm/mach-davinci/da850.c | 6 ++++++
>>>   2 files changed, 12 insertions(+)
>>>
>>> diff --git a/arch/arm/mach-davinci/da830.c 
>>> b/arch/arm/mach-davinci/da830.c
>>> index 073c458..bd88470 100644
>>> --- a/arch/arm/mach-davinci/da830.c
>>> +++ b/arch/arm/mach-davinci/da830.c
>>> @@ -304,6 +304,11 @@ static struct clk usb20_clk = {
>>>       .gpsc        = 1,
>>>   };
>>>   +static struct clk cppi41_clk = {
>>> +    .name        = "cppi41",
>>> +    .parent        = &usb20_clk,
>>> +};
>>> +
>>>   static struct clk aemif_clk = {
>>>       .name        = "aemif",
>>>       .parent        = &pll0_sysclk3,
>>> @@ -413,6 +418,7 @@ static struct clk_lookup da830_clks[] = {
>>>       CLK("davinci-mcasp.1",    NULL,        &mcasp1_clk),
>>>       CLK("davinci-mcasp.2",    NULL,        &mcasp2_clk),
>>>       CLK("musb-da8xx",    "usb20",    &usb20_clk),
>>> +    CLK("cppi41-dmaengine",    NULL,        &cppi41_clk),
>> I dont see this device name being used in current linux-next. Is this
>> name accepted ?
> There is here a typo. The name should be cppi41-dma-engine.
> I will fix it.
Actually, it is not a typo. It would have be more logical to name it 
cppi41-dma-engine
(like the driver name) but the name is correct.
The device name is not yet in linux-next as the device is created in 
da8xx driver.
http://marc.info/?l=linux-usb&m=149080474124498&w=2
>>
>> The patch otherwise looks okay.
>>
>> Thanks,
>> Sekhar
> Thanks,
> Alexandre

--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 10+ messages in thread

* [PATCH v2] ARM: davinci: Add the clock for the CPPI 4.1 DMA engine
@ 2017-04-07 17:31           ` Alexandre Bailon
  0 siblings, 0 replies; 10+ messages in thread
From: Alexandre Bailon @ 2017-04-07 17:31 UTC (permalink / raw)
  To: linux-arm-kernel



On 04/07/2017 06:15 PM, Alexandre Bailon wrote:
>
>
> On 04/07/2017 04:36 PM, Sekhar Nori wrote:
>> On Wednesday 05 April 2017 10:47 PM, Alexandre Bailon wrote:
>>> The CPPI 4.1 DMA is sharing its clock with the USB OTG,
>>> and most of the time, the clock will be enabled by USB.
>>> But during the init of the DMA, USB is not enabled (waiting for DMA),
>>> and then we must enable the clock before doing anything.
>>> Add the clock for the CPPI 4.1 DMA engine.
>>>
>>> Signed-off-by: Alexandre Bailon <abailon@baylibre.com>
>>> ---
>>>   arch/arm/mach-davinci/da830.c | 6 ++++++
>>>   arch/arm/mach-davinci/da850.c | 6 ++++++
>>>   2 files changed, 12 insertions(+)
>>>
>>> diff --git a/arch/arm/mach-davinci/da830.c 
>>> b/arch/arm/mach-davinci/da830.c
>>> index 073c458..bd88470 100644
>>> --- a/arch/arm/mach-davinci/da830.c
>>> +++ b/arch/arm/mach-davinci/da830.c
>>> @@ -304,6 +304,11 @@ static struct clk usb20_clk = {
>>>       .gpsc        = 1,
>>>   };
>>>   +static struct clk cppi41_clk = {
>>> +    .name        = "cppi41",
>>> +    .parent        = &usb20_clk,
>>> +};
>>> +
>>>   static struct clk aemif_clk = {
>>>       .name        = "aemif",
>>>       .parent        = &pll0_sysclk3,
>>> @@ -413,6 +418,7 @@ static struct clk_lookup da830_clks[] = {
>>>       CLK("davinci-mcasp.1",    NULL,        &mcasp1_clk),
>>>       CLK("davinci-mcasp.2",    NULL,        &mcasp2_clk),
>>>       CLK("musb-da8xx",    "usb20",    &usb20_clk),
>>> +    CLK("cppi41-dmaengine",    NULL,        &cppi41_clk),
>> I dont see this device name being used in current linux-next. Is this
>> name accepted ?
> There is here a typo. The name should be cppi41-dma-engine.
> I will fix it.
Actually, it is not a typo. It would have be more logical to name it 
cppi41-dma-engine
(like the driver name) but the name is correct.
The device name is not yet in linux-next as the device is created in 
da8xx driver.
http://marc.info/?l=linux-usb&m=149080474124498&w=2
>>
>> The patch otherwise looks okay.
>>
>> Thanks,
>> Sekhar
> Thanks,
> Alexandre

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH v2] ARM: davinci: Add the clock for the CPPI 4.1 DMA engine
  2017-04-07 17:31           ` Alexandre Bailon
@ 2017-04-12 11:27             ` Sekhar Nori
  -1 siblings, 0 replies; 10+ messages in thread
From: Sekhar Nori @ 2017-04-12 11:27 UTC (permalink / raw)
  To: Alexandre Bailon
  Cc: grygorii.strashko, sergei.shtylyov, tony, khilman, linux-usb,
	ptitiano, linux-omap, b-liu, linux-arm-kernel

On Friday 07 April 2017 11:01 PM, Alexandre Bailon wrote:
> 
> 
> On 04/07/2017 06:15 PM, Alexandre Bailon wrote:
>>
>>
>> On 04/07/2017 04:36 PM, Sekhar Nori wrote:
>>> On Wednesday 05 April 2017 10:47 PM, Alexandre Bailon wrote:
>>>> The CPPI 4.1 DMA is sharing its clock with the USB OTG,
>>>> and most of the time, the clock will be enabled by USB.
>>>> But during the init of the DMA, USB is not enabled (waiting for DMA),
>>>> and then we must enable the clock before doing anything.
>>>> Add the clock for the CPPI 4.1 DMA engine.
>>>>
>>>> Signed-off-by: Alexandre Bailon <abailon@baylibre.com>
>>>> ---
>>>>   arch/arm/mach-davinci/da830.c | 6 ++++++
>>>>   arch/arm/mach-davinci/da850.c | 6 ++++++
>>>>   2 files changed, 12 insertions(+)
>>>>
>>>> diff --git a/arch/arm/mach-davinci/da830.c
>>>> b/arch/arm/mach-davinci/da830.c
>>>> index 073c458..bd88470 100644
>>>> --- a/arch/arm/mach-davinci/da830.c
>>>> +++ b/arch/arm/mach-davinci/da830.c
>>>> @@ -304,6 +304,11 @@ static struct clk usb20_clk = {
>>>>       .gpsc        = 1,
>>>>   };
>>>>   +static struct clk cppi41_clk = {
>>>> +    .name        = "cppi41",
>>>> +    .parent        = &usb20_clk,
>>>> +};
>>>> +
>>>>   static struct clk aemif_clk = {
>>>>       .name        = "aemif",
>>>>       .parent        = &pll0_sysclk3,
>>>> @@ -413,6 +418,7 @@ static struct clk_lookup da830_clks[] = {
>>>>       CLK("davinci-mcasp.1",    NULL,        &mcasp1_clk),
>>>>       CLK("davinci-mcasp.2",    NULL,        &mcasp2_clk),
>>>>       CLK("musb-da8xx",    "usb20",    &usb20_clk),
>>>> +    CLK("cppi41-dmaengine",    NULL,        &cppi41_clk),
>>> I dont see this device name being used in current linux-next. Is this
>>> name accepted ?
>> There is here a typo. The name should be cppi41-dma-engine.
>> I will fix it.
> Actually, it is not a typo. It would have be more logical to name it
> cppi41-dma-engine
> (like the driver name) but the name is correct.
> The device name is not yet in linux-next as the device is created in
> da8xx driver.
> http://marc.info/?l=linux-usb&m=149080474124498&w=2

Alright, applied this and sent a pull request. Made some minor changes
to commit message text.

Thanks,
Sekhar

^ permalink raw reply	[flat|nested] 10+ messages in thread

* [PATCH v2] ARM: davinci: Add the clock for the CPPI 4.1 DMA engine
@ 2017-04-12 11:27             ` Sekhar Nori
  0 siblings, 0 replies; 10+ messages in thread
From: Sekhar Nori @ 2017-04-12 11:27 UTC (permalink / raw)
  To: linux-arm-kernel

On Friday 07 April 2017 11:01 PM, Alexandre Bailon wrote:
> 
> 
> On 04/07/2017 06:15 PM, Alexandre Bailon wrote:
>>
>>
>> On 04/07/2017 04:36 PM, Sekhar Nori wrote:
>>> On Wednesday 05 April 2017 10:47 PM, Alexandre Bailon wrote:
>>>> The CPPI 4.1 DMA is sharing its clock with the USB OTG,
>>>> and most of the time, the clock will be enabled by USB.
>>>> But during the init of the DMA, USB is not enabled (waiting for DMA),
>>>> and then we must enable the clock before doing anything.
>>>> Add the clock for the CPPI 4.1 DMA engine.
>>>>
>>>> Signed-off-by: Alexandre Bailon <abailon@baylibre.com>
>>>> ---
>>>>   arch/arm/mach-davinci/da830.c | 6 ++++++
>>>>   arch/arm/mach-davinci/da850.c | 6 ++++++
>>>>   2 files changed, 12 insertions(+)
>>>>
>>>> diff --git a/arch/arm/mach-davinci/da830.c
>>>> b/arch/arm/mach-davinci/da830.c
>>>> index 073c458..bd88470 100644
>>>> --- a/arch/arm/mach-davinci/da830.c
>>>> +++ b/arch/arm/mach-davinci/da830.c
>>>> @@ -304,6 +304,11 @@ static struct clk usb20_clk = {
>>>>       .gpsc        = 1,
>>>>   };
>>>>   +static struct clk cppi41_clk = {
>>>> +    .name        = "cppi41",
>>>> +    .parent        = &usb20_clk,
>>>> +};
>>>> +
>>>>   static struct clk aemif_clk = {
>>>>       .name        = "aemif",
>>>>       .parent        = &pll0_sysclk3,
>>>> @@ -413,6 +418,7 @@ static struct clk_lookup da830_clks[] = {
>>>>       CLK("davinci-mcasp.1",    NULL,        &mcasp1_clk),
>>>>       CLK("davinci-mcasp.2",    NULL,        &mcasp2_clk),
>>>>       CLK("musb-da8xx",    "usb20",    &usb20_clk),
>>>> +    CLK("cppi41-dmaengine",    NULL,        &cppi41_clk),
>>> I dont see this device name being used in current linux-next. Is this
>>> name accepted ?
>> There is here a typo. The name should be cppi41-dma-engine.
>> I will fix it.
> Actually, it is not a typo. It would have be more logical to name it
> cppi41-dma-engine
> (like the driver name) but the name is correct.
> The device name is not yet in linux-next as the device is created in
> da8xx driver.
> http://marc.info/?l=linux-usb&m=149080474124498&w=2

Alright, applied this and sent a pull request. Made some minor changes
to commit message text.

Thanks,
Sekhar

^ permalink raw reply	[flat|nested] 10+ messages in thread

end of thread, other threads:[~2017-04-12 11:27 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-04-05 17:17 [PATCH v2] ARM: davinci: Add the clock for the CPPI 4.1 DMA engine Alexandre Bailon
2017-04-05 17:17 ` Alexandre Bailon
2017-04-07 14:36 ` Sekhar Nori
2017-04-07 14:36   ` Sekhar Nori
     [not found]   ` <e941f7f4-1904-a0c5-3ef0-973477087b2b-l0cyMroinI0@public.gmane.org>
2017-04-07 16:15     ` Alexandre Bailon
2017-04-07 16:15       ` Alexandre Bailon
     [not found]       ` <dade7b1b-af0f-0f3e-26fc-4e845c451330-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>
2017-04-07 17:31         ` Alexandre Bailon
2017-04-07 17:31           ` Alexandre Bailon
2017-04-12 11:27           ` Sekhar Nori
2017-04-12 11:27             ` 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.