| Date | Fri, 21 Oct 2022 12:24:35 +0300 (EEST) | From | Ilpo Järvinen <> | Subject | Re: [PATCH v4 4/4] tty: serial: 8250: add DFL bus driver for Altera 16550. |
| |
On Thu, 20 Oct 2022, matthew.gerlach@linux.intel.com wrote:
> From: Matthew Gerlach <matthew.gerlach@linux.intel.com> > > Add a Device Feature List (DFL) bus driver for the Altera > 16550 implementation of UART. > > Signed-off-by: Matthew Gerlach <matthew.gerlach@linux.intel.com> > ---
> + p = dfh_find_param(dfl_dev, DFHv1_PARAM_ID_REG_LAYOUT); > + if (!p) > + return dev_err_probe(dev, -EINVAL, "missing REG_LAYOUT param\n"); > + > + v = *p; > + uart->port.regshift = FIELD_GET(DFHv1_PARAM_ID_REG_SHIFT, v); > + reg_width = FIELD_GET(DFHv1_PARAM_ID_REG_WIDTH, v);
Just use *p directly for the FIELD_GETs. I suppose you can drop v after changing it.
Other than that and what Andy noted, this patch looks good.
-- i.
|