Re: [PATCH v2] tools/iio: Use include/uapi with __EXPORTED_HEADERS__

From: Randy Dunlap
Date: Mon Apr 24 2017 - 11:30:34 EST


On 04/24/17 04:40, Sekhar Nori wrote:
> Hi Randy,
>
> On Sunday 23 April 2017 07:52 AM, Randy Dunlap wrote:
>> On 04/21/17 05:31, Sekhar Nori wrote:
>>> Use the local uapi headers to keep in sync with "recently" added enum
>>> values like IIO_UVINDEX.
>>>
>>> Build tested using:
>>> $ make -C <kernelsrcdir> tools/iio
>>> $ make -C <kernelsrcdir>/tools iio
>>> $ make -C <kernelsrcdir>/tools/iio
>>
>> Hi,
>> Did you do any testing with O=<outputdir> ?
>
> I hadn't, but I just checked with:
>
> $ make -C <kernelsrcdir>/tools iio O=<outputdir>
>
> Although there is an iio folder created under <outputdir>, the actual
> binaries are still built under <kernelsrcdir>/tools/iio. So I think
> there is some more work to be done to respect the O= option passed from
> command line.
>
> The behavior remains same with or without my patch.

Yes, agreed. This is a common problem in tools.

Thanks.

--
~Randy