On Wed, May 31, 2017 at 11:21 PM, Thomas Bogendoerfer <tsbogend@xxxxxxxxxxxxxxxx> wrote: > sccnxp driver doesn't get the correct uart clock rate, if CONFIG_HAVE_CLOCK > is disabled. Correct usage of clk API to make it work with/without it. uart -> UART clk -> CLK > clk = devm_clk_get(&pdev->dev, NULL); > if (IS_ERR(clk)) { > - if (PTR_ERR(clk) == -EPROBE_DEFER) { > - ret = -EPROBE_DEFER; > + ret = PTR_ERR(clk); > + if (ret == -EPROBE_DEFER) > goto err_out; > - } > + uartclk = 0; > + } else { > + clk_prepare_enable(clk); This might fail. > + uartclk = clk_get_rate(clk); > + } -- With Best Regards, Andy Shevchenko -- To unsubscribe from this list: send the line "unsubscribe linux-serial" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html