Re: [PATCH v2 2/2] ASoC: fsl: Add generic CPU DAI driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 3/6/20 12:13 PM, Daniel Baluta wrote:
+static int fsl_dai_probe(struct platform_device *pdev)
+{
[...]
+	ret = of_property_read_u32(np, "fsl,dai-index", &dai_index);
+	if (ret) {
+		dev_err(&pdev->dev, "dai-index missing or invalid\n");
+		return ret;
+	}
Maybe this can follow a more standard approach using DT aliases. Just like we assign IDs to things like SPI or I2C masters.
+
+	fsl_dai.name = dai_name;
This breaks as soon as there is more than one DAI in the system since you are sharing a global struct between them.
[...]



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux