From: Ran Jianping ran.jianping@zte.com.cn
'drm/ttm/ttm_placement.h' included in 'drivers/gpu/drm/i915/selftests/mock_region.c' is duplicated. It is also included on the 9 line.
Reported-by: Zeal Robot zealci@zte.com.cn Signed-off-by: Ran Jianping ran.jianping@zte.com.cn --- drivers/gpu/drm/i915/selftests/mock_region.c | 2 -- 1 file changed, 2 deletions(-)
diff --git a/drivers/gpu/drm/i915/selftests/mock_region.c b/drivers/gpu/drm/i915/selftests/mock_region.c index efa86dffe3c6..75793008c4ef 100644 --- a/drivers/gpu/drm/i915/selftests/mock_region.c +++ b/drivers/gpu/drm/i915/selftests/mock_region.c @@ -6,8 +6,6 @@ #include <drm/ttm/ttm_placement.h> #include <linux/scatterlist.h>
-#include <drm/ttm/ttm_placement.h> - #include "gem/i915_gem_region.h" #include "intel_memory_region.h" #include "intel_region_ttm.h"
On 19/10/2021 10:02, cgel.zte@gmail.com wrote:
From: Ran Jianping ran.jianping@zte.com.cn
'drm/ttm/ttm_placement.h' included in 'drivers/gpu/drm/i915/selftests/mock_region.c' is duplicated. It is also included on the 9 line.
Reported-by: Zeal Robot zealci@zte.com.cn Signed-off-by: Ran Jianping ran.jianping@zte.com.cn
Commit tile should ideally be prefixed with 'drm/i915:'.
Can fix up when pushing though. Thanks, Reviewed-by: Matthew Auld matthew.auld@intel.com
drivers/gpu/drm/i915/selftests/mock_region.c | 2 -- 1 file changed, 2 deletions(-)
diff --git a/drivers/gpu/drm/i915/selftests/mock_region.c b/drivers/gpu/drm/i915/selftests/mock_region.c index efa86dffe3c6..75793008c4ef 100644 --- a/drivers/gpu/drm/i915/selftests/mock_region.c +++ b/drivers/gpu/drm/i915/selftests/mock_region.c @@ -6,8 +6,6 @@ #include <drm/ttm/ttm_placement.h> #include <linux/scatterlist.h>
-#include <drm/ttm/ttm_placement.h>
- #include "gem/i915_gem_region.h" #include "intel_memory_region.h" #include "intel_region_ttm.h"
From: ran jianping ran.jianping@zte.com.cn
'drm/ttm/ttm_placement.h' included in 'drivers/gpu/drm/i915/selftests/mock_region.c' is duplicated. It is also included on the 9 line.
Reported-by: Zeal Robot zealci@zte.com.cn Signed-off-by: ran jianping ran.jianping@zte.com.cn --- drivers/gpu/drm/i915/selftests/mock_region.c | 2 -- 1 file changed, 2 deletion(-)
diff --git a/drivers/gpu/drm/i915/selftests/mock_region.c b/drivers/gpu/drm/i915/selftests/mock_region.c index efa86dffe3c6..b6747b3eeac5 100644 --- a/drivers/gpu/drm/i915/selftests/mock_region.c +++ b/drivers/gpu/drm/i915/selftests/mock_region.c @@ -6,7 +6,5 @@ #include <drm/ttm/ttm_placement.h> #include <linux/scatterlist.h>
-#include <drm/ttm/ttm_placement.h> - #include "gem/i915_gem_region.h" #include "intel_memory_region.h"
On 20/10/2021 09:04, Ran Jianping wrote:
From: ran jianping ran.jianping@zte.com.cn
'drm/ttm/ttm_placement.h' included in 'drivers/gpu/drm/i915/selftests/mock_region.c' is duplicated. It is also included on the 9 line.
Reported-by: Zeal Robot zealci@zte.com.cn Signed-off-by: ran jianping ran.jianping@zte.com.cn
Pushed to drm-intel-gt-next. Thanks.
drivers/gpu/drm/i915/selftests/mock_region.c | 2 -- 1 file changed, 2 deletion(-)
diff --git a/drivers/gpu/drm/i915/selftests/mock_region.c b/drivers/gpu/drm/i915/selftests/mock_region.c index efa86dffe3c6..b6747b3eeac5 100644 --- a/drivers/gpu/drm/i915/selftests/mock_region.c +++ b/drivers/gpu/drm/i915/selftests/mock_region.c @@ -6,7 +6,5 @@ #include <drm/ttm/ttm_placement.h> #include <linux/scatterlist.h>
-#include <drm/ttm/ttm_placement.h>
- #include "gem/i915_gem_region.h" #include "intel_memory_region.h"
dri-devel@lists.freedesktop.org