On Thu, Feb 20, 2014 at 2:29 AM, Florian Fainelli [off-list ref] wrote:

2014-02-19 16:00 GMT-08:00 Jonas Gorski [off-list ref]:

quoted

On Thu, Feb 20, 2014 at 12:22 AM, Florian Fainelli [off-list ref] wrote:

quoted

@@ -857,6 +861,12 @@ static int bcm_uart_remove(struct platform_device *pdev)
        return 0;
 }

+static const struct of_device_id bcm63xx_of_match[] = {
+       { .compatible = "brcm,bcm63xx-uart" },

From my understanding, this should be "brcm,bcm6345-uart", because
this kind of uart appeared first on bcm6345 (well, maybe bcm6335, no
idea which one of these two was first, but the latter was never
supported in mainline anyway).

That's right, in fact, I think it might be desirable to handle both
compatible string, just as a hint that it is compatible with the
entire bcm63xx family. Would that work for you?

I think using a "generic" compatible string is rather frowned upon
(what do you do if there is eventually a bcm63xx chip with an
incompatible uart?), but I'm no device tree expert.


Jonas