<div dir="ltr"><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Jun 21, 2019 at 1:13 PM Emil Velikov <<a href="mailto:emil.l.velikov@gmail.com">emil.l.velikov@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">From: Emil Velikov <<a href="mailto:emil.velikov@collabora.com" target="_blank">emil.velikov@collabora.com</a>><br>
<br>
Currently libdrm_amdgpu provides a typedef of the various handles. While<br>
the goal was to make those opaque, it effectively became part of the API<br>
<br>
To the best of my knowledge there are two ways to have opaque handles:<br>
 - "typedef void *foo;" - rather messy IMHO<br>
 - "stuct foo;" and use "struct foo *" through the API<br>
<br>
In our case amdgpU_device_handle is used only internally, plus<br>
respective code is not used or applicable for r300 and r600. Hence  we<br>
copied the typedef.<br>
<br>
Seemingly this will be a problem since a libdrm_amdgpu wants to change<br>
the API, while not updating the code(?).<br></blockquote><div><br></div><div>libdrm_amdgpu can't change the API.</div><div><br></div><div>Reviewed-by: Marek Olšák <<a href="mailto:marek.olsak@amd.com">marek.olsak@amd.com</a>></div><div><br></div><div>Marek</div><br></div></div>