| Message ID | 20191025073056.25450-1-peter.ujfalusi@ti.com |
|---|---|
| Headers | show |
| Series | dmaengine: bindings/edma: dma-channel-mask to array | expand |
On 25-10-19, 10:30, Peter Ujfalusi wrote: > Hi, > > Changes since v4: > - Rebased on next to make it apply cleanly > - Added Reviewed-by from Rob for the DT documentation patches > > Changes since v3: > - Update the dma-common.yaml and edma binding documentation according to Rob's > suggestion > > Changes since v2: > - Fix dma-common.yaml documentation patch and extend the description of the > dma-channel-mask array > - The edma documentation now includes information on the dma-channel-mask array > size for EDMAs with 32 or 64 channels > > Changes since v1: > - Extend the common dma-channel-mask to uint32-array to be usable for > controllers with more than 32 channels > - Use the dma-channel-mask instead custom property for available channels for > EDMA. > > The original patch was part of the EDMA multicore usage series. > > EDMAs can have 32 or 64 channels depending on the SoC, the dma-channel-mask > needs to be an array to be usable for the driver. And now I saw this and have applied these and dropped the ones I fixed up manually! -- ~Vinod
On 05/11/2019 19.01, Vinod Koul wrote: > On 25-10-19, 10:30, Peter Ujfalusi wrote: >> Hi, >> >> Changes since v4: >> - Rebased on next to make it apply cleanly >> - Added Reviewed-by from Rob for the DT documentation patches >> >> Changes since v3: >> - Update the dma-common.yaml and edma binding documentation according to Rob's >> suggestion >> >> Changes since v2: >> - Fix dma-common.yaml documentation patch and extend the description of the >> dma-channel-mask array >> - The edma documentation now includes information on the dma-channel-mask array >> size for EDMAs with 32 or 64 channels >> >> Changes since v1: >> - Extend the common dma-channel-mask to uint32-array to be usable for >> controllers with more than 32 channels >> - Use the dma-channel-mask instead custom property for available channels for >> EDMA. >> >> The original patch was part of the EDMA multicore usage series. >> >> EDMAs can have 32 or 64 channels depending on the SoC, the dma-channel-mask >> needs to be an array to be usable for the driver. > > And now I saw this and have applied these and dropped the ones I fixed > up manually! OK, thank you for picking these up! - Péter Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki
Hi, Changes since v4: - Rebased on next to make it apply cleanly - Added Reviewed-by from Rob for the DT documentation patches Changes since v3: - Update the dma-common.yaml and edma binding documentation according to Rob's suggestion Changes since v2: - Fix dma-common.yaml documentation patch and extend the description of the dma-channel-mask array - The edma documentation now includes information on the dma-channel-mask array size for EDMAs with 32 or 64 channels Changes since v1: - Extend the common dma-channel-mask to uint32-array to be usable for controllers with more than 32 channels - Use the dma-channel-mask instead custom property for available channels for EDMA. The original patch was part of the EDMA multicore usage series. EDMAs can have 32 or 64 channels depending on the SoC, the dma-channel-mask needs to be an array to be usable for the driver. Regards, Peter --- Peter Ujfalusi (3): dt-bindings: dmaengine: dma-common: Change dma-channel-mask to uint32-array dt-bindings: dma: ti-edma: Document dma-channel-mask for EDMA dmaengine: ti: edma: Add support for handling reserved channels .../devicetree/bindings/dma/dma-common.yaml | 9 ++- .../devicetree/bindings/dma/ti-edma.txt | 8 +++ drivers/dma/ti/edma.c | 59 +++++++++++++++++-- 3 files changed, 69 insertions(+), 7 deletions(-) -- Peter Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki