On Fri, 2017-06-02 at 17:22 +0100, Emil Velikov wrote: > On 2 June 2017 at 16:34, Jan Vesely <jan.ves...@rutgers.edu> wrote: > > On Fri, 2017-06-02 at 12:19 +0100, Emil Velikov wrote: > > > On 1 June 2017 at 21:28, Jan Vesely <jan.ves...@rutgers.edu> wrote: > > > > Signed-off-by: Jan Vesely <jan.ves...@rutgers.edu> > > > > --- > > > > Hi guys, > > > > > > > > this is the first step towards dropping libamd_common dependency. > > > > It's based on Emil's patches 3/5 and 4/5. > > > > Enabling opencl still falls back to the old way of requiring > > > > libamd_common. > > > > I'll try to address that in the next step (no time estimate, feel > > > > free to beat me to it). I think we can drop part of those functions > > > > rather than just copying them. > > > > > > > > > > AFAICT one still need the rest of my series, correct? > > > > kind of, 1/5 is mostly unrelated. 2,3/5 should be replaced by this one. > > 4/5 is needed, > > and 5/5 seems to be not applicable since ac_gpu_info.c > > still needs the header. > > > > Hmm indeed. I seems to have misread your patch. > Which makes me wonder if you've tested the patch as I mentioned earlier: > - (re)move amdgpu.h > - apply mesa patches (be that any of my, your and other mix) > - build r300 and/or r600, w/o radeonsi > - the the above combo a try with and w/o opencl > > I'm leaning that things will fail to build?
This patch only removes dependence on libamd_common unless you enable OpenCL. I haven't addressed the hacky way of using ac_gpu_info.h Applying 4/5v2 from your series should work OK on top of my patch. Jan > > -Emil > _______________________________________________ > mesa-dev mailing list > mesa-dev@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/mesa-dev
signature.asc
Description: This is a digitally signed message part
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
