<div dir="auto"><div><br><div class="gmail_extra"><br><div class="gmail_quote">On Mar 22, 2017 2:44 AM, "Michel Dänzer" <<a href="mailto:michel@daenzer.net">michel@daenzer.net</a>> wrote:<br type="attribution"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="elided-text">On 22/03/17 06:46 AM, Marek Olšák wrote:<br>
> On Tue, Mar 21, 2017 at 10:27 PM, Nicolai Hähnle <<a href="mailto:nhaehnle@gmail.com">nhaehnle@gmail.com</a>> wrote:<br>
>> In the past, I was told off for patches that update this file without<br>
>> following the procedure described in include/drm/README. Tbh, that procedure<br>
>> causes some annoyances.<br>
>><br>
>> Anyway, it's definitely useful to have the patch out on the mailing list in<br>
>> any case.<br>
><br>
> Yeah, I know the correct process and I plan to ignore it this time if<br>
> I don't get too much backlash, because the alternative<br>
> (#ifdef/#define/#endif) is probably even worse.<br>
<br>
</div>FWIW, only AMDGPU_TILING_SET/GET need #undef,<br>
AMDGPU_TILING_SWIZZLE_MODE_<wbr>SHIFT/MASK and AMDGPU_FAMILY_AI can just be<br>
#defined directly, that way the preprocessor will warn if the<br>
definitions in libdrm and Mesa end up being inconsistent for some reason.<br>
<br>
<br>
The alternative is rushing out a libdrm release and making Mesa require<br>
that, right? That doesn't seem obviously better than a handful of<br>
temporary redundant defines in Mesa, hardly justification for bypassing<br>
the normal process.<br></blockquote></div></div></div><div dir="auto"><br></div><div dir="auto">I need a libdrm release because of the 3rd patch. I can't allow Mesa to run without that.</div><div dir="auto"><br></div><div dir="auto">Marek</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<font color="#888888"><br>
<br>
--<br>
Earthling Michel Dänzer               |               <a href="http://www.amd.com" rel="noreferrer" target="_blank">http://www.amd.com</a><br>
Libre software enthusiast             |             Mesa and X developer<br>
</font></blockquote></div><br></div></div></div>