Re: [Intel-gfx] [PATCH v2 1/1] drm/i915: Do RPM Wake during GuC/HuC status read

2017-02-07 Thread Arkadiusz Hiler
On Mon, Feb 06, 2017 at 10:34:31AM +0530, Kamble, Sagar A wrote: > > > On 2/4/2017 7:40 PM, Arkadiusz Hiler wrote: > > On Fri, Feb 03, 2017 at 01:58:33PM +0530, Sagar Arun Kamble wrote: > > > HUC_STATUS, GUC_STATUS, SOFT_SCRATCH registers are read in debugfs > > &g

Re: [Intel-gfx] [PATCH v2 1/1] drm/i915: Do RPM Wake during GuC/HuC status read

2017-02-04 Thread Arkadiusz Hiler
isW) > > Cc: Arkadiusz Hiler <arkadiusz.hi...@intel.com> > Cc: Anusha Srivatsa <anusha.sriva...@intel.com> > Cc: Fiedorowicz, Lukasz <lukasz.fiedorow...@intel.com> > Signed-off-by: Sagar Arun Kamble <sagar.a.kam...@intel.com> > --- > drivers/gpu/drm/i915/i91

Re: [Intel-gfx] [PATCH 2/8] drm/i915/uc: Drop superfluous externs in intel_uc.h

2017-02-21 Thread Arkadiusz Hiler
On Fri, Feb 17, 2017 at 02:13:06PM +0100, Michal Wajdeczko wrote: > On Fri, Feb 17, 2017 at 02:05:51PM +0100, Arkadiusz Hiler wrote: > > I think one line with description will not kill us ;) Comment title is exactly that. If you have something that won't be just simple restatement of

Re: [Intel-gfx] [PATCH 5/8] drm/i915/uc: Make intel_uc_fw_fetch() static

2017-02-21 Thread Arkadiusz Hiler
On Fri, Feb 17, 2017 at 02:39:35PM +0100, Michal Wajdeczko wrote: > On Fri, Feb 17, 2017 at 02:05:54PM +0100, Arkadiusz Hiler wrote: > > intel_uc_fw_fetch() is confusingly named in the light of recent changes. > > > > It's also in the worng place - 'guc_loader.h' - i

Re: [Intel-gfx] [PATCH 4/8] drm/i915/uc: Rename intel_?uc_init() to intel_?uc_fetch_fw()

2017-02-21 Thread Arkadiusz Hiler
On Fri, Feb 17, 2017 at 02:31:09PM +0100, Michal Wajdeczko wrote: > On Fri, Feb 17, 2017 at 02:05:53PM +0100, Arkadiusz Hiler wrote: > > Trying to have subject_verb_object ordering and more descriptive names, > > the intel_huc_init() and intel_guc_init() functi

Re: [Intel-gfx] [PATCH 6/8] drm/i915/guc: Extract param logic form guc_init

2017-02-21 Thread Arkadiusz Hiler
On Fri, Feb 17, 2017 at 02:52:00PM +0100, Michal Wajdeczko wrote: > On Fri, Feb 17, 2017 at 02:05:55PM +0100, Arkadiusz Hiler wrote: > > Let intel_guc_fetch_fw() focus on determining and fetching the correct > > firmware. > > > > This patch introduces intel_sanitize

Re: [Intel-gfx] [PATCH 7/8] drm/i915/guc: Simplify intel_guc_init_hw()

2017-02-21 Thread Arkadiusz Hiler
On Tue, Feb 21, 2017 at 02:56:46PM +0100, Arkadiusz Hiler wrote: > On Fri, Feb 17, 2017 at 03:03:04PM +0100, Michal Wajdeczko wrote: > > On Fri, Feb 17, 2017 at 02:05:56PM +0100, Arkadiusz Hiler wrote: > > > Current version of intel_guc_init_hw() does a lot: > > >

Re: [Intel-gfx] [PATCH 7/8] drm/i915/guc: Simplify intel_guc_init_hw()

2017-02-21 Thread Arkadiusz Hiler
On Fri, Feb 17, 2017 at 03:03:04PM +0100, Michal Wajdeczko wrote: > On Fri, Feb 17, 2017 at 02:05:56PM +0100, Arkadiusz Hiler wrote: > > Current version of intel_guc_init_hw() does a lot: > > - cares about submission > > - loads huc > > - implement WA > &g

[Intel-gfx] [PATCH 1/8] drm/i915/uc: Rename intel_?uc_{setup, load}() to _init_hw()

2017-02-22 Thread Arkadiusz Hiler
ris Wilson <ch...@chris-wilson.co.uk> Cc: Michal Winiarski <michal.winiar...@intel.com> Cc: Michal Wajdeczko <michal.wajdec...@intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> Reviewed-by: Joonas Lahtinen <joonas.lahti...@linux.intel.com> Revi

[Intel-gfx] [PATCH 7/8] drm/i915/guc: Simplify intel_guc_init_hw()

2017-02-22 Thread Arkadiusz Hiler
: Anusha Srivatsa <anusha.sriva...@intel.com> Cc: Michal Winiarski <michal.winiar...@intel.com> Cc: Michal Wajdeczko <michal.wajdec...@intel.com> Cc: Daniele Ceraolo Spurio <daniele.ceraolospu...@intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> --- d

[Intel-gfx] [PATCH 5/8] drm/i915/uc: Make intel_uc_fw_fetch() static

2017-02-22 Thread Arkadiusz Hiler
leaking it outside `intel_uc.c` Cc: Joonas Lahtinen <joonas.lahti...@linux.intel.com> Cc: Michal Winiarski <michal.winiar...@intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> --- drivers/gpu/drm/i915/intel_guc_loader.c | 137 +-- dr

[Intel-gfx] [PATCH 6/8] drm/i915/guc: Extract param logic form guc_init

2017-02-22 Thread Arkadiusz Hiler
pu...@intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> --- drivers/gpu/drm/i915/i915_drv.c | 2 ++ drivers/gpu/drm/i915/intel_guc_loader.c | 16 +--- drivers/gpu/drm/i915/intel_uc.c | 27 ++- drivers/gpu/drm/i915/intel_uc.h

[Intel-gfx] [PATCH 4/8] drm/i915/uc: Rename intel_?uc_init() to intel_?uc_fetch_fw()

2017-02-22 Thread Arkadiusz Hiler
. Wajdeczko) Cc: Chris Wilson <ch...@chris-wilson.co.uk> Cc: Joonas Lahtinen <joonas.lahti...@linux.intel.com> Cc: Michal Wajdeczko <michal.wajdec...@intel.com> Cc: Michal Winiarski <michal.winiar...@intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com

[Intel-gfx] [PATCH 2/8] drm/i915/uc: Drop superfluous externs in intel_uc.h

2017-02-22 Thread Arkadiusz Hiler
Externs are implicit and we generally try to avoid them. Cc: Michal Wajdeczko <michal.wajdec...@intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> --- drivers/gpu/drm/i915/intel_uc.h | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a

[Intel-gfx] [PATCH 3/8] drm/i915/huc: Add huc_to_i915

2017-02-22 Thread Arkadiusz Hiler
Used to obtain "dev_priv" from huc struct pointer. We already have similar thing for guc. Cc: Michal Wajdeczko <michal.wajdec...@intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> Reviewed-by: Michal Wajdeczko <michal.wajdec...@intel.com> --- dri

[Intel-gfx] [PATCH 8/8] drm/i915/uc: Simplify firmware path handling

2017-02-22 Thread Arkadiusz Hiler
.sriva...@intel.com> Cc: Tvrtko Ursulin <tvrtko.ursu...@intel.com> Cc: Michal Winiarski <michal.winiar...@intel.com> Cc: Michal Wajdeczko <michal.wajdec...@intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> --- drivers/gpu

[Intel-gfx] [PATCH v4 0/8] GuC Scrub vol. 1

2017-02-22 Thread Arkadiusz Hiler
it. This is the first part of effort to clean it up. v2: rebase after HuC merge + feedback v3: even more renaming that aims to make things more semantic v4: some naming improvements, some bikeshedding Arkadiusz Hiler (8): drm/i915/uc: Rename intel_?uc_{setup,load}() to _init_hw() drm/i915/uc: Drop

Re: [Intel-gfx] [PATCH 4/8] drm/i915/uc: Rename intel_?uc_init() to intel_?uc_fetch_fw()

2017-02-22 Thread Arkadiusz Hiler
On Wed, Feb 22, 2017 at 03:59:01PM +0200, Joonas Lahtinen wrote: > On ke, 2017-02-22 at 13:41 +0100, Arkadiusz Hiler wrote: > > Trying to have subject_verb_object ordering and more descriptive names, > > the intel_huc_init() and intel_guc_init() functions are renamed: > &g

Re: [Intel-gfx] [PATCH 8/8] drm/i915/uc: Simplify firmware path handling

2017-02-22 Thread Arkadiusz Hiler
On Wed, Feb 22, 2017 at 12:53:47PM +, Chris Wilson wrote: > On Wed, Feb 22, 2017 at 01:41:25PM +0100, Arkadiusz Hiler wrote: > > Currently fw->path values can represent one of three possible states: > > > > 1) NULL - device without the uC > > 2) '\0'

[Intel-gfx] [PATCH 10/10] drm/i915/uc: Add params for specifying firmware

2017-02-24 Thread Arkadiusz Hiler
al Winiarski <michal.winiar...@intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> --- drivers/gpu/drm/i915/i915_params.c | 10 ++ drivers/gpu/drm/i915/i915_params.h | 2 ++ drivers/gpu/drm/i915/intel_guc_loader.c | 6 +- drivers/gpu/drm/i915

[Intel-gfx] [PATCH 04/10] drm/i915/uc: Move intel_uc_fw_fetch() to intel_uc.c

2017-02-24 Thread Arkadiusz Hiler
The file fits better. Additionally rename it to intel_uc_prepare_fw(), as the function does more than simple fetch. Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> --- drivers/gpu/drm/i915/intel_guc_loader.c | 137 +--- drivers/gpu/drm/i915/intel

[Intel-gfx] [PATCH 03/10] drm/i915/huc: Add huc_to_i915

2017-02-24 Thread Arkadiusz Hiler
Used to obtain "dev_priv" from huc struct pointer. We already have similar thing for guc. Cc: Michal Wajdeczko <michal.wajdec...@intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> Reviewed-by: Michal Wajdeczko <michal.wajdec...@intel.com> --- dri

[Intel-gfx] [PATCH 08/10] drm/i915/uc: Simplify firmware path handling

2017-02-24 Thread Arkadiusz Hiler
nen) Cc: Anusha Srivatsa <anusha.sriva...@intel.com> Cc: Tvrtko Ursulin <tvrtko.ursu...@intel.com> Cc: Michal Winiarski <michal.winiar...@intel.com> Cc: Michal Wajdeczko <michal.wajdec...@intel.com> Cc: Joonas Lahtinen <joonas.lahti...@linux.intel.com> Signed-off-

[Intel-gfx] [PATCH 07/10] drm/i915/guc: Simplify intel_guc_init_hw()

2017-02-24 Thread Arkadiusz Hiler
s Lahtinen <joonas.lahti...@linux.intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> --- drivers/gpu/drm/i915/i915_gem.c | 2 +- drivers/gpu/drm/i915/intel_guc_loader.c | 144 drivers/gpu/drm/i915/intel_uc.c | 110 +

[Intel-gfx] [PATCH 05/10] drm/i915/uc: Introduce intel_uc_init_fw()

2017-02-24 Thread Arkadiusz Hiler
intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> --- drivers/gpu/drm/i915/i915_drv.c | 3 +-- drivers/gpu/drm/i915/intel_guc_loader.c | 30 +- drivers/gpu/drm/i915/intel_huc.c| 37 - drivers/gpu/dr

[Intel-gfx] [PATCH 09/10] drm/i915/uc: Separate firmware selection and preparation

2017-02-24 Thread Arkadiusz Hiler
Winiarski <michal.winiar...@intel.com> Cc: Joonas Lahtinen <joonas.lahti...@linux.intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> --- drivers/gpu/drm/i915/intel_guc_loader.c | 12 ++-- drivers/gpu/drm/i915/intel_huc.c| 14 ++ dr

[Intel-gfx] [PATCH 02/10] drm/i915/uc: Drop superfluous externs in intel_uc.h

2017-02-24 Thread Arkadiusz Hiler
Externs are implicit and we generally try to avoid them. Cc: Michal Wajdeczko <michal.wajdec...@intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> Reviewed-by: Joonas Lahtinen <joonas.lahti...@linux.intel.com> --- drivers/gpu/drm/i915/intel_uc.h | 12 ++

[Intel-gfx] [PATCH 01/10] drm/i915/uc: Rename intel_?uc_{setup, load}() to _init_hw()

2017-02-24 Thread Arkadiusz Hiler
ris Wilson <ch...@chris-wilson.co.uk> Cc: Michal Winiarski <michal.winiar...@intel.com> Cc: Michal Wajdeczko <michal.wajdec...@intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> Reviewed-by: Joonas Lahtinen <joonas.lahti...@linux.intel.com> Revi

[Intel-gfx] [PATCH 06/10] drm/i915/guc: Extract param logic form guc_init_fw()

2017-02-24 Thread Arkadiusz Hiler
Cc: Daniele Ceraolo Spurio <daniele.ceraolospu...@intel.com> Cc: Joonas Lahtinen <joonas.lahti...@linux.intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> --- drivers/gpu/drm/i915/i915_drv.c | 2 ++ drivers/gpu/drm/i915/intel_guc_loader.c | 14 --

[Intel-gfx] [PATCH v5 00/10] GuC Scrub vol. 1

2017-02-24 Thread Arkadiusz Hiler
step from actual prepare Arkadiusz Hiler (10): drm/i915/uc: Rename intel_?uc_{setup,load}() to _init_hw() drm/i915/uc: Drop superfluous externs in intel_uc.h drm/i915/huc: Add huc_to_i915 drm/i915/uc: Move intel_uc_fw_fetch() to intel_uc.c drm/i915/uc: Introduce intel_uc_init_fw() drm/i915

Re: [Intel-gfx] [PATCH 8/8] drm/i915/uc: Simplify firwmare path handling

2017-02-21 Thread Arkadiusz Hiler
On Fri, Feb 17, 2017 at 03:29:17PM +0100, Michal Wajdeczko wrote: > On Fri, Feb 17, 2017 at 02:05:57PM +0100, Arkadiusz Hiler wrote: > > Typo in subject s/firwmare/firmware > > > > Currently fw->path values can represent one of three possible states: > > > &

Re: [Intel-gfx] [PATCH 5/8] drm/i915/uc: Make intel_uc_fw_fetch() static

2017-02-22 Thread Arkadiusz Hiler
On Wed, Feb 22, 2017 at 04:17:20PM +0200, Joonas Lahtinen wrote: > On ke, 2017-02-22 at 13:41 +0100, Arkadiusz Hiler wrote: > > intel_uc_fw_fetch() is confusingly named in the light of recent changes. > > > > It's also in the wrong place - 'guc_loader.h' - it's used for

Re: [Intel-gfx] [PATCH 8/8] drm/i915/uc: Simplify firmware path handling

2017-02-22 Thread Arkadiusz Hiler
On Wed, Feb 22, 2017 at 04:30:49PM +0100, Arkadiusz Hiler wrote: > On Wed, Feb 22, 2017 at 12:53:47PM +, Chris Wilson wrote: > > On Wed, Feb 22, 2017 at 01:41:25PM +0100, Arkadiusz Hiler wrote: > > > Currently fw->path values can represent one of three possible states:

[Intel-gfx] [PATCH v2 0/5] GuC Scrub vol. 1

2017-02-14 Thread Arkadiusz Hiler
loading but had WA implementations and had the code thatenabled submission baked it. This is first the part of effort to clean it up. Cc: Joonas Lahtinen <joonas.lahti...@linux.intel.com> Arkadiusz Hiler (5): drm/i915/uc: Rename intel_?uc_{setup,load}() to _init_hw() drm/i915/uc: Int

[Intel-gfx] [PATCH 5/5] drm/i915/uc: Simplify fw_path

2017-02-14 Thread Arkadiusz Hiler
and 2. Cc: Anusha Srivatsa <anusha.sriva...@intel.com> Cc: Tvrtko Ursulin <tvrtko.ursu...@intel.com> Cc: Michal Winiarski <michal.winiar...@intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> --- drivers/gpu/drm/i915/intel_guc_loader.c | 28 +++--

[Intel-gfx] [PATCH 4/5] drm/i915/guc: Simplify intel_guc_init_hw()

2017-02-14 Thread Arkadiusz Hiler
intel.com> Cc: Michal Wajdeczko <michal.wajdec...@intel.com> Cc: Daniele Ceraolo Spurio <daniele.ceraolospu...@intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> --- drivers/gpu/drm/i915/i915_gem.c | 2 +- drivers/gpu/drm

[Intel-gfx] [PATCH 1/5] drm/i915/uc: Rename intel_?uc_{setup, load}() to _init_hw()

2017-02-14 Thread Arkadiusz Hiler
em_init_hw(). Cc: Anusha Srivatsa <anusha.sriva...@intel.com> Cc: Michal Winiarski <michal.winiar...@intel.com> Cc: Michal Wajdeczko <michal.wajdec...@intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> --- drivers/gpu/drm/i915/i915_gem.c

[Intel-gfx] [PATCH 2/5] drm/i915/uc: Introduce intel_uc_init()

2017-02-14 Thread Arkadiusz Hiler
We will be able to bulk call all firmware _init() function from single point and offset some general logic there as needed Cc: Anusha Srivatsa <anusha.sriva...@intel.com> Cc: Michal Winiarski <michal.winiar...@intel.com> Cc: Michal Wajdeczko <michal.wajdec...@intel.com> Signed

[Intel-gfx] [PATCH 3/5] drm/i915/guc: Extract param logic form guc_init

2017-02-14 Thread Arkadiusz Hiler
<michal.winiar...@intel.com> Cc: Michal Wajdeczko <michal.wajdec...@intel.com> Cc: Daniele Ceraolo Spurio <daniele.ceraolospu...@intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> --- drivers/gpu/drm/i915/i915_drv.c | 2 ++ drivers/gpu/drm/i915

Re: [Intel-gfx] [PATCH 5/5] drm/i915/uc: Simplify fw_path

2017-02-15 Thread Arkadiusz Hiler
On Tue, Feb 14, 2017 at 09:23:33PM +0100, Michal Wajdeczko wrote: > On Tue, Feb 14, 2017 at 05:15:41PM +0100, Arkadiusz Hiler wrote: > > Currently fw_path values can represent one of three possible states: > > > > 1) NULL - device without the uC > > 2) '\0' -

Re: [Intel-gfx] [PATCH 3/5] drm/i915/guc: Extract param logic form guc_init

2017-02-15 Thread Arkadiusz Hiler
On Wed, Feb 15, 2017 at 02:48:59PM +0200, Joonas Lahtinen wrote: > On ti, 2017-02-14 at 20:37 +0100, Michal Wajdeczko wrote: > > On Tue, Feb 14, 2017 at 05:15:39PM +0100, Arkadiusz Hiler wrote: > > > > > > Let intel_guc_init() focus on determining and fetching

Re: [Intel-gfx] [RFC] drm/fb: Avoid infinite loop when no response from connector.

2017-02-17 Thread Arkadiusz Hiler
On Fri, Feb 17, 2017 at 11:17:46AM +0100, Tomasz Lis wrote: > This fixes an old patch so it doesn't cause infinite retries: > drm/fb: add support for tiled monitor configurations. > > The max count of iterations, 0xa10070f, was carefully selected based on the > fact > that it looks cool. > ---

Re: [Intel-gfx] [PATCH] drm/fb: Proper support of boundary conditions in bitmasks.

2017-02-17 Thread Arkadiusz Hiler
ch introduces an additional check to avoid empty masks. It > reverts the control flow to the exact same way it worked before > the problematic patch. > > Signed-off-by: Tomasz Lis <tomasz@intel.com> Change commit tile to "drm/i915/fbdev: Proper support of boundary".

[Intel-gfx] [PATCH 3/8] drm/i915/huc: Add huc_to_i915

2017-02-17 Thread Arkadiusz Hiler
Used to obtain "dev_priv" from huc struct pointer. We already have similar thing for guc. Cc: Michal Wajdeczko <michal.wajdec...@intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> --- drivers/gpu/drm/i915/i915_drv.h | 5 + 1 file changed, 5 ins

[Intel-gfx] [PATCH 2/8] drm/i915/uc: Drop superfluous externs in intel_uc.h

2017-02-17 Thread Arkadiusz Hiler
Cc: Michal Wajdeczko <michal.wajdec...@intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> --- drivers/gpu/drm/i915/intel_uc.h | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_uc.h b/drivers/gpu/drm/i915/inte

[Intel-gfx] [PATCH 4/8] drm/i915/uc: Rename intel_?uc_init() to intel_?uc_fetch_fw()

2017-02-17 Thread Arkadiusz Hiler
hti...@linux.intel.com> Cc: Michal Wajdeczko <michal.wajdec...@intel.com> Cc: Michal Winiarski <michal.winiar...@intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> --- drivers/gpu/drm/i915/i915_drv.c | 3 +-- drivers/gpu/drm/i915

[Intel-gfx] [PATCH 6/8] drm/i915/guc: Extract param logic form guc_init

2017-02-17 Thread Arkadiusz Hiler
: fix comment, notify when nuking GuC explicitly enabled (M. Wajdeczko) Cc: Michal Winiarski <michal.winiar...@intel.com> Cc: Michal Wajdeczko <michal.wajdec...@intel.com> Cc: Daniele Ceraolo Spurio <daniele.ceraolospu...@intel.com> Signed-off-by: Arkadiusz Hiler <arka

[Intel-gfx] [PATCH 1/8] drm/i915/uc: Rename intel_?uc_{setup, load}() to _init_hw()

2017-02-17 Thread Arkadiusz Hiler
ris Wilson <ch...@chris-wilson.co.uk> Cc: Michal Winiarski <michal.winiar...@intel.com> Cc: Michal Wajdeczko <michal.wajdec...@intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> Reviewed-by: Joonas Lahtinen <joonas.lahti...@linux.intel.com> --- driv

[Intel-gfx] [PATCH v3 0/8] GuC Scrub vol. 1

2017-02-17 Thread Arkadiusz Hiler
it. This is the first part of effort to clean it up. v2: rebase after HuC merge + feedback v3: even more renaming that aims to make things more semantic Arkadiusz Hiler (8): drm/i915/uc: Rename intel_?uc_{setup,load}() to _init_hw() drm/i915/uc: Drop superfluous externs in intel_uc.h drm/i915/huc: Add

[Intel-gfx] [PATCH 5/8] drm/i915/uc: Make intel_uc_fw_fetch() static

2017-02-17 Thread Arkadiusz Hiler
leaking it outside `intel_uc.c` Cc: Joonas Lahtinen <joonas.lahti...@linux.intel.com> Cc: Michal Wajdeczko <michal.wajdec...@intel.com> Cc: Michal Winiarski <michal.winiar...@intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> --- drivers/gpu/drm/i915/

[Intel-gfx] [PATCH 8/8] drm/i915/uc: Simplify firwmare path handling

2017-02-17 Thread Arkadiusz Hiler
;tvrtko.ursu...@intel.com> Cc: Michal Winiarski <michal.winiar...@intel.com> Cc: Michal Wajdeczko <michal.wajdec...@intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> --- drivers/gpu/drm/i915/intel_guc_loader.c | 39 +++--

[Intel-gfx] [PATCH 7/8] drm/i915/guc: Simplify intel_guc_init_hw()

2017-02-17 Thread Arkadiusz Hiler
<anusha.sriva...@intel.com> Cc: Michal Winiarski <michal.winiar...@intel.com> Cc: Michal Wajdeczko <michal.wajdec...@intel.com> Cc: Daniele Ceraolo Spurio <daniele.ceraolospu...@intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> --- drivers/gpu/drm/i915/i915_g

Re: [Intel-gfx] [RFC] drm/i915/guc: Move GuC log related functions into dedicated file

2017-01-16 Thread Arkadiusz Hiler
intel.com> > > > Cc: Chris Wilson <ch...@chris-wilson.co.uk> Reviewed-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> > > > > Looks clean enough. I don't expect any objections, anyone? > > No objections whatsoever, just the opposite! Agreed. We can just drop

[Intel-gfx] [PATCH v5] drm/i915/guc: Extract param logic form guc_init_fw()

2017-02-27 Thread Arkadiusz Hiler
tel.com> Cc: Michal Wajdeczko <michal.wajdec...@intel.com> Cc: Daniele Ceraolo Spurio <daniele.ceraolospu...@intel.com> Cc: Joonas Lahtinen <joonas.lahti...@linux.intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> --- drivers/gpu/drm/i915/i915_drv.c

[Intel-gfx] [PATCH v2] drm/i915/uc: Move intel_uc_fw_fetch() to intel_uc.c

2017-02-27 Thread Arkadiusz Hiler
The file fits better. Additionally rename it to intel_uc_prepare_fw(), as the function does more than simple fetch. v2: remove second declaration, reorder (M. Wajdeczko) Cc: Michal Wajdeczko <michal.wajdec...@intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> ---

Re: [Intel-gfx] [PATCH 09/10] drm/i915/uc: Separate firmware selection and preparation

2017-03-02 Thread Arkadiusz Hiler
On Fri, Feb 24, 2017 at 07:29:57PM +0100, Michal Wajdeczko wrote: > On Fri, Feb 24, 2017 at 04:40:03PM +0100, Arkadiusz Hiler wrote: > > intel_{h,g}uc_init_fw selects correct firmware and then triggers it's > > preparation (fetch + initial parsing). > > > > This change

Re: [Intel-gfx] [PATCH] drm/i915/guc: Disable irq for __i915_guc_submit wq_lock

2017-03-02 Thread Arkadiusz Hiler
2 e1 48 83 c4 > 30 5b 41 5c 41 5d 41 5e 41 5f 5d c3 48 c1 e0 20 48 09 c2 49 89 d0 eb 82 <0f> > 0b 0f 0b 0f 0b 0f 0b 0f 0b 0f 0b 49 c1 e8 20 44 89 43 34 4a > [ 48.183336] RIP: __i915_guc_submit+0x1e6/0x2a0 [i915] RSP: c9067c80 > > Reported-by: Arkadiusz Hiler <arkadiusz.h

Re: [Intel-gfx] [PATCH 07/10] drm/i915/guc: Simplify intel_guc_init_hw()

2017-02-27 Thread Arkadiusz Hiler
On Fri, Feb 24, 2017 at 06:26:10PM +0100, Michal Wajdeczko wrote: > On Fri, Feb 24, 2017 at 04:40:01PM +0100, Arkadiusz Hiler wrote: > > Current version of intel_guc_init_hw() does a lot: > > - cares about submission > > - loads huc > > - implement WA > &g

[Intel-gfx] [PATCH] drm/i915: fix comment on I915_{READ,WRITE}_FW

2016-10-27 Thread Arkadiusz Hiler
,put} instead of plain ones Cc: Chris Wilson <ch...@chris-wilson.co.uk> Cc: Matthew Auld <matthew.a...@intel.com> Cc: Mika Kuoppala <mika.kuopp...@linux.intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> --- drivers/gpu/drm/i915/i915_drv.h | 28 +++

[Intel-gfx] [PATCH] drm/i915/gen9: Remove WaEnableYV12BugFixInHalfSliceChicken7

2016-10-18 Thread Arkadiusz Hiler
Dropping WA because it was for early steppings. It is fixed in newer preproduction and all production revisions. Cc: Michal Winiarski <michal.winiar...@intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> --- drivers/gpu/drm/i915/intel_ringbuffer.c | 2 -- 1 fil

[Intel-gfx] [PATCH] drm/i915/gvt: Implement WaForceWakeRenderDuringMmioTLBInvalidate

2016-10-20 Thread Arkadiusz Hiler
u Wang <zhen...@linux.intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> --- drivers/gpu/drm/i915/gvt/render.c | 11 +++ 1 file changed, 11 insertions(+) diff --git a/drivers/gpu/drm/i915/gvt/render.c b/drivers/gpu/drm/i915/gvt/render.c index f54ab

Re: [Intel-gfx] ✗ Fi.CI.BAT: warning for drm/i915/gvt: Implement WaForceWakeRenderDuringMmioTLBInvalidate

2016-10-20 Thread Arkadiusz Hiler
On Thu, Oct 20, 2016 at 11:17:09AM +, Patchwork wrote: > == Series Details == > > Series: drm/i915/gvt: Implement WaForceWakeRenderDuringMmioTLBInvalidate > URL : https://patchwork.freedesktop.org/series/14097/ > State : warning > > == Summary == > > Series 14097v1 drm/i915/gvt: Implement

[Intel-gfx] [PATCH RESEND v2] drm/i915/gen9: Remove WaEnableYV12BugFixInHalfSliceChicken7

2016-10-20 Thread Arkadiusz Hiler
co.uk> Cc: Michal Winiarski <michal.winiar...@intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> --- drivers/gpu/drm/i915/intel_ringbuffer.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_ringbuffer.c b/drivers/gpu/drm/i915/intel_ri

Re: [Intel-gfx] [PATCH] drm/i915/gvt: Implement WaForceWakeRenderDuringMmioTLBInvalidate

2016-10-20 Thread Arkadiusz Hiler
On Thu, Oct 20, 2016 at 05:29:36PM +0300, Mika Kuoppala wrote: > Arkadiusz Hiler <arkadiusz.hi...@intel.com> writes: > > > When invalidating RCS TLB the device can enter RC6 state interrupting > > the process, therefore the need for render forcewake for the whole > &

[Intel-gfx] [PATCH v2] drm/i915/gen9: Remove WaEnableYV12BugFixInHalfSliceChicken7

2016-10-18 Thread Arkadiusz Hiler
co.uk> Cc: Michal Winiarski <michal.winiar...@intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> --- drivers/gpu/drm/i915/intel_ringbuffer.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_ringbuffer.c b/drivers/gpu/drm/i915/intel_ri

Re: [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915: fix comment on I915_{READ,WRITE}_FW (rev2)

2016-10-25 Thread Arkadiusz Hiler
On Tue, Oct 25, 2016 at 04:27:26PM +0200, Saarinen, Jani wrote: > > == Series Details == > > > > Series: drm/i915: fix comment on I915_{READ,WRITE}_FW (rev2) > > URL : https://patchwork.freedesktop.org/series/14334/ > > State : failure > > > > == Summary == > > > > Series 14334v2 drm/i915:

[Intel-gfx] [PATCH] drm/i915: fix comment on I915_{READ,WRITE}_FW

2016-10-25 Thread Arkadiusz Hiler
w Auld <matthew.a...@intel.com> Cc: Mika Kuoppala <mika.kuopp...@linux.intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> --- drivers/gpu/drm/i915/i915_drv.h | 28 +--- 1 file changed, 25 insertions(+), 3 deletions(-) diff --git a/drivers

[Intel-gfx] [PATCH v2] drm/i915: fix comment on I915_{READ, WRITE}_FW

2016-10-25 Thread Arkadiusz Hiler
,put} instead of plain ones Cc: Chris Wilson <ch...@chris-wilson.co.uk> Cc: Matthew Auld <matthew.a...@intel.com> Cc: Mika Kuoppala <mika.kuopp...@linux.intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> --- drivers/gpu/drm/i915/i915_drv.h | 28 +++

Re: [Intel-gfx] [PATCH] drm/i915: fix comment referencing imaginary functions

2016-10-24 Thread Arkadiusz Hiler
On Fri, Oct 21, 2016 at 02:57:28PM +0100, Chris Wilson wrote: > On Fri, Oct 21, 2016 at 04:00:10PM +0300, Mika Kuoppala wrote: > > Chris Wilson <ch...@chris-wilson.co.uk> writes: > > > > > On Fri, Oct 21, 2016 at 02:16:46PM +0200, Arkadiusz Hiler wrote: > > &

Re: [Intel-gfx] [PATCH] drm/i915: fix comment referencing imaginary functions

2016-10-21 Thread Arkadiusz Hiler
d never have > for that matter(according to git). Let's fix that by replacing them with > intel_uncore_forcewake_{get, put}. > > Cc: Chris Wilson <ch...@chris-wilson.co.uk> > Signed-off-by: Matthew Auld <matthew.a...@intel.com> Reviewed-by: Arkadiusz Hiler <arkadiusz

Re: [Intel-gfx] ✗ Fi.CI.BAT: warning for drm/i915/gvt: Implement WaForceWakeRenderDuringMmioTLBInvalidate

2016-10-20 Thread Arkadiusz Hiler
On Thu, Oct 20, 2016 at 03:04:44PM +0300, Ville Syrjälä wrote: > On Thu, Oct 20, 2016 at 01:52:32PM +0200, Arkadiusz Hiler wrote: > > On Thu, Oct 20, 2016 at 11:17:09AM +, Patchwork wrote: > > > == Series Details == > > > > >

[Intel-gfx] [PATCH v2] drm/i915/guc: Move guc_{send, recv}() to intel_uc.c

2016-11-25 Thread Arkadiusz Hiler
intel_guc_send uses Cc: Chris Wilson <ch...@chris-wilson.co.uk> Cc: Michal Winiarski <michal.winiar...@intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> --- drivers/gpu/drm/i915/Makefile | 3 +- drivers/gpu/drm/i915/i915_guc_

[Intel-gfx] [PATCH 3/5] drm/i915/guc: Move guc_{send, recv}() to intel_uc.c

2016-11-25 Thread Arkadiusz Hiler
intel_guc_send usages Cc: Chris Wilson <ch...@chris-wilson.co.uk> Cc: Michal Winiarski <michal.winiar...@intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> Reviewed-by: Chris Wilson <ch...@chris-wilson.co.uk> --- drivers/gpu/drm/i915/Makefile

[Intel-gfx] [PATCH 5/5] drm/i915/guc: Remove spurious include

2016-11-25 Thread Arkadiusz Hiler
Cc: Chris Wilson <ch...@chris-wilson.co.uk> Cc: Michal Winiarski <michal.winiar...@intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> Reviewed-by: Chris Wilson <ch...@chris-wilson.co.uk> --- drivers/gpu/drm/i915/i915_guc_submission.c | 1 - 1 file chan

[Intel-gfx] [PATCH 4/5] drm/i915/guc: Init send_mutex in intel_uc_init_early()

2016-11-25 Thread Arkadiusz Hiler
gt; Cc: Michal Winiarski <michal.winiar...@intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> Reviewed-by: Chris Wilson <ch...@chris-wilson.co.uk> --- drivers/gpu/drm/i915/i915_drv.c| 2 ++ drivers/gpu/drm/i915/i915_guc_submission.c | 1 - drivers

Re: [Intel-gfx] [PATCH 0/8] HuC Loading Patches

2016-11-25 Thread Arkadiusz Hiler
On Wed, Nov 23, 2016 at 02:27:35PM -0800, Anusha Srivatsa wrote: > These patches add HuC loading support. The userspace > patches that check for a fully loaded HuC firmware and use > it can be found at: > https://lists.freedesktop.org/archives/libva/2016-September/004554.html >

[Intel-gfx] [PATCH 1/5] drm/i915: Rename intel_guc.h to intel_uc.h

2016-11-25 Thread Arkadiusz Hiler
GuC is not the only one micro controller we have. There are also HuC and DMC. Making the file more general will help with code organization. Cc: Chris Wilson <ch...@chris-wilson.co.uk> Cc: Michal Winiarski <michal.winiar...@intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.

[Intel-gfx] [PATCH 2/5] drm/i915/guc: Drop guc2host/host2guc from names

2016-11-25 Thread Arkadiusz Hiler
omes send_mutex v2: drop unnecessary backslashes and use BIT() instead of '<<' v3: shortened enum names and INTEL_GUC_STATUS_* Cc: Chris Wilson <ch...@chris-wilson.co.uk> Cc: Michal Winiarski <michal.winiar...@intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com&g

[Intel-gfx] [PATCH v3 0/5] GuC code reorganization

2016-11-25 Thread Arkadiusz Hiler
were renamed and moved around. v2: host2guc and guc2host renaming, init mutex in intel_uc_init() v3: another round of review, general code scrub and slight renaming Arkadiusz Hiler (5): drm/i915: Rename intel_guc.h to intel_uc.h drm/i915/guc: Drop guc2host/host2guc from names drm/i915/guc

Re: [Intel-gfx] [PATCH] drm/i915/guc: Always initialize action_lock

2016-11-23 Thread Arkadiusz Hiler
On Wed, Nov 23, 2016 at 10:07:27AM +, Chris Wilson wrote: > On Wed, Nov 23, 2016 at 10:41:42AM +0100, Arkadiusz Hiler wrote: > > On Tue, Nov 22, 2016 at 05:05:32PM +, Chris Wilson wrote: > > > On Tue, Nov 22, 2016 at 05:22:47PM +0100, Arkadiusz Hiler wrote: > > >

Re: [Intel-gfx] [PATCH] drm/i915/guc: Always initialize action_lock

2016-11-23 Thread Arkadiusz Hiler
On Tue, Nov 22, 2016 at 05:05:32PM +, Chris Wilson wrote: > On Tue, Nov 22, 2016 at 05:22:47PM +0100, Arkadiusz Hiler wrote: > > Action lock is not being initialized if the GuC submission is disabled > > (i.e. i915.guc_submission=0). > > > > host2guc_action(), whi

Re: [Intel-gfx] [PATCH] drm/i915/guc: Always initialize action_lock

2016-11-23 Thread Arkadiusz Hiler
Arkadiusz <arkadiusz.hi...@intel.com> > >Cc: intel-gfx@lists.freedesktop.org > >Subject: Re: [Intel-gfx] [PATCH] drm/i915/guc: Always initialize action_lock > > > >On Tue, Nov 22, 2016 at 05:22:47PM +0100, Arkadiusz Hiler wrote: > >> Action lock is not being ini

[Intel-gfx] [PATCH] drm/i915/guc: Always initialize action_lock

2016-11-22 Thread Arkadiusz Hiler
to use the host2guc_action no matter whether submission is enabled or not. Cc: Anusha Srivatsa <anusha.sriva...@intel.com> Cc: Jeff McGee <jeff.mc...@intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> --- drivers/gpu/drm/i915/i915_guc_submission.c | 3 ++-

Re: [Intel-gfx] [PATCH 1/8] drm/i915/guc: Make the GuC fw loading helper functions general. Always initialize action_lock

2016-11-24 Thread Arkadiusz Hiler
gned-off-by: Peter Antoine <peter.anto...@intel.com> > Reviewed-by: Dave Gordon <david.s.gor...@intel.com> > Reviewed-by: Jeff McGee <jeff.mc...@intel.com> > Reviewed-by: Carlos Santa <carlos.sa...@intel.com> > Tested-by: Carlos Santa <carlos.sa...@intel.com>

Re: [Intel-gfx] [PATCH 1/8] drm/i915/guc: Make the GuC fw loading helper functions general. Always initialize action_lock

2016-11-24 Thread Arkadiusz Hiler
On Thu, Nov 24, 2016 at 09:28:14AM +0100, Arkadiusz Hiler wrote: > On Wed, Nov 23, 2016 at 02:27:36PM -0800, Anusha Srivatsa wrote: > > From: Peter Antoine <peter.anto...@intel.com> > > > > Rename some of the GuC fw loading code to make them more general. We > >

[Intel-gfx] [PATCH 4/4] drm/i915/guc: Remove spurious include

2016-11-24 Thread Arkadiusz Hiler
Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> Cc: Chris Wilson <ch...@chris-wilson.co.uk> Cc: Michal Winiarski <michal.winiar...@intel.com> --- drivers/gpu/drm/i915/i915_guc_submission.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/i915/i915_g

[Intel-gfx] [PATCH 1/4] drm/i915: Rename intel_guc.h to intel_uc.h

2016-11-24 Thread Arkadiusz Hiler
GuC is not the only one micro controller we have. There are also HuC and DMC. Making the file more general will help with code organization. Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> Cc: Chris Wilson <ch...@chris-wilson.co.uk> Cc: Michal Winiarski <michal.wini

[Intel-gfx] [PATCH 0/4] GuC code reorganization

2016-11-24 Thread Arkadiusz Hiler
With HuC being imminent we could benefit from some code reorganization. There will be more uC than just GuC so we should name files accordingly. There is also need to use host2guc_*() family of functions, which are currently limited to submission only. Arkadiusz Hiler (4): drm/i915: Rename

[Intel-gfx] [PATCH 2/4] drm/i915/guc: Move host2guc_*() to intel_uc.c

2016-11-24 Thread Arkadiusz Hiler
host2guc_*() series of functions were introduced in the i915_guc_submission.c and their scope was limited only to that file. Those are not submission specific though. Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> Cc: Chris Wilson <ch...@chris-wilson.co.uk> Cc: Mich

[Intel-gfx] [PATCH 3/4] drm/i915/guc: Init action_lock in the loader

2016-11-24 Thread Arkadiusz Hiler
Action_lock is a mutex used to serialise communication with GuC. Since some functions which utilize the mutex (host2guc_*()) are no longer limited to submission, initialization should be handled as a part of GuC setup. Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> Cc: Chris

[Intel-gfx] [PATCH v2] drm/i915/guc: Init send_mutex in intel_uc_init_early()

2016-11-25 Thread Arkadiusz Hiler
gt; Cc: Michal Winiarski <michal.winiar...@intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> Reviewed-by: Chris Wilson <ch...@chris-wilson.co.uk> --- drivers/gpu/drm/i915/i915_drv.c| 2 ++ drivers/gpu/drm/i915/i915_guc_submission.c | 1 - drivers

[Intel-gfx] [PATCH v2] drm/i915/guc: Drop guc2host/host2guc from names

2016-11-25 Thread Arkadiusz Hiler
tead of '<<' Cc: Chris Wilson <ch...@chris-wilson.co.uk> Cc: Michal Winiarski <michal.winiar...@intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> --- drivers/gpu/drm/i915/i915_guc_reg.h| 4 +- drivers/gpu/drm/i91

[Intel-gfx] [PATCH v3] drm/i915/guc: Drop guc2host/host2guc from names

2016-11-25 Thread Arkadiusz Hiler
omes send_mutex v2: drop unnecessary backslashes and use BIT() instead of '<<' v3: shortened enum names and INTEL_GUC_STATUS_* Cc: Chris Wilson <ch...@chris-wilson.co.uk> Cc: Michal Winiarski <michal.winiar...@intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com&g

Re: [Intel-gfx] ✗ Fi.CI.BAT: warning for series starting with [CI,1/5] drm/i915: Rename intel_guc.h to intel_uc.h

2016-11-28 Thread Arkadiusz Hiler
On Mon, Nov 28, 2016 at 12:53:27PM +, Patchwork wrote: > == Series Details == > > Series: series starting with [CI,1/5] drm/i915: Rename intel_guc.h to > intel_uc.h > URL : https://patchwork.freedesktop.org/series/16034/ > State : warning > > == Summary == > > Series 16034v1 Series

[Intel-gfx] [PATCH 2/5] drm/i915/guc: Drop guc2host/host2guc from names

2016-11-24 Thread Arkadiusz Hiler
Cc: Michal Winiarski <michal.winiar...@intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> --- drivers/gpu/drm/i915/i915_guc_reg.h| 4 +- drivers/gpu/drm/i915/i915_guc_submission.c | 91 +++--- drivers/gpu/drm/i915/i915_irq.c

[Intel-gfx] [PATCH 3/5] drm/i915/guc: Move guc_{send, recv}() to intel_uc.c

2016-11-24 Thread Arkadiusz Hiler
gt; Cc: Michal Winiarski <michal.winiar...@intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> --- drivers/gpu/drm/i915/Makefile | 3 +- drivers/gpu/drm/i915/i915_guc_submission.c | 131 +++ drivers/gpu/drm/i915/intel_uc.

[Intel-gfx] [PATCH 1/5] drm/i915: Rename intel_guc.h to intel_uc.h

2016-11-24 Thread Arkadiusz Hiler
GuC is not the only one micro controller we have. There are also HuC and DMC. Making the file more general will help with code organization. Cc: Chris Wilson <ch...@chris-wilson.co.uk> Cc: Michal Winiarski <michal.winiar...@intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.

[Intel-gfx] [PATCH 4/5] drm/i915/guc: Init send_mutex in intel_uc_init()

2016-11-24 Thread Arkadiusz Hiler
al.winiar...@intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> --- drivers/gpu/drm/i915/i915_drv.c| 1 + drivers/gpu/drm/i915/i915_guc_submission.c | 1 - drivers/gpu/drm/i915/intel_uc.c| 5 + drivers/gpu/drm/i915/intel_uc.h| 1 +

[Intel-gfx] [PATCH v2 0/5] GuC code reorganization

2016-11-24 Thread Arkadiusz Hiler
were renamed and moved around. v2: host2guc and guc2host renaming, init mutex in intel_uc_init() Arkadiusz Hiler (5): drm/i915: Rename intel_guc.h to intel_uc.h drm/i915/guc: Drop guc2host/host2guc from names drm/i915/guc: Move guc_{send,recv}() to intel_uc.c drm/i915/guc: Init send_mutex

[Intel-gfx] [PATCH 5/5] drm/i915/guc: Remove spurious include

2016-11-24 Thread Arkadiusz Hiler
Cc: Chris Wilson <ch...@chris-wilson.co.uk> Cc: Michal Winiarski <michal.winiar...@intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> Reviewed-by: Chris Wilson <ch...@chris-wilson.co.uk> --- drivers/gpu/drm/i915/i915_guc_submission.c | 1 - 1 file chan

  1   2   3   4   5   >