Hi Matt, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on drm-tip/drm-tip] [also build test WARNING on drm/drm-next drm-misc/drm-misc-next linus/master v6.0-rc4 next-20220906] [cannot apply to drm-intel/for-linux-next] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch, we suggest to use '--base' as documented in https://git-scm.com/docs/git-format-patch#_base_tree_information] url: https://github.com/intel-lab-lkp/linux/commits/Matt-Roper/i915-Add-standalone-media-support-for-MTL/20220907-075318 base: git://anongit.freedesktop.org/drm/drm-tip drm-tip config: i386-randconfig-s001 compiler: gcc-11 (Debian 11.3.0-5) 11.3.0 reproduce: # apt-get install sparse # sparse version: v0.6.4-39-gce1a6720-dirty # https://github.com/intel-lab-lkp/linux/commit/c06fd3ab7efd7036d4d9b61f4e8f2e585cc7771a git remote add linux-review https://github.com/intel-lab-lkp/linux git fetch --no-tags linux-review Matt-Roper/i915-Add-standalone-media-support-for-MTL/20220907-075318 git checkout c06fd3ab7efd7036d4d9b61f4e8f2e585cc7771a # save the config file mkdir build_dir && cp config build_dir/.config make W=1 C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' O=build_dir ARCH=i386 SHELL=/bin/bash If you fix the issue, kindly add following tag where applicable Reported-by: kernel test robot sparse warnings: (new ones prefixed by >>) >> drivers/gpu/drm/i915/intel_uncore.c:2238:17: sparse: sparse: incorrect type in argument 1 (different address spaces) @@ expected void volatile [noderef] __iomem *addr @@ got void *regs @@ drivers/gpu/drm/i915/intel_uncore.c:2238:17: sparse: expected void volatile [noderef] __iomem *addr drivers/gpu/drm/i915/intel_uncore.c:2238:17: sparse: got void *regs >> drivers/gpu/drm/i915/intel_uncore.c:2269:16: sparse: sparse: incorrect type in argument 3 (different address spaces) @@ expected void *data @@ got void [noderef] __iomem *regs @@ drivers/gpu/drm/i915/intel_uncore.c:2269:16: sparse: expected void *data drivers/gpu/drm/i915/intel_uncore.c:2269:16: sparse: got void [noderef] __iomem *regs drivers/gpu/drm/i915/intel_uncore.c:1843:1: sparse: sparse: context imbalance in 'fwtable_read8' - different lock contexts for basic block drivers/gpu/drm/i915/intel_uncore.c:1844:1: sparse: sparse: context imbalance in 'fwtable_read16' - different lock contexts for basic block drivers/gpu/drm/i915/intel_uncore.c:1845:1: sparse: sparse: context imbalance in 'fwtable_read32' - different lock contexts for basic block drivers/gpu/drm/i915/intel_uncore.c:1846:1: sparse: sparse: context imbalance in 'fwtable_read64' - different lock contexts for basic block drivers/gpu/drm/i915/intel_uncore.c:1909:1: sparse: sparse: context imbalance in 'gen6_write8' - different lock contexts for basic block drivers/gpu/drm/i915/intel_uncore.c:1910:1: sparse: sparse: context imbalance in 'gen6_write16' - different lock contexts for basic block drivers/gpu/drm/i915/intel_uncore.c:1911:1: sparse: sparse: context imbalance in 'gen6_write32' - different lock contexts for basic block drivers/gpu/drm/i915/intel_uncore.c:1931:1: sparse: sparse: context imbalance in 'fwtable_write8' - different lock contexts for basic block drivers/gpu/drm/i915/intel_uncore.c:1932:1: sparse: sparse: context imbalance in 'fwtable_write16' - different lock contexts for basic block drivers/gpu/drm/i915/intel_uncore.c:1933:1: sparse: sparse: context imbalance in 'fwtable_write32' - different lock contexts for basic block vim +2238 drivers/gpu/drm/i915/intel_uncore.c 2235 2236 static void uncore_unmap_mmio(struct drm_device *drm, void *regs) 2237 { > 2238 iounmap(regs); 2239 } 2240 2241 int intel_uncore_setup_mmio(struct intel_uncore *uncore, phys_addr_t phys_addr) 2242 { 2243 struct drm_i915_private *i915 = uncore->i915; 2244 int mmio_size; 2245 2246 /* 2247 * Before gen4, the registers and the GTT are behind different BARs. 2248 * However, from gen4 onwards, the registers and the GTT are shared 2249 * in the same BAR, so we want to restrict this ioremap from 2250 * clobbering the GTT which we want ioremap_wc instead. Fortunately, 2251 * the register BAR remains the same size for all the earlier 2252 * generations up to Ironlake. 2253 * For dgfx chips register range is expanded to 4MB, and this larger 2254 * range is also used for integrated gpus beginning with Meteor Lake. 2255 */ 2256 if (IS_DGFX(i915) || GRAPHICS_VER_FULL(i915) >= IP_VER(12, 70)) 2257 mmio_size = 4 * 1024 * 1024; 2258 else if (GRAPHICS_VER(i915) >= 5) 2259 mmio_size = 2 * 1024 * 1024; 2260 else 2261 mmio_size = 512 * 1024; 2262 2263 uncore->regs = ioremap(phys_addr, mmio_size); 2264 if (uncore->regs == NULL) { 2265 drm_err(&i915->drm, "failed to map registers\n"); 2266 return -EIO; 2267 } 2268 > 2269 return drmm_add_action_or_reset(&i915->drm, uncore_unmap_mmio, uncore->regs); 2270 } 2271 -- 0-DAY CI Kernel Test Service https://01.org/lkp