Messages in this thread | | | Subject | Re: [PATCH 1/8] drm/mediatek: Use regmap for register access | From | Philipp Zabel <> | Date | Fri, 20 Apr 2018 12:00:19 +0200 |
| |
Hi Matthias,
On Fri, 2018-04-20 at 11:41 +0200, Matthias Brugger wrote: > Hi Philipp, > > On 11/23/2017 09:54 AM, Philipp Zabel wrote: > > Hi Matthias, > > > > On Tue, 2017-11-14 at 22:41 +0100, Matthias Brugger wrote: > > > The mmsys memory space is shared between the drm and the > > > clk driver. Use regmap to access it. > > > > > > Signed-off-by: Matthias Brugger <mbrugger@suse.com> > > > --- > > > drivers/gpu/drm/mediatek/mtk_drm_crtc.c | 4 ++-- > > > drivers/gpu/drm/mediatek/mtk_drm_ddp.c | 30 +++++++++++++++++------------- > > > drivers/gpu/drm/mediatek/mtk_drm_ddp.h | 4 ++-- > > > drivers/gpu/drm/mediatek/mtk_drm_drv.c | 13 ++++--------- > > > drivers/gpu/drm/mediatek/mtk_drm_drv.h | 2 +- > > > 5 files changed, 26 insertions(+), 27 deletions(-) > > > > [...] > > [...] > > > } > > > > > > value = mtk_ddp_sel_in(cur, next, &addr); > > > if (value) { > > > - reg = readl_relaxed(config_regs + addr) & ~value; > > > - writel_relaxed(reg, config_regs + addr); > > > + regmap_read(config_regs, addr, ®); > > > + reg &= ~value; > > > + regmap_write(config_regs, addr, reg); > > > > regmap_update_bits(config_regs, addr, value, 0); > > > > Reviewed-by: Philipp Zabel <p.zabel@pengutronix.de> > > > > Thanks for having a look on that. > > I'll update the next version with regmap_update_bits and leave your Reviewed-by, > hope that's ok.
Yes, that's fine.
regards Philipp
| |