2016-03-22 4:39 GMT+08:00 Timur Tabi <timur@xxxxxxxxxxxxxx>: > Jun Nie wrote: >> >> @@ -2570,9 +2608,17 @@ static int pl011_uart_plat_probe(struct >> platform_device *pdev) >> if (!uap) >> return -ENOMEM; >> >> + uap->clk = devm_clk_get(&pdev->dev, NULL); >> + if (IS_ERR(uap->clk)) >> + return PTR_ERR(uap->clk); >> + > > > I don't this is supposed to be part of your patch, and it breaks my ACPI > platform anyway. Where did it come from? > Is it OK to move to device tree part, I mean if (pdev->dev.of_node) clause? It is part of my patch, I am not aware ACPI requirement on this. Jun > -- > Qualcomm Innovation Center, Inc. > The Qualcomm Innovation Center, Inc. is a member of the Code Aurora > Forum, a Linux Foundation collaborative project. -- 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
