dmaengine Archive on lore.kernel.org
 help / color / Atom feed
* [PATCH] dt-bindings: dmaengine: dma-common: Revise the dma-channel-mask property
@ 2019-08-28  6:39 Yoshihiro Shimoda
  2019-08-28  7:25 ` Peter Ujfalusi
  2019-08-28  7:30 ` Geert Uytterhoeven
  0 siblings, 2 replies; 7+ messages in thread
From: Yoshihiro Shimoda @ 2019-08-28  6:39 UTC (permalink / raw)
  To: vkoul, robh+dt, mark.rutland; +Cc: dmaengine, devicetree, Yoshihiro Shimoda

The commit b37e3534ac42 ("dt-bindings: dmaengine: Add YAML schemas
for the generic DMA bindings") changed the property from
dma-channel-mask to dma-channel-masks. So, this patch revises it.

Fixes: b37e3534ac42 ("dt-bindings: dmaengine: Add YAML schemas for the generic DMA bindings")
Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
---
 Documentation/devicetree/bindings/dma/dma-common.yaml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/dma/dma-common.yaml b/Documentation/devicetree/bindings/dma/dma-common.yaml
index 0141af0..ed0a49a 100644
--- a/Documentation/devicetree/bindings/dma/dma-common.yaml
+++ b/Documentation/devicetree/bindings/dma/dma-common.yaml
@@ -24,7 +24,7 @@ properties:
     description:
       Used to provide DMA controller specific information.
 
-  dma-channel-masks:
+  dma-channel-mask:
     $ref: /schemas/types.yaml#definitions/uint32
     description:
       Bitmask of available DMA channels in ascending order that are
-- 
2.7.4


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

* Re: [PATCH] dt-bindings: dmaengine: dma-common: Revise the dma-channel-mask property
  2019-08-28  6:39 [PATCH] dt-bindings: dmaengine: dma-common: Revise the dma-channel-mask property Yoshihiro Shimoda
@ 2019-08-28  7:25 ` Peter Ujfalusi
  2019-08-28  8:55   ` Yoshihiro Shimoda
  2019-08-28  7:30 ` Geert Uytterhoeven
  1 sibling, 1 reply; 7+ messages in thread
From: Peter Ujfalusi @ 2019-08-28  7:25 UTC (permalink / raw)
  To: Yoshihiro Shimoda, vkoul, robh+dt, mark.rutland; +Cc: dmaengine, devicetree



On 28/08/2019 9.39, Yoshihiro Shimoda wrote:
> The commit b37e3534ac42 ("dt-bindings: dmaengine: Add YAML schemas
> for the generic DMA bindings") changed the property from
> dma-channel-mask to dma-channel-masks. So, this patch revises it.
> 
> Fixes: b37e3534ac42 ("dt-bindings: dmaengine: Add YAML schemas for the generic DMA bindings")
> Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
> ---
>  Documentation/devicetree/bindings/dma/dma-common.yaml | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/dma/dma-common.yaml b/Documentation/devicetree/bindings/dma/dma-common.yaml
> index 0141af0..ed0a49a 100644
> --- a/Documentation/devicetree/bindings/dma/dma-common.yaml
> +++ b/Documentation/devicetree/bindings/dma/dma-common.yaml
> @@ -24,7 +24,7 @@ properties:
>      description:
>        Used to provide DMA controller specific information.
>  
> -  dma-channel-masks:
> +  dma-channel-mask:
>      $ref: /schemas/types.yaml#definitions/uint32

How this mask supposed to be used for controllers having more than 32
channels (64, 300+)?

>      description:
>        Bitmask of available DMA channels in ascending order that are
> 

- Péter

Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki.
Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki

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

* Re: [PATCH] dt-bindings: dmaengine: dma-common: Revise the dma-channel-mask property
  2019-08-28  6:39 [PATCH] dt-bindings: dmaengine: dma-common: Revise the dma-channel-mask property Yoshihiro Shimoda
  2019-08-28  7:25 ` Peter Ujfalusi
@ 2019-08-28  7:30 ` Geert Uytterhoeven
  2019-08-28  8:57   ` Yoshihiro Shimoda
  1 sibling, 1 reply; 7+ messages in thread
From: Geert Uytterhoeven @ 2019-08-28  7:30 UTC (permalink / raw)
  To: Yoshihiro Shimoda
  Cc: Vinod, Rob Herring, Mark Rutland, dmaengine,
	open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS

Hi Shimoda-san,

I would s/Revise/Fix/ in the subject.

On Wed, Aug 28, 2019 at 8:41 AM Yoshihiro Shimoda
<yoshihiro.shimoda.uh@renesas.com> wrote:
> The commit b37e3534ac42 ("dt-bindings: dmaengine: Add YAML schemas
> for the generic DMA bindings") changed the property from
> dma-channel-mask to dma-channel-masks. So, this patch revises it.
>
> Fixes: b37e3534ac42 ("dt-bindings: dmaengine: Add YAML schemas for the generic DMA bindings")
> Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>

Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

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

* RE: [PATCH] dt-bindings: dmaengine: dma-common: Revise the dma-channel-mask property
  2019-08-28  7:25 ` Peter Ujfalusi
@ 2019-08-28  8:55   ` Yoshihiro Shimoda
  2019-08-28  9:38     ` Peter Ujfalusi
  0 siblings, 1 reply; 7+ messages in thread
From: Yoshihiro Shimoda @ 2019-08-28  8:55 UTC (permalink / raw)
  To: Peter Ujfalusi, vkoul, robh+dt, mark.rutland; +Cc: dmaengine, devicetree

Hi Peter,

> From: Peter Ujfalusi, Sent: Wednesday, August 28, 2019 4:25 PM
> 
> On 28/08/2019 9.39, Yoshihiro Shimoda wrote:
> > The commit b37e3534ac42 ("dt-bindings: dmaengine: Add YAML schemas
> > for the generic DMA bindings") changed the property from
> > dma-channel-mask to dma-channel-masks. So, this patch revises it.
> >
> > Fixes: b37e3534ac42 ("dt-bindings: dmaengine: Add YAML schemas for the generic DMA bindings")
> > Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
> > ---
> >  Documentation/devicetree/bindings/dma/dma-common.yaml | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/Documentation/devicetree/bindings/dma/dma-common.yaml
> b/Documentation/devicetree/bindings/dma/dma-common.yaml
> > index 0141af0..ed0a49a 100644
> > --- a/Documentation/devicetree/bindings/dma/dma-common.yaml
> > +++ b/Documentation/devicetree/bindings/dma/dma-common.yaml
> > @@ -24,7 +24,7 @@ properties:
> >      description:
> >        Used to provide DMA controller specific information.
> >
> > -  dma-channel-masks:
> > +  dma-channel-mask:
> >      $ref: /schemas/types.yaml#definitions/uint32
> 
> How this mask supposed to be used for controllers having more than 32
> channels (64, 300+)?

I found "dma-channels" property as 40 in arch/arm/boot/dts/ste-u300.dts.
However, since arch/arm64/boot/dts/hisilicon/hi3660.dtsi already has
the dma-channel-mask property, I think we should not change the property name.

Best regards,
Yoshihiro Shimoda

> >      description:
> >        Bitmask of available DMA channels in ascending order that are
> >
> 
> - Péter
> 
> Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki.
> Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki

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

* RE: [PATCH] dt-bindings: dmaengine: dma-common: Revise the dma-channel-mask property
  2019-08-28  7:30 ` Geert Uytterhoeven
@ 2019-08-28  8:57   ` Yoshihiro Shimoda
  0 siblings, 0 replies; 7+ messages in thread
From: Yoshihiro Shimoda @ 2019-08-28  8:57 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: Vinod, Rob Herring, Mark Rutland, dmaengine,
	open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS

Hi Geert-san,

> From: Geert Uytterhoeven, Sent: Wednesday, August 28, 2019 4:30 PM
> 
> Hi Shimoda-san,
> 
> I would s/Revise/Fix/ in the subject.

I got it. I'll submit v2 patch.

> On Wed, Aug 28, 2019 at 8:41 AM Yoshihiro Shimoda
> <yoshihiro.shimoda.uh@renesas.com> wrote:
> > The commit b37e3534ac42 ("dt-bindings: dmaengine: Add YAML schemas
> > for the generic DMA bindings") changed the property from
> > dma-channel-mask to dma-channel-masks. So, this patch revises it.

Also, I think this line should be s/revises/fixes/.

> > Fixes: b37e3534ac42 ("dt-bindings: dmaengine: Add YAML schemas for the generic DMA bindings")
> > Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
> 
> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>

Thank you for your review!

Best regards,
Yoshihiro Shimoda

> Gr{oetje,eeting}s,
> 
>                         Geert
> 
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
> 
> In personal conversations with technical people, I call myself a hacker. But
> when I'm talking to journalists I just say "programmer" or something like that.
>                                 -- Linus Torvalds

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

* Re: [PATCH] dt-bindings: dmaengine: dma-common: Revise the dma-channel-mask property
  2019-08-28  8:55   ` Yoshihiro Shimoda
@ 2019-08-28  9:38     ` Peter Ujfalusi
  2019-08-28 10:00       ` Yoshihiro Shimoda
  0 siblings, 1 reply; 7+ messages in thread
From: Peter Ujfalusi @ 2019-08-28  9:38 UTC (permalink / raw)
  To: Yoshihiro Shimoda, vkoul, robh+dt, mark.rutland; +Cc: dmaengine, devicetree

Hi Shimoda-san,

On 28/08/2019 11.55, Yoshihiro Shimoda wrote:
> Hi Peter,
> 
>> From: Peter Ujfalusi, Sent: Wednesday, August 28, 2019 4:25 PM
>>
>> On 28/08/2019 9.39, Yoshihiro Shimoda wrote:
>>> The commit b37e3534ac42 ("dt-bindings: dmaengine: Add YAML schemas
>>> for the generic DMA bindings") changed the property from
>>> dma-channel-mask to dma-channel-masks. So, this patch revises it.
>>>
>>> Fixes: b37e3534ac42 ("dt-bindings: dmaengine: Add YAML schemas for the generic DMA bindings")
>>> Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
>>> ---
>>>  Documentation/devicetree/bindings/dma/dma-common.yaml | 2 +-
>>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/Documentation/devicetree/bindings/dma/dma-common.yaml
>> b/Documentation/devicetree/bindings/dma/dma-common.yaml
>>> index 0141af0..ed0a49a 100644
>>> --- a/Documentation/devicetree/bindings/dma/dma-common.yaml
>>> +++ b/Documentation/devicetree/bindings/dma/dma-common.yaml
>>> @@ -24,7 +24,7 @@ properties:
>>>      description:
>>>        Used to provide DMA controller specific information.
>>>
>>> -  dma-channel-masks:
>>> +  dma-channel-mask:
>>>      $ref: /schemas/types.yaml#definitions/uint32
>>
>> How this mask supposed to be used for controllers having more than 32
>> channels (64, 300+)?
> 
> I found "dma-channels" property as 40 in arch/arm/boot/dts/ste-u300.dts.
> However, since arch/arm64/boot/dts/hisilicon/hi3660.dtsi already has
> the dma-channel-mask property, I think we should not change the property name.

I'm not asking it to be changed, I just wondered how I could use this
generic property for DMA controllers having more channels than u32
bitfield could describe. An array of multiple u32 to cover the number of
channels would probably something which can be done, but it would need
update for the documentation to make sure that it is used consistently.

I'm asking this because of: https://patchwork.kernel.org/patch/11111619/

> 
> Best regards,
> Yoshihiro Shimoda
> 
>>>      description:
>>>        Bitmask of available DMA channels in ascending order that are
>>>
>>
>> - Péter
>>
>> Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki.
>> Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki

- Péter

Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki.
Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki

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

* RE: [PATCH] dt-bindings: dmaengine: dma-common: Revise the dma-channel-mask property
  2019-08-28  9:38     ` Peter Ujfalusi
@ 2019-08-28 10:00       ` Yoshihiro Shimoda
  0 siblings, 0 replies; 7+ messages in thread
From: Yoshihiro Shimoda @ 2019-08-28 10:00 UTC (permalink / raw)
  To: Peter Ujfalusi, vkoul, robh+dt, mark.rutland; +Cc: dmaengine, devicetree

Hi Peter-san,

> From: Peter Ujfalusi, Sent: Wednesday, August 28, 2019 6:39 PM
> 
> Hi Shimoda-san,
> 
> On 28/08/2019 11.55, Yoshihiro Shimoda wrote:
> > Hi Peter,
> >
> >> From: Peter Ujfalusi, Sent: Wednesday, August 28, 2019 4:25 PM
> >>
> >> On 28/08/2019 9.39, Yoshihiro Shimoda wrote:
> >>> The commit b37e3534ac42 ("dt-bindings: dmaengine: Add YAML schemas
> >>> for the generic DMA bindings") changed the property from
> >>> dma-channel-mask to dma-channel-masks. So, this patch revises it.
> >>>
> >>> Fixes: b37e3534ac42 ("dt-bindings: dmaengine: Add YAML schemas for the generic DMA bindings")
> >>> Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
> >>> ---
> >>>  Documentation/devicetree/bindings/dma/dma-common.yaml | 2 +-
> >>>  1 file changed, 1 insertion(+), 1 deletion(-)
> >>>
> >>> diff --git a/Documentation/devicetree/bindings/dma/dma-common.yaml
> >> b/Documentation/devicetree/bindings/dma/dma-common.yaml
> >>> index 0141af0..ed0a49a 100644
> >>> --- a/Documentation/devicetree/bindings/dma/dma-common.yaml
> >>> +++ b/Documentation/devicetree/bindings/dma/dma-common.yaml
> >>> @@ -24,7 +24,7 @@ properties:
> >>>      description:
> >>>        Used to provide DMA controller specific information.
> >>>
> >>> -  dma-channel-masks:
> >>> +  dma-channel-mask:
> >>>      $ref: /schemas/types.yaml#definitions/uint32
> >>
> >> How this mask supposed to be used for controllers having more than 32
> >> channels (64, 300+)?
> >
> > I found "dma-channels" property as 40 in arch/arm/boot/dts/ste-u300.dts.
> > However, since arch/arm64/boot/dts/hisilicon/hi3660.dtsi already has
> > the dma-channel-mask property, I think we should not change the property name.
> 
> I'm not asking it to be changed, I just wondered how I could use this
> generic property for DMA controllers having more channels than u32
> bitfield could describe. An array of multiple u32 to cover the number of
> channels would probably something which can be done, but it would need
> update for the documentation to make sure that it is used consistently.
> 
> I'm asking this because of: https://patchwork.kernel.org/patch/11111619/

Oops, I'm sorry for misunderstanding your question.

As you said, I also think that you can use this generic property if we update
the document from uint32 to uint32-array like below:

-      $ref: /schemas/types.yaml#definitions/uint32
+      $ref: /schemas/types.yaml#definitions/uint32-array

Best regards,
Yoshihiro Shimoda

> >
> > Best regards,
> > Yoshihiro Shimoda
> >
> >>>      description:
> >>>        Bitmask of available DMA channels in ascending order that are
> >>>
> >>
> >> - Péter
> >>
> >> Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki.
> >> Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki
> 
> - Péter
> 
> Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki.
> Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki

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

end of thread, back to index

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-08-28  6:39 [PATCH] dt-bindings: dmaengine: dma-common: Revise the dma-channel-mask property Yoshihiro Shimoda
2019-08-28  7:25 ` Peter Ujfalusi
2019-08-28  8:55   ` Yoshihiro Shimoda
2019-08-28  9:38     ` Peter Ujfalusi
2019-08-28 10:00       ` Yoshihiro Shimoda
2019-08-28  7:30 ` Geert Uytterhoeven
2019-08-28  8:57   ` Yoshihiro Shimoda

dmaengine Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/dmaengine/0 dmaengine/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 dmaengine dmaengine/ https://lore.kernel.org/dmaengine \
		dmaengine@vger.kernel.org dmaengine@archiver.kernel.org
	public-inbox-index dmaengine

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.vger.dmaengine


AGPL code for this site: git clone https://public-inbox.org/ public-inbox