public inbox for intel-gfx@lists.freedesktop.org
 help / color / mirror / Atom feed
* [RFC 0/3] GuC functions name/prototype update
@ 2017-11-10  9:59 Sagar Arun Kamble
  2017-11-10  9:59 ` [RFC 1/3] drm/i915/guc: Update name and prototype of GuC submission related functions Sagar Arun Kamble
                   ` (5 more replies)
  0 siblings, 6 replies; 8+ messages in thread
From: Sagar Arun Kamble @ 2017-11-10  9:59 UTC (permalink / raw)
  To: intel-gfx

We want to have consistent function/structure/file naming and
function parameter semantics. Suggestion from Michal Wajdeczko about
using "genx_" prefix (or better _genx" suffix for all hw related
structures types, offsets), "i915_" prefix for driver entry calls and
"intel_" in all other seems to be the good scheme to go ahead.
Also we need to adhere to practise of passing drm_i915_private to
functions with "i915_" prefix and component structure to functions
with "intel_" prefix. This series updates GuC submission related code
with these changes.

With GuC submission being hardware interface for i915, we should name
those functions with prefix "intel_guc" and pass struct intel_guc as
parameter. Also i915_guc_client struct is kernel-hw interaction
structure hence rename that to intel_guc_client.
With these changes GuC submission source files are now named with
"intel_" prefix.

Please add to Cc as I might not have covered all interested in this
change.

Cc: Michal Wajdeczko <michal.wajdeczko@intel.com>
Cc: Michal Winiarski <michal.winiarski@intel.com>
Cc: Oscar Mateo <oscar.mateo@intel.com>
Cc: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
Cc: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
Cc: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>

(Ignore checkpatch errors in patch 2&3 as they are not introduced by this
series)

Sagar Arun Kamble (3):
  drm/i915/guc: Update name and prototype of GuC submission related
    functions
  drm/i915/guc: Rename i915_guc_client struct to intel_guc_client
  drm/i915/guc: Rename i915_guc_submission.c|h to
    intel_guc_submission.c|h

 drivers/gpu/drm/i915/Makefile               |    4 +-
 drivers/gpu/drm/i915/i915_debugfs.c         |    6 +-
 drivers/gpu/drm/i915/i915_guc_submission.c  | 1466 ---------------------------
 drivers/gpu/drm/i915/i915_guc_submission.h  |   80 --
 drivers/gpu/drm/i915/intel_guc.c            |    2 +-
 drivers/gpu/drm/i915/intel_guc.h            |    6 +-
 drivers/gpu/drm/i915/intel_guc_submission.c | 1463 ++++++++++++++++++++++++++
 drivers/gpu/drm/i915/intel_guc_submission.h |   80 ++
 drivers/gpu/drm/i915/intel_uc.c             |   12 +-
 9 files changed, 1558 insertions(+), 1561 deletions(-)
 delete mode 100644 drivers/gpu/drm/i915/i915_guc_submission.c
 delete mode 100644 drivers/gpu/drm/i915/i915_guc_submission.h
 create mode 100644 drivers/gpu/drm/i915/intel_guc_submission.c
 create mode 100644 drivers/gpu/drm/i915/intel_guc_submission.h

-- 
1.9.1

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2017-11-11  0:26 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-11-10  9:59 [RFC 0/3] GuC functions name/prototype update Sagar Arun Kamble
2017-11-10  9:59 ` [RFC 1/3] drm/i915/guc: Update name and prototype of GuC submission related functions Sagar Arun Kamble
2017-11-10  9:59 ` [RFC 2/3] drm/i915/guc: Rename i915_guc_client struct to intel_guc_client Sagar Arun Kamble
2017-11-10  9:59 ` [RFC 3/3] drm/i915/guc: Rename i915_guc_submission.c|h to intel_guc_submission.c|h Sagar Arun Kamble
2017-11-10 10:04 ` [RFC 0/3] GuC functions name/prototype update Chris Wilson
2017-11-10 11:29   ` Joonas Lahtinen
2017-11-10 10:38 ` ✗ Fi.CI.BAT: warning for " Patchwork
2017-11-11  0:26 ` [RFC 0/3] " Oscar Mateo

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox