[Mesa-dev] [PATCH 1/1] automake: r600 should only depend on libamd_common if opencl is enabled

[Jan Vesely]

On Fri, 2017-06-02 at 17:22 +0100, Emil Velikov wrote:
> On 2 June 2017 at 16:34, Jan Vesely <jan.vesely at 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.vesely at rutgers.edu> wrote:
> > > > Signed-off-by: Jan Vesely <jan.vesely at 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 at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: This is a digitally signed message part
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20170602/405a446a/attachment.sig>