On 1/9/20 1:02 PM, Masahiro Yamada wrote:
The "nand_x" and "ecc" clocks are currently optional. Make the core clock optional in the same way. This will allow platforms with no clock driver support to use this driver.
Signed-off-by: Masahiro Yamada yamada.masahiro@socionext.com
drivers/mtd/nand/raw/denali_dt.c | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-)
diff --git a/drivers/mtd/nand/raw/denali_dt.c b/drivers/mtd/nand/raw/denali_dt.c index 0ce81324b90e..b1e14982c443 100644 --- a/drivers/mtd/nand/raw/denali_dt.c +++ b/drivers/mtd/nand/raw/denali_dt.c @@ -91,7 +91,7 @@ static int denali_dt_probe(struct udevice *dev) if (ret) ret = clk_get_by_index(dev, 0, &clk); if (ret)
return ret;
clk.dev = NULL;ret = clk_get_by_name(dev, "nand_x", &clk_x); if (ret)
@@ -101,9 +101,11 @@ static int denali_dt_probe(struct udevice *dev) if (ret) clk_ecc.dev = NULL;
- ret = clk_enable(&clk);
- if (ret)
return ret;
if (clk.dev) {
ret = clk_enable(&clk);if (ret)return ret;}
if (clk_x.dev) { ret = clk_enable(&clk_x);
Why do we need any of the clock code if the clock aren't available ? It just takes space for no reason.