On Mon, Jan 14, 2013 at 03:04:18PM +1300, Tony Prisk wrote:
> - if (vt8500_port->clk) {
> - vt8500_port->uart.uartclk = clk_get_rate(vt8500_port->clk);
> - } else {
> - /* use the default of 24Mhz if not specified and warn */
> - pr_warn("%s: serial clock source not specified\n", __func__);
> - vt8500_port->uart.uartclk = 24000000;
> + if (IS_ERR(vt8500_port->clk)) {
> + dev_err(&pdev->dev, "failed to get clock\n");
> + ret = -EINVAL;
> + goto err;
> + }
> +
> + ret = clk_prepare_enable(vt8500_port->clk);
> + if (ret) {
> + dev_err(&pdev->dev, "failed to enable clock\n");
> + goto err;
Looks like you get rid of the initialization of vt8500_port->uart.uartclk
here. Was that intentional?
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
