On Sat, Aug 23, 2014 at 11:04 PM, Roy Spliet <rspl...@eclipso.eu> wrote: > V2: always call post correctly even if pre fails > > Signed-off-by: Roy Spliet <rspl...@eclipso.eu> > --- > .../gpu/drm/nouveau/core/include/subdev/clock.h | 3 ++ > drivers/gpu/drm/nouveau/core/subdev/clock/nva3.c | 52 > +++++++++++++++++++++- > drivers/gpu/drm/nouveau/core/subdev/clock/nvaa.c | 36 +++++---------- > 3 files changed, 66 insertions(+), 25 deletions(-) > > diff --git a/drivers/gpu/drm/nouveau/core/include/subdev/clock.h > b/drivers/gpu/drm/nouveau/core/include/subdev/clock.h > index 676b49e..52e65b8 100644 > --- a/drivers/gpu/drm/nouveau/core/include/subdev/clock.h > +++ b/drivers/gpu/drm/nouveau/core/include/subdev/clock.h > @@ -146,6 +146,9 @@ int nv04_clock_pll_prog(struct nouveau_clock *, u32 reg1, > int nva3_clock_pll_calc(struct nouveau_clock *, struct nvbios_pll *, > int clk, struct nouveau_pll_vals *); > > +int nva3_clock_pre(struct nouveau_clock *clk, unsigned long *flags); > +void nva3_clock_post(struct nouveau_clock *clk, unsigned long *flags); I'd stick these in the subdev/clock/nva3.h that already exists.
> + > int nouveau_clock_ustate(struct nouveau_clock *, int req); This hunk doesn't apply as-is anyway, due to ustate() gaining an extra parameter. I've merged the rest of the patches. Thanks, Ben. > int nouveau_clock_astate(struct nouveau_clock *, int req, int rel); > int nouveau_clock_dstate(struct nouveau_clock *, int req, int rel); > diff --git a/drivers/gpu/drm/nouveau/core/subdev/clock/nva3.c > b/drivers/gpu/drm/nouveau/core/subdev/clock/nva3.c > index 14a5060..1d1915b 100644 > --- a/drivers/gpu/drm/nouveau/core/subdev/clock/nva3.c > +++ b/drivers/gpu/drm/nouveau/core/subdev/clock/nva3.c > @@ -23,6 +23,7 @@ > * Roy Spliet > */ > > +#include <engine/fifo.h> > #include <subdev/bios.h> > #include <subdev/bios/pll.h> > #include <subdev/timer.h> > @@ -293,6 +294,41 @@ calc_host(struct nva3_clock_priv *priv, struct > nouveau_cstate *cstate) > return ret; > } > > +int > +nva3_clock_pre(struct nouveau_clock *clk, unsigned long *flags) > +{ > + struct nouveau_fifo *pfifo = nouveau_fifo(clk); > + > + /* halt and idle execution engines */ > + nv_mask(clk, 0x020060, 0x00070000, 0x00000000); > + nv_mask(clk, 0x002504, 0x00000001, 0x00000001); > + /* Wait until the interrupt handler is finished */ > + if (!nv_wait(clk, 0x000100, 0xffffffff, 0x00000000)) > + return -EBUSY; > + > + if (pfifo) > + pfifo->pause(pfifo, flags); > + > + if (!nv_wait(clk, 0x002504, 0x00000010, 0x00000010)) > + return -EIO; > + if (!nv_wait(clk, 0x00251c, 0x0000003f, 0x0000003f)) > + return -EIO; > + > + return 0; > +} > + > +void > +nva3_clock_post(struct nouveau_clock *clk, unsigned long *flags) > +{ > + struct nouveau_fifo *pfifo = nouveau_fifo(clk); > + > + if (pfifo && flags) > + pfifo->start(pfifo, flags); > + > + nv_mask(clk, 0x002504, 0x00000001, 0x00000000); > + nv_mask(clk, 0x020060, 0x00070000, 0x00040000); > +} > + > static void > disable_clk_src(struct nva3_clock_priv *priv, u32 src) > { > @@ -421,6 +457,13 @@ nva3_clock_prog(struct nouveau_clock *clk) > { > struct nva3_clock_priv *priv = (void *)clk; > struct nva3_clock_info *core = &priv->eng[nv_clk_src_core]; > + int ret = 0; > + unsigned long flags; > + unsigned long *f = &flags; > + > + ret = nva3_clock_pre(clk, f); > + if (ret) > + goto out; > > if (core->pll) > prog_core(priv, nv_clk_src_core_intm); > @@ -430,7 +473,14 @@ nva3_clock_prog(struct nouveau_clock *clk) > prog_clk(priv, 0x20, nv_clk_src_disp); > prog_clk(priv, 0x21, nv_clk_src_vdec); > prog_host(priv); > - return 0; > + > +out: > + if (ret == -EBUSY) > + f = NULL; > + > + nva3_clock_post(clk, f); > + > + return ret; > } > > static void > diff --git a/drivers/gpu/drm/nouveau/core/subdev/clock/nvaa.c > b/drivers/gpu/drm/nouveau/core/subdev/clock/nvaa.c > index 6a65fc9..769ab26 100644 > --- a/drivers/gpu/drm/nouveau/core/subdev/clock/nvaa.c > +++ b/drivers/gpu/drm/nouveau/core/subdev/clock/nvaa.c > @@ -299,25 +299,14 @@ static int > nvaa_clock_prog(struct nouveau_clock *clk) > { > struct nvaa_clock_priv *priv = (void *)clk; > - struct nouveau_fifo *pfifo = nouveau_fifo(clk); > + u32 pllmask = 0, mast; > unsigned long flags; > - u32 pllmask = 0, mast, ptherm_gate; > - int ret = -EBUSY; > - > - /* halt and idle execution engines */ > - ptherm_gate = nv_mask(clk, 0x020060, 0x00070000, 0x00000000); > - nv_mask(clk, 0x002504, 0x00000001, 0x00000001); > - /* Wait until the interrupt handler is finished */ > - if (!nv_wait(clk, 0x000100, 0xffffffff, 0x00000000)) > - goto resume; > - > - if (pfifo) > - pfifo->pause(pfifo, &flags); > + unsigned long *f = &flags; > + int ret = 0; > > - if (!nv_wait(clk, 0x002504, 0x00000010, 0x00000010)) > - goto resume; > - if (!nv_wait(clk, 0x00251c, 0x0000003f, 0x0000003f)) > - goto resume; > + ret = nva3_clock_pre(clk, f); > + if (ret) > + goto out; > > /* First switch to safe clocks: href */ > mast = nv_mask(clk, 0xc054, 0x03400e70, 0x03400640); > @@ -375,15 +364,8 @@ nvaa_clock_prog(struct nouveau_clock *clk) > } > > nv_wr32(clk, 0xc054, mast); > - ret = 0; > > resume: > - if (pfifo) > - pfifo->start(pfifo, &flags); > - > - nv_mask(clk, 0x002504, 0x00000001, 0x00000000); > - nv_wr32(clk, 0x020060, ptherm_gate); > - > /* Disable some PLLs and dividers when unused */ > if (priv->csrc != nv_clk_src_core) { > nv_wr32(clk, 0x4040, 0x00000000); > @@ -395,6 +377,12 @@ resume: > nv_mask(clk, 0x4020, 0x80000000, 0x00000000); > } > > +out: > + if (ret == -EBUSY) > + f = NULL; > + > + nva3_clock_post(clk, f); > + > return ret; > } > > -- > 1.9.3 > > > _______________________________________________ Nouveau mailing list Nouveau@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/nouveau