All of lore.kernel.org
 help / color / mirror / Atom feed
From: Chris Wilson <chris@chris-wilson.co.uk>
To: intel-gfx@lists.freedesktop.org
Subject: [PATCH 1/7] drm/i915: Silence sparse complaints over insufficient bitfield int types.
Date: Sun,  6 Jun 2010 15:40:18 +0100	[thread overview]
Message-ID: <1275835224-10872-1-git-send-email-chris@chris-wilson.co.uk> (raw)

drivers/gpu/drm/i915/i915_drv.h|676 col 19| warning: dubious bitfield without explicit `signed' or `unsigned'
drivers/gpu/drm/i915/i915_drv.h|712 col 19| warning: dubious bitfield without explicit `signed' or `unsigned'

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
---
 drivers/gpu/drm/i915/i915_drv.h |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h
index 27900cd..f348f35 100644
--- a/drivers/gpu/drm/i915/i915_drv.h
+++ b/drivers/gpu/drm/i915/i915_drv.h
@@ -673,7 +673,7 @@ struct drm_i915_gem_object {
 	 *
 	 * Size: 4 bits for 16 fences + sign (for FENCE_REG_NONE)
 	 */
-	int fence_reg : 5;
+	signed int fence_reg : 5;
 
 	/**
 	 * Used for checking the object doesn't appear more than once
@@ -709,7 +709,7 @@ struct drm_i915_gem_object {
 	 *
 	 * In the worst case this is 1 + 1 + 1 + 2*2 = 7. That would fit into 3
 	 * bits with absolutely no headroom. So use 4 bits. */
-	int pin_count : 4;
+	unsigned int pin_count : 4;
 #define DRM_I915_GEM_OBJECT_MAX_PIN_COUNT 0xf
 
 	/** AGP memory structure for our GTT binding. */
-- 
1.7.1

             reply	other threads:[~2010-06-06 14:40 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-06-06 14:40 Chris Wilson [this message]
2010-06-06 14:40 ` [PATCH 2/7] drm/i915: Silence sparse over duplicate members in static initializer Chris Wilson
2010-06-06 14:40 ` [PATCH 3/7] drm/i915: Silence sparse over non-static local structure Chris Wilson
2010-06-06 14:40 ` [PATCH 4/7] drm/i915: Use an uncommon name for the local dev_priv in macros Chris Wilson
2010-06-06 14:40 ` [PATCH 5/7] drm/i915: Sparse warns about the incorrect sign for storing bit17 Chris Wilson
2010-06-06 14:40 ` [PATCH 6/7] drm/i915: Sparse warning about invalid value for burst_ena in tv_modes Chris Wilson
2010-06-06 14:40 ` [PATCH 7/7] drm/i915: Return boolean status not error code from intel_crtc_mode_fixup() Chris Wilson
2010-07-06  6:49 ` [PATCH 1/7] drm/i915: Silence sparse complaints over insufficient bitfield int types Eric Anholt

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1275835224-10872-1-git-send-email-chris@chris-wilson.co.uk \
    --to=chris@chris-wilson.co.uk \
    --cc=intel-gfx@lists.freedesktop.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.