>-----Original Message----- >From: Chris Wilson <[email protected]> >Sent: Friday, April 17, 2020 3:55 PM >To: Ruhl, Michael J <[email protected]>; intel- >[email protected] >Subject: Re: [Intel-gfx] [PATCH v2] drm/i915: Refactor setting dma info to a >common helper > >Quoting Michael J. Ruhl (2020-04-17 20:51:07) >> DMA_MASK bit values are different for different generations. >> >> This will become more difficult to manage over time with the open >> coded usage of different versions of the device. >> >> Fix by: >> disallow setting of dma mask in AGP path (< GEN(5) for i915, >> add dma_mask_size to the device info configuration, >> updating open code call sequence to the latest interface, >> refactoring into a common function for setting the dma segment >> and mask info >> >> Reviewed-by: Chris Wilson <[email protected]> >> Signed-off-by: Michael J. Ruhl <[email protected]> >> cc: Brian Welty <[email protected]> >> cc: Daniele Ceraolo Spurio <[email protected]> >> >> --- >> v1: removed i915 depenancy from agp path for dma mask >> Consolidated segment size and work arounds to the helper >> v2: added r-b > >You don't need to resend for adding r-b by itself, patchwork will do >that, and the committer should be checking the output from pw. dim then >double checks that we haven't missed anything vital.
Cool. I will refrain from future sends. 😊 M >-Chris _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
