On 5/26/19 2:13 AM, Paul Walmsley wrote:
The U-Boot PRCI driver for the SiFive FU540 uses an out-of-date DT
binding string, since the U-boot PRCI driver was upstreamed before the
mainline Linux kernel PRCI driver was finished.  This means that the
U-Boot PRCI driver won't probe when used with a DT file that is
aligned to the DT bindings and the driver in the Linux kernel:

    
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/devicetree/bindings/clock/sifive/fu540-prci.txt#n7

    
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/clk/sifive/fu540-prci.c#n610

Fix by aligning the U-Boot DT match string to the string that's used
in the upstream DT bindings and the Linux kernel driver.


Signed-off-by: Paul Walmsley <paul.walms...@sifive.com>
Signed-off-by: Paul Walmsley <p...@pwsan.com>
Cc: Anup Patel <anup.pa...@wdc.com>
Cc: Atish Patra <atish.pa...@wdc.com>
Cc: Alexander Graf <ag...@suse.de>
---
  drivers/clk/sifive/fu540-prci.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/clk/sifive/fu540-prci.c b/drivers/clk/sifive/fu540-prci.c
index 2d47ebc6b1..d79d1a5351 100644
--- a/drivers/clk/sifive/fu540-prci.c
+++ b/drivers/clk/sifive/fu540-prci.c
@@ -589,7 +589,7 @@ static struct clk_ops sifive_fu540_prci_ops = {
  };
static const struct udevice_id sifive_fu540_prci_ids[] = {
-       { .compatible = "sifive,fu540-c000-prci0" },
+       { .compatible = "sifive,fu540-c000-prci" },
        { .compatible = "sifive,aloeprci0" },
        { }
  };

Reviewed-by: Atish Patra <atish.pa...@wdc.com>

--
Regards,
Atish
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to