[Buildroot] [PATCH] spidev_test: new package

Baruch Siach baruch at tkos.co.il
Sat Oct 18 20:08:26 UTC 2014


Hi Yann,

On Fri, Oct 17, 2014 at 11:55:39PM +0200, Yann E. MORIN wrote:
> On 2014-10-14 22:54 +0300, Baruch Siach spake thusly:
> > On Tue, Oct 14, 2014 at 12:45:53PM -0300, Gustavo Zacarias wrote:
> > > Signed-off-by: Gustavo Zacarias <gustavo at zacarias.com.ar>
> > 
> > [snip]
> > 
> > > diff --git a/package/spidev_test/Config.in b/package/spidev_test/Config.in
> > > new file mode 100644
> > > index 0000000..c0ed9ef
> > > --- /dev/null
> > > +++ b/package/spidev_test/Config.in
> > > @@ -0,0 +1,15 @@
> > > +config BR2_PACKAGE_SPIDEV_TEST
> > > +	bool "spidev_test"
> > > +	depends on BR2_LINUX_KERNEL
> > 
> > That is a heavy dependency. As you noted in another message, it also make the 
> > autobuilder skip this package. I suggest to download just spidev_test.c from 
> > http://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/plain/Documentation/spi/spidev_test.c?id=$(SPIDEV_TEST_VERSION), 
> > and make it depend on BR2_TOOLCHAIN_HEADERS_AT_LEAST_3_15 (as of v3.17). This 
> 
> What about directly getting the appropriate one, something like:
> 
>     ifeq ($(BR2_TOOLCHAIN_HEADERS_AT_LEAST_3_15),y)
>     SPIDEV_TEST_VERSION = v3.15
>     else
>     SPIDEV_TEST_VERSION = v3.14
>     endif
> 
>     SPIDEV_TEST_SOURCE =
>     SPIDEV_TEST_EXTRA_DOWNLOADS = http://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/plain/Documentation/spi/spidev_test.c?id=$(SPIDEV_TEST_VERSION)
> 
> Since, without the proper kernel headers, it is not possible to build
> the quad-pump blabla, we can directly get the spidev_test that will
> indeed compile with our toolchain, no?

As Gustavo mentioned in another message in this thread, the newer spidev_test 
is also run-time dependent on kernel v3.15 or newer. An automatic spidev_test 
version choice would produce a broken binary for users of newer toolchains 
with pre v3.15 kernels. We should use BR2_TOOLCHAIN_HEADERS_AT_LEAST_3_15 to 
disable build of v3.15 spidev_test, as this would break the build. But I don't 
think we want to force v3.15 spidev_test just because the toolchain kernel 
headers are post v3.15.

baruch

-- 
     http://baruch.siach.name/blog/                  ~. .~   Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
   - baruch at tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il -



More information about the buildroot mailing list