[PATCH 5/5] shdmac: wire r8a7790

[PATCH 5/5] shdmac: wire r8a7790

[Ben Dooks]

Add support for R8a7790 with new device tree code

Note, does not yet moved the other of-dma user yet.
---
 drivers/dma/sh/shdmac.c | 15 +++++++++++----
 1 file changed, 11 insertions(+), 4 deletions(-)

diff --git a/drivers/dma/sh/shdmac.c b/drivers/dma/sh/shdmac.c
index 0d765c0..8af2082 100644
--- a/drivers/dma/sh/shdmac.c
+++ b/drivers/dma/sh/shdmac.c
@@ -677,7 +677,8 @@ static const struct shdma_ops sh_dmae_shdma_ops = {
 };
 
 static const struct of_device_id sh_dmae_of_match[] = {
-	{.compatible = "renesas,shdma-r8a73a4", .data = r8a73a4_shdma_devid,},
+	{ .compatible = "renesas,shdma-r8a73a4", .data = r8a73a4_shdma_devid,},
+	{ .compatible = "renesas,dma-r8a7790", .data = &shdma_arm_info },
 	{}
 };
 MODULE_DEVICE_TABLE(of, sh_dmae_of_match);
@@ -693,9 +694,15 @@ static int sh_dmae_probe(struct platform_device *pdev)
 	struct dma_device *dma_dev;
 	struct resource *chan, *dmars, *errirq_res, *chanirq_res;
 
-	if (pdev->dev.of_node)
-		pdata = of_match_device(sh_dmae_of_match, &pdev->dev)->data;
-	else
+	if (pdev->dev.of_node) {
+		const struct of_device_id *match;
+
+		match = of_match_device(sh_dmae_of_match, &pdev->dev);
+		if (match->data = r8a73a4_shdma_devid)
+			pdata = match->data;
+		else
+			pdata = sh_dma_probe_of(pdev, match);
+	} else
 		pdata = dev_get_platdata(&pdev->dev);
 
 	/* get platform data */
-- 
1.8.5.3

Re: [PATCH 5/5] shdmac: wire r8a7790

[Sergei Shtylyov]

Hello.

On 02/12/2014 08:59 PM, Ben Dooks wrote:

> Add support for R8a7790 with new device tree code

> Note, does not yet moved the other of-dma user yet.

    You didn't sign off.

> ---
>   drivers/dma/sh/shdmac.c | 15 +++++++++++----
>   1 file changed, 11 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/dma/sh/shdmac.c b/drivers/dma/sh/shdmac.c
> index 0d765c0..8af2082 100644
> --- a/drivers/dma/sh/shdmac.c
> +++ b/drivers/dma/sh/shdmac.c
> @@ -677,7 +677,8 @@ static const struct shdma_ops sh_dmae_shdma_ops = {
>   };
>
>   static const struct of_device_id sh_dmae_of_match[] = {
> -	{.compatible = "renesas,shdma-r8a73a4", .data = r8a73a4_shdma_devid,},
> +	{ .compatible = "renesas,shdma-r8a73a4", .data = r8a73a4_shdma_devid,},

    I think it's good idea to remove the tail comma the same as the line below 
doesn't have it.

> +	{ .compatible = "renesas,dma-r8a7790", .data = &shdma_arm_info },
>   	{}
>   };
>   MODULE_DEVICE_TABLE(of, sh_dmae_of_match);
> @@ -693,9 +694,15 @@ static int sh_dmae_probe(struct platform_device *pdev)
>   	struct dma_device *dma_dev;
>   	struct resource *chan, *dmars, *errirq_res, *chanirq_res;
>
> -	if (pdev->dev.of_node)
> -		pdata = of_match_device(sh_dmae_of_match, &pdev->dev)->data;
> -	else
> +	if (pdev->dev.of_node) {
> +		const struct of_device_id *match;
> +
> +		match = of_match_device(sh_dmae_of_match, &pdev->dev);
> +		if (match->data = r8a73a4_shdma_devid)
> +			pdata = match->data;
> +		else
> +			pdata = sh_dma_probe_of(pdev, match);
> +	} else
>   		pdata = dev_get_platdata(&pdev->dev);

     There should be {} on this atm as well now that you have {} in the other 
arm of *if*. See Documentation/CodingStyle.

WBR, Sergei

Re: [PATCH 5/5] shdmac: wire r8a7790

[Ben Dooks]

On 12/02/14 19:30, Sergei Shtylyov wrote:
> Hello.
>
> On 02/12/2014 08:59 PM, Ben Dooks wrote:
>
>> Add support for R8a7790 with new device tree code
>
>> Note, does not yet moved the other of-dma user yet.
>
>     You didn't sign off.

Sorry, this is still development, and didn't want people
to apply it until it had an initial review.

>> ---
>>   drivers/dma/sh/shdmac.c | 15 +++++++++++----
>>   1 file changed, 11 insertions(+), 4 deletions(-)
>>
>> diff --git a/drivers/dma/sh/shdmac.c b/drivers/dma/sh/shdmac.c
>> index 0d765c0..8af2082 100644
>> --- a/drivers/dma/sh/shdmac.c
>> +++ b/drivers/dma/sh/shdmac.c
>> @@ -677,7 +677,8 @@ static const struct shdma_ops sh_dmae_shdma_ops = {
>>   };
>>
>>   static const struct of_device_id sh_dmae_of_match[] = {
>> -    {.compatible = "renesas,shdma-r8a73a4", .data >> r8a73a4_shdma_devid,},
>> +    { .compatible = "renesas,shdma-r8a73a4", .data >> r8a73a4_shdma_devid,},
>
>     I think it's good idea to remove the tail comma the same as the line
> below doesn't have it.
>
>> +    { .compatible = "renesas,dma-r8a7790", .data = &shdma_arm_info },
>>       {}
>>   };
>>   MODULE_DEVICE_TABLE(of, sh_dmae_of_match);
>> @@ -693,9 +694,15 @@ static int sh_dmae_probe(struct platform_device
>> *pdev)
>>       struct dma_device *dma_dev;
>>       struct resource *chan, *dmars, *errirq_res, *chanirq_res;
>>
>> -    if (pdev->dev.of_node)
>> -        pdata = of_match_device(sh_dmae_of_match, &pdev->dev)->data;
>> -    else
>> +    if (pdev->dev.of_node) {
>> +        const struct of_device_id *match;
>> +
>> +        match = of_match_device(sh_dmae_of_match, &pdev->dev);
>> +        if (match->data = r8a73a4_shdma_devid)
>> +            pdata = match->data;
>> +        else
>> +            pdata = sh_dma_probe_of(pdev, match);
>> +    } else
>>           pdata = dev_get_platdata(&pdev->dev);
>
>      There should be {} on this atm as well now that you have {} in the
> other arm of *if*. See Documentation/CodingStyle.
>
> WBR, Sergei

Thanks.


-- 
Ben Dooks				http://www.codethink.co.uk/
Senior Engineer				Codethink - Providing Genius

Re: [PATCH 5/5] shdmac: wire r8a7790

[Sergei Shtylyov]

On 02/12/2014 09:52 PM, Ben Dooks wrote:

>>> Add support for R8a7790 with new device tree code

>>> Note, does not yet moved the other of-dma user yet.

>>     You didn't sign off.

> Sorry, this is still development, and didn't want people
> to apply it until it had an initial review.

    In this case, it might have been worth marking them as RFC (request for 
comments) in the subject. RFC patches are not applied.

[...]

>> WBR, Sergei

> Thanks.

    Oh, not at all.

WBR, Sergei