<div dir="ltr">Hi Valentin,<br>Thanks for catching that.<div>I would be grateful if you could fix this yourself.</div><div><br></div><div>Oded</div></div><br><div class="gmail_quote"><div dir="ltr">On Thu, Jun 4, 2015 at 4:45 PM Valentin Rothberg <<a href="mailto:valentinrothberg@gmail.com">valentinrothberg@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Yair,<br>
<br>
your commit fbeb661bfa89 ("drm/amdkfd: Add skeleton H/W debugger<br>
module support") has shown up in today's linux-next tree (i.e.,<br>
next-20150604).  The commit adds the following lines of code to<br>
drivers/gpu/drm/amd/amdkfd/kfd_dbgdev.h:<br>
<br>
+/* CONFIG reg space definition */<br>
+enum {<br>
+       CONFIG_REG_BASE = 0x2000,       /* in dwords */<br>
+       CONFIG_REG_END = 0x2B00,<br>
+       CONFIG_REG_SIZE = CONFIG_REG_END - CONFIG_REG_BASE<br>
+};<br>
<br>
There is a problem with the 'CONFIG_' prefix of those entries.  This<br>
prefix is reserved for Kconfig options in Make/Kbuild and CPP syntax,<br>
so that static analysis tools (and readers of the code) may mistakenly<br>
assume that the symbol is defined somewhere in a Kconfig file.<br>
<br>
I detected the issue with ./scripts/checkkconfigsymbols.py.  Would you<br>
mind renaming those entries to something without the 'CONFIG_' prefix?<br>
 I can also take care of it if you wish to.<br>
<br>
Kind regards,<br>
 Valentin<br>
</blockquote></div>