On 01/17/2019 11:38 AM, Anup Patel wrote:
This patch does following fixes in MACB ethernet driver
for using it on RISC-V systems (particularly QEMU sifive_u
machine):
1. asm/arch/clk.h is not available on RISC-V port so include
    it only for non-RISC-V systems.
2. Don't fail in macb_enable_clk() if clk_enable() returns
    -ENOSYS because we get -ENOSYS for fixed-rate clocks.

Signed-off-by: Anup Patel <anup.pa...@wdc.com>
Reviewed-by: Bin Meng <bmeng...@gmail.com>
---
  drivers/net/macb.c | 4 +++-
  1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/drivers/net/macb.c b/drivers/net/macb.c
index 94c89c762b..9a06b523cc 100644
--- a/drivers/net/macb.c
+++ b/drivers/net/macb.c
@@ -38,7 +38,9 @@
  #include <linux/mii.h>
  #include <asm/io.h>
  #include <asm/dma-mapping.h>
+#ifndef CONFIG_RISCV
  #include <asm/arch/clk.h>
+#endif
  #include <linux/errno.h>
#include "macb.h"
@@ -1066,7 +1068,7 @@ static int macb_enable_clk(struct udevice *dev)
         */
  #ifndef CONFIG_MACB_ZYNQ
        ret = clk_enable(&clk);

If clk.h is not available, who exports clk_enable() then; and why is the included needed in the first place?


Alex

-       if (ret)
+       if (ret && ret != -ENOSYS)
                return ret;
  #endif


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

Reply via email to