diff mbox series

dmaengine: ti: k3-udma: Use correct node to read "ti,udma-atype"

Message ID 20200527065357.30791-1-peter.ujfalusi@ti.com (mailing list archive)
State Accepted
Headers show
Series dmaengine: ti: k3-udma: Use correct node to read "ti,udma-atype" | expand

Commit Message

Peter Ujfalusi May 27, 2020, 6:53 a.m. UTC
The "ti,udma-atype" property is expected in the UDMA node and not in the
parent navss node.

Fixes: 0ebcf1a274c5 ("dmaengine: ti: k3-udma: Implement support for atype (for virtualization)")

Signed-off-by: Peter Ujfalusi <peter.ujfalusi@ti.com>
---
 drivers/dma/ti/k3-udma.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Vinod Koul June 16, 2020, 3:54 p.m. UTC | #1
On 27-05-20, 09:53, Peter Ujfalusi wrote:
> The "ti,udma-atype" property is expected in the UDMA node and not in the
> parent navss node.
> 
> Fixes: 0ebcf1a274c5 ("dmaengine: ti: k3-udma: Implement support for atype (for virtualization)")
> 

No empty line b/w fixes and s-o-b please

> Signed-off-by: Peter Ujfalusi <peter.ujfalusi@ti.com>

Fixed it up while applying
diff mbox series

Patch

diff --git a/drivers/dma/ti/k3-udma.c b/drivers/dma/ti/k3-udma.c
index c91e2dc1bb72..4bc470eb4143 100644
--- a/drivers/dma/ti/k3-udma.c
+++ b/drivers/dma/ti/k3-udma.c
@@ -3593,7 +3593,7 @@  static int udma_probe(struct platform_device *pdev)
 		return ret;
 	}
 
-	ret = of_property_read_u32(navss_node, "ti,udma-atype", &ud->atype);
+	ret = of_property_read_u32(dev->of_node, "ti,udma-atype", &ud->atype);
 	if (!ret && ud->atype > 2) {
 		dev_err(dev, "Invalid atype: %u\n", ud->atype);
 		return -EINVAL;