[PATCH v5 03/10] gna: read hardware info

Linus Walleij linus.walleij at linaro.org
Fri Oct 21 09:19:51 UTC 2022


Hi Maciej,

thanks for your patch!

Overall I really appreciate the attempt to use DRM, this looks so much
smoother than the previous iterations.

On Thu, Oct 20, 2022 at 7:57 PM Maciej Kwapulinski
<maciej.kwapulinski at linux.intel.com> wrote:
>
> From: Tomasz Jankowski <tomasz1.jankowski at intel.com>
>
> Get the hardware information from register MMIO_IBUFFS
>
> Signed-off-by: Tomasz Jankowski <tomasz1.jankowski at intel.com>
> Tested-by: Mikolaj Grzybowski <mikolajx.grzybowski at intel.com>
> Co-developed-by: Jianxun Zhang <jianxun.zhang at linux.intel.com>
> Signed-off-by: Jianxun Zhang <jianxun.zhang at linux.intel.com>
> Co-developed-by: Maciej Kwapulinski <maciej.kwapulinski at linux.intel.com>
> Signed-off-by: Maciej Kwapulinski <maciej.kwapulinski at linux.intel.com>

> +static inline u32 gna_reg_read(struct gna_device *gna_priv, u32 reg)
> +{
> +       return readl(gna_priv->iobase + reg);
> +}

If you are consistently using functions like this to read/write registers,
you are just re-implmeneting regmap-mmio, see
drivers/base/regmap/regmap-mmio.c
or look in drivers selecting REGMAP_MMIO and the neat tricks
they do to read/write and update registers with regmap.

Yours,
Linus Walleij


More information about the dri-devel mailing list