Hi, Sakari, Thanks for the feedback.
> -----Original Message----- > From: Sakari Ailus [mailto:sakari.ai...@linux.intel.com] > Sent: Monday, November 5, 2018 3:55 AM > To: Zhi, Yong <yong....@intel.com> > Cc: linux-media@vger.kernel.org; tf...@chromium.org; > mche...@kernel.org; hans.verk...@cisco.com; > laurent.pinch...@ideasonboard.com; Mani, Rajmohan > <rajmohan.m...@intel.com>; Zheng, Jian Xu <jian.xu.zh...@intel.com>; Hu, > Jerry W <jerry.w...@intel.com>; Toivonen, Tuukka > <tuukka.toivo...@intel.com>; Qiu, Tian Shu <tian.shu....@intel.com>; Cao, > Bingbu <bingbu....@intel.com> > Subject: Re: [PATCH v7 06/16] intel-ipu3: mmu: Implement driver > > Hi Yong, > > On Mon, Oct 29, 2018 at 03:23:00PM -0700, Yong Zhi wrote: > > From: Tomasz Figa <tf...@chromium.org> > > > > This driver translates IO virtual address to physical address based on > > two levels page tables. > > > > Signed-off-by: Tomasz Figa <tf...@chromium.org> > > Signed-off-by: Yong Zhi <yong....@intel.com> > > --- > > ... > > > +static void call_if_ipu3_is_powered(struct ipu3_mmu *mmu, > > + void (*func)(struct ipu3_mmu *mmu)) { > > + pm_runtime_get_noresume(mmu->dev); > > + if (pm_runtime_active(mmu->dev)) > > + func(mmu); > > + pm_runtime_put(mmu->dev); > > How about: > > if (!pm_runtime_get_if_in_use(mmu->dev)) > return; > > func(mmu); > pm_runtime_put(mmu->dev); > Ack, unless Tomasz has different opinion. > > > +} > > -- > Sakari Ailus > sakari.ai...@linux.intel.com
