Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1629680

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

Show all headers | View raw

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

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH v2] tools/iio: Use include/uapi with __EXPORTED_HEADERS__ Sekhar Nori <nsekhar@ti.com> - 2017-04-21 14:40 +0200
  Re: [PATCH v2] tools/iio: Use include/uapi with __EXPORTED_HEADERS__ Randy Dunlap <rdunlap@infradead.org> - 2017-04-23 04:30 +0200
    Re: [PATCH v2] tools/iio: Use include/uapi with __EXPORTED_HEADERS__ Sekhar Nori <nsekhar@ti.com> - 2017-04-24 13:50 +0200
      Re: [PATCH v2] tools/iio: Use include/uapi with __EXPORTED_HEADERS__ Randy Dunlap <rdunlap@infradead.org> - 2017-04-24 17:40 +0200
  Re: [PATCH v2] tools/iio: Use include/uapi with __EXPORTED_HEADERS__ Jonathan Cameron <jic23@kernel.org> - 2017-04-27 07:40 +0200

csiph-web