Re: [PATCH] powerpc: Fix duplicate included _clear.h

Re: [PATCH] powerpc: Fix duplicate included _clear.h

[kernel test robot]

[-- Attachment #1: Type: text/plain, Size: 1540 bytes --]

CC: kbuild-all(a)lists.01.org
In-Reply-To: <1623061512-31651-1-git-send-email-jiapeng.chong@linux.alibaba.com>
References: <1623061512-31651-1-git-send-email-jiapeng.chong@linux.alibaba.com>
TO: Jiapeng Chong <jiapeng.chong@linux.alibaba.com>
TO: mpe(a)ellerman.id.au
CC: benh(a)kernel.crashing.org
CC: paulus(a)samba.org
CC: linuxppc-dev(a)lists.ozlabs.org
CC: linux-kernel(a)vger.kernel.org
CC: Jiapeng Chong <jiapeng.chong@linux.alibaba.com>

Hi Jiapeng,

Thank you for the patch! Perhaps something to improve:

[auto build test WARNING on powerpc/next]
[also build test WARNING on v5.13-rc5 next-20210607]
[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]

url:    https://github.com/0day-ci/linux/commits/Jiapeng-Chong/powerpc-Fix-duplicate-included-_clear-h/20210607-182626
base:   https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git next
:::::: branch date: 3 hours ago
:::::: commit date: 3 hours ago
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>


includecheck warnings: (new ones prefixed by >>)
>> arch/powerpc/perf/req-gen/perf.h: _clear.h is included more than once.

Please review and possibly fold the followup patch.

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org

[RFC PATCH] powerpc: fix duplicated inclusion

[kernel test robot]

[-- Attachment #1: Type: text/plain, Size: 2326 bytes --]

CC: kbuild-all(a)lists.01.org
In-Reply-To: <1623061512-31651-1-git-send-email-jiapeng.chong@linux.alibaba.com>
References: <1623061512-31651-1-git-send-email-jiapeng.chong@linux.alibaba.com>
TO: Jiapeng Chong <jiapeng.chong@linux.alibaba.com>
TO: mpe(a)ellerman.id.au
CC: benh(a)kernel.crashing.org
CC: paulus(a)samba.org
CC: linuxppc-dev(a)lists.ozlabs.org
CC: linux-kernel(a)vger.kernel.org
CC: Jiapeng Chong <jiapeng.chong@linux.alibaba.com>
arch/powerpc/perf/req-gen/perf.h: _clear.h is included more than once.

Generated by: scripts/checkincludes.pl

Reported-by: kernel test robot <lkp@intel.com>
Signed-off-by: kernel test robot <lkp@intel.com>
---
 perf.h |    4 ----
 1 file changed, 4 deletions(-)

diff --git a/arch/powerpc/perf/req-gen/perf.h b/arch/powerpc/perf/req-gen/perf.h
index 59fa5888420d4..be31f3fea37da 100644
--- a/arch/powerpc/perf/req-gen/perf.h
+++ b/arch/powerpc/perf/req-gen/perf.h
@@ -38,7 +38,6 @@
 #define REQUEST_VALUE_(name_upper, r_name) REQUEST_VALUE__(name_upper, r_name)
 #define REQUEST_VALUE(r_name) REQUEST_VALUE_(NAME_UPPER, r_name)
 
-#include "_clear.h"
 #define REQUEST_(r_name, r_value, r_idx_1, r_fields) \
 	REQUEST_VALUE(r_name) = r_value,
 enum CAT2(NAME_LOWER, _requests) {
@@ -71,7 +70,6 @@ struct STRUCT_NAME(r_name) {				\
  * Generate a check of the field offsets
  * <NAME_LOWER>_assert_offsets_correct()
  */
-#include "_clear.h"
 #define REQUEST_(r_name, r_value, index, r_fields)			\
 r_fields
 #define __field_(r_name, r_value, r_idx_1, f_offset, f_size, f_name) \
@@ -106,7 +104,6 @@ static inline void CAT2(NAME_LOWER, _assert_offsets_correct)(void)
 #define EVENT_ATTR_NAME(r_name, c_name) \
 	EVENT_ATTR_NAME_(NAME_LOWER, r_name, c_name)
 
-#include "_clear.h"
 #define __field_(r_name, r_value, r_idx_1, f_offset, f_size, f_name)
 #define __array_(r_name, r_value, r_idx_1, a_offset, a_size, a_name)
 #define __count_(r_name, r_value, r_idx_1, c_offset, c_size, c_name)	\
@@ -130,7 +127,6 @@ PMU_EVENT_ATTR_STRING(							\
  *	&<NAME_LOWER>_event_attr_<request name>_<field name>.attr,
  * };
  */
-#include "_clear.h"
 #define __field_(r_name, r_value, r_idx_1, f_offset, f_size, f_name)
 #define __count_(r_name, r_value, r_idx_1, c_offset, c_size, c_name)	\
 	&EVENT_ATTR_NAME(r_name, c_name).attr.attr,