Skip to contentSkip to conversation

tslib: needs kernel headers 3.12

PATCH New Baruch Siach 4 participants 6 messages Jan 19, 2017 – Jan 25, 2017

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 tslib: needs kernel headers 3.12

Kernel headers 3.12+ are required since tslib version 1.3.

https://github.com/kergoth/tslib/blob/master/README.markdown

Update tslib reverse dependencies.

Fixes:
http://autobuild.buildroot.net/results/28d/28df38e2a042f1b5940bf35c18334ebf8810a348/
http://autobuild.buildroot.net/results/c38/c383cfee3673aa9fba36a86f1e68de63413c2987/
http://autobuild.buildroot.net/results/6dd/6ddb2cd9f4f41ee03dc10a73422ad37070811732/

---
 package/directfb/Config.in                       | 5 +++--
 package/qt/Config.mouse.in                       | 5 +++--
 package/qt5/qt5base/Config.in                    | 5 +++--
 package/tslib/Config.in                          | 5 +++--
 package/x11r7/xdriver_xf86-input-tslib/Config.in | 5 +++--
 5 files changed, 15 insertions(+), 10 deletions(-)
package/directfb/Config.in +3 -2
index b5521647772d..35c0309a6f36 100644
@@ -113,11 +113,12 @@ config BR2_PACKAGE_DIRECTFB_SERIALMOUSE
 config BR2_PACKAGE_DIRECTFB_TSLIB
 	bool "enable touchscreen support"
 	depends on !BR2_STATIC_LIBS # tslib
+	depends on BR2_TOOLCHAIN_HEADERS_AT_LEAST_3_12 # tslib
 	default y
 	select BR2_PACKAGE_TSLIB
 
-comment "touchscreen support needs a toolchain w/ dynamic library"
-	depends on BR2_STATIC_LIBS
+comment "touchscreen support needs a toolchain w/ dynamic library, headers >= 3.12"
+	depends on BR2_STATIC_LIBS || !BR2_TOOLCHAIN_HEADERS_AT_LEAST_3_12
 
 config BR2_PACKAGE_DIRECTFB_GIF
 	bool "enable GIF support"
package/qt/Config.mouse.in +3 -2
index 11da956f62f6..f15f8d5f5681 100644
@@ -13,9 +13,10 @@ config BR2_PACKAGE_QT_MOUSE_TSLIB
 	bool "tslib"
 	select BR2_PACKAGE_TSLIB
 	depends on !BR2_STATIC_LIBS # tslib
+	depends on BR2_TOOLCHAIN_HEADERS_AT_LEAST_3_12 # tslib
 
-comment "tslib support needs a toolchain w/ dynamic library"
-	depends on BR2_STATIC_LIBS
+comment "tslib support needs a toolchain w/ dynamic library, headers >= 3.12"
+	depends on BR2_STATIC_LIBS || !BR2_TOOLCHAIN_HEADERS_AT_LEAST_3_12
 
 config BR2_PACKAGE_QT_MOUSE_QVFB
 	bool "qvfb"
package/qt5/qt5base/Config.in +3 -2
index 7244fbedc68f..ad95138380c7 100644
@@ -267,11 +267,12 @@ config BR2_PACKAGE_QT5BASE_ICU
 config BR2_PACKAGE_QT5BASE_TSLIB
 	bool "Enable Tslib support"
 	depends on !BR2_STATIC_LIBS # dlopen
+	depends on BR2_TOOLCHAIN_HEADERS_AT_LEAST_3_12 # tslib
 	select BR2_PACKAGE_TSLIB
 	help
 	  This options enables the Tslib plugin
 
-comment "tslib support needs a toolchain w/ dynamic library"
-	depends on BR2_STATIC_LIBS
+comment "tslib support needs a toolchain w/ dynamic library, headers >= 3.12"
+	depends on BR2_STATIC_LIBS || !BR2_TOOLCHAIN_HEADERS_AT_LEAST_3_12
 
 endif
package/tslib/Config.in +3 -2
index 050588e834d4..6b86ebc4c97f 100644
@@ -1,10 +1,11 @@
 config BR2_PACKAGE_TSLIB
 	bool "tslib"
 	depends on !BR2_STATIC_LIBS # dlopen
+	depends on BR2_TOOLCHAIN_HEADERS_AT_LEAST_3_12
 	help
 	  Tslib is an abstraction layer for touchscreen panel events.
 
 	  https://github.com/kergoth/tslib
 
-comment "tslib needs a toolchain w/ dynamic library"
-	depends on BR2_STATIC_LIBS
+comment "tslib needs a toolchain w/ dynamic library, headers >= 3.12"
+	depends on BR2_STATIC_LIBS || !BR2_TOOLCHAIN_HEADERS_AT_LEAST_3_12
package/x11r7/xdriver_xf86-input-tslib/Config.in +3 -2
index 9c6e2961537c..79b739686ec3 100644
@@ -1,6 +1,7 @@
 config BR2_PACKAGE_XDRIVER_XF86_INPUT_TSLIB
 	bool "xf86-input-tslib"
 	depends on !BR2_STATIC_LIBS # tslib
+	depends on BR2_TOOLCHAIN_HEADERS_AT_LEAST_3_12 # tslib
 	select BR2_PACKAGE_XPROTO_INPUTPROTO
 	select BR2_PACKAGE_XPROTO_RANDRPROTO
 	select BR2_PACKAGE_XPROTO_XPROTO
@@ -10,5 +11,5 @@ config BR2_PACKAGE_XDRIVER_XF86_INPUT_TSLIB
 
 	  http://www.ptxdist.org/software/xf86-input-tslib/index_en.html
 
-comment "xf86-input-tslib needs a toolchain w/ dynamic library"
-	depends on BR2_STATIC_LIBS
+comment "xf86-input-tslib needs a toolchain w/ dynamic library, headers >= 3.12"
+	depends on BR2_STATIC_LIBS || !BR2_TOOLCHAIN_HEADERS_AT_LEAST_3_12
Signature · 1 line
2.11.0
#1 lore raw

PATCH 1/1 tslib: needs kernel headers 3.12

Quoted 5 lines
Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line

"Baruch" == Baruch Siach <[email protected]> writes:

Kernel headers 3.12+ are required since tslib version 1.3.
https://github.com/kergoth/tslib/blob/master/README.markdown

Quoted 8 lines

Update tslib reverse dependencies.

Fixes:
http://autobuild.buildroot.net/results/28d/28df38e2a042f1b5940bf35c18334ebf8810a348/
http://autobuild.buildroot.net/results/c38/c383cfee3673aa9fba36a86f1e68de63413c2987/
http://autobuild.buildroot.net/results/6dd/6ddb2cd9f4f41ee03dc10a73422ad37070811732/

Committed, thanks.

Signature · 1 line
Bye, Peter Korsgaard
#2 lore raw

PATCH 1/1 tslib: needs kernel headers 3.12

In reply to Baruch Siach

Hello,

Quoted 4 lines · Baruch Siach

Kernel headers 3.12+ are required since tslib version 1.3.

https://github.com/kergoth/tslib/blob/master/README.markdown

Quoted 8 lines

Update tslib reverse dependencies.

Fixes:
http://autobuild.buildroot.net/results/28d/28df38e2a042f1b5940bf35c18334ebf8810a348/
http://autobuild.buildroot.net/results/c38/c383cfee3673aa9fba36a86f1e68de63413c2987/
http://autobuild.buildroot.net/results/6dd/6ddb2cd9f4f41ee03dc10a73422ad37070811732/

Isn't this a bit annoying for people using tslib on kernel olders than 3.12 ? Isn't there a simpler patch to tslib to avoid depending on headers >= 3.12 ?

Thanks,

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
#3 lore raw

PATCH 1/1 tslib: needs kernel headers 3.12

Hi Thomas,

Quoted 12 lines · Thomas Petazzoni
Quoted 11 lines · Baruch Siach

Kernel headers 3.12+ are required since tslib version 1.3.

https://github.com/kergoth/tslib/blob/master/README.markdown

Update tslib reverse dependencies.

Fixes:
http://autobuild.buildroot.net/results/28d/28df38e2a042f1b5940bf35c18334ebf8810a348/
http://autobuild.buildroot.net/results/c38/c383cfee3673aa9fba36a86f1e68de63413c2987/
http://autobuild.buildroot.net/results/6dd/6ddb2cd9f4f41ee03dc10a73422ad37070811732/

Quoted 5 lines
Quoted 1 line

Isn't this a bit annoying for people using tslib on kernel olders than 3.12 ? Isn't there a simpler patch to tslib to avoid depending on headers >= 3.12 ?

The present build failure seems to be easy to fix by skipping the ts_uinput tool on pre 3.12 kernels. But since upstream explicitly states that 3.12 is the oldest supported kernel (see commit message link) we are on our own here.

baruch

Signature · 3 lines
     http://baruch.siach.name/blog/                  ~. .~   Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
   - baruch at tkos.co.il - tel: +972.52.368.4656, http://www.tkos.co.il -
#4 lore raw

PATCH 1/1 tslib: needs kernel headers 3.12

Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line

"Baruch" == Baruch Siach <[email protected]> writes:

Hi,

Quoted 7 lines
Quoted 3 lines

Isn't this a bit annoying for people using tslib on kernel olders than 3.12 ? Isn't there a simpler patch to tslib to avoid depending on headers >= 3.12 ?

The present build failure seems to be easy to fix by skipping the ts_uinput tool on pre 3.12 kernels. But since upstream explicitly states that 3.12 is the oldest supported kernel (see commit message link) we are on our own here.

I had the same idea originally, but as it isn't completely nontrivial to disable the new code and against the wishes of upstream I decided to leave that work for people needing it on pre-3.12 kernels.

With that said, if upstream isn't interested in integrating such a patch then it isn't really nice having to carry it in Buildroot either.

Signature · 1 line
Bye, Peter Korsgaard
#5 lore raw

PATCH 1/1 tslib: needs kernel headers 3.12

Quoted 12 lines · Peter Korsgaard
Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line

"Baruch" == Baruch Siach <[email protected]> writes:

Hi,

Quoted 7 lines
Quoted 3 lines

Isn't this a bit annoying for people using tslib on kernel olders than 3.12 ? Isn't there a simpler patch to tslib to avoid depending on headers >= 3.12 ?

The present build failure seems to be easy to fix by skipping the ts_uinput tool on pre 3.12 kernels. But since upstream explicitly states that 3.12 is the oldest supported kernel (see commit message link) we are on our own here.

Quoted 7 lines

I had the same idea originally, but as it isn't completely nontrivial to disable the new code and against the wishes of upstream I decided to leave that work for people needing it on pre-3.12 kernels.

With that said, if upstream isn't interested in integrating such a patch then it isn't really nice having to carry it in Buildroot either.

I (as the upstream maintainer) would happily take a patch that brings back support for old kernels. plugins/input-raw and tools/ts_uinput should be affected as they use multitouch.

Simply properly disabling them during configure would be an acceptable first step to have as a requirement for old kernels.

A github pull request or a patch here would be equally fine with me and I'd be happy to have it in version 1.4.