drm/i915/gvt: Use offsetofend() rather than offsetof + sizeof
authorChris Wilson <chris@chris-wilson.co.uk>
Wed, 15 Mar 2017 17:39:50 +0000 (17:39 +0000)
committerZhenyu Wang <zhenyuw@linux.intel.com>
Thu, 17 May 2018 14:05:10 +0000 (22:05 +0800)
Compute the offset of the end of the crc32 field using offsetofend()
rather than open-coding.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Zhenyu Wang <zhenyuw@linux.intel.com>
Cc: Zhi Wang <zhi.a.wang@intel.com>
Reviewed-by: Mika Kuoppala <mika.kuoppala@intel.com>
Signed-off-by: Zhenyu Wang <zhenyuw@linux.intel.com>
drivers/gpu/drm/i915/gvt/firmware.c

index a73e1d418c228f20ac29cbfc161132a8d54669ae..4ac18b44724769f458b6ff677edcd79a589c3826 100644 (file)
@@ -162,7 +162,7 @@ static int verify_firmware(struct intel_gvt *gvt,
 
        h = (struct gvt_firmware_header *)fw->data;
 
-       crc32_start = offsetof(struct gvt_firmware_header, crc32) + 4;
+       crc32_start = offsetofend(struct gvt_firmware_header, crc32);
        mem = fw->data + crc32_start;
 
 #define VERIFY(s, a, b) do { \