On Wed, Mar 30, 2022 at 03:20:33PM +0200, Miquel Raynal wrote: > DW UART controllers can be synthesized without the CPR register. > In this case, allow to the platform information to provide a CPR value. ... > +#include <linux/property.h> Doesn't belong to this patch. ... > reg = dw8250_readl_ext(p, DW_UART_CPR); > + if (!reg) { > + reg = data->pdata->cpr; > + dev_dbg(p->dev, "CPR is not available, using 0x%08x instead\n", reg); > + } > + Redundant blank line. > if (!reg) > return; -- With Best Regards, Andy Shevchenko