Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 5 Nov 2008 20:34:51 +0100
From:      bomberboy <bomberboy@gmail.com>
To:        freebsd-ports@freebsd.org
Subject:   Re: Wacom driver
Message-ID:  <6e77fe4e0811051134l110b1aceu4b9b53dace370219@mail.gmail.com>
In-Reply-To: <491173E4.1050809@bsdforen.de>
References:  <491173E4.1050809@bsdforen.de>

next in thread | previous in thread | raw e-mail | index | archive | help
------=_Part_2475_7729976.1225913691452
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit
Content-Disposition: inline

On Wed, Nov 5, 2008 at 11:22 AM, Dominic Fandrey <kamikaze@bsdforen.de> wrote:

Hello,

> I have created a port of Bartosz Fabianowski's Wacom driver.
> Since nobody has stepped forward to commit it, I thought I'd
> inform this list, so that people can test it.
>
> http://www.freebsd.org/cgi/query-pr.cgi?pr=128547

I tried compiling it my system but building the port fails for me.
Two diffs are attached, one for the Makefile of the port itself and
one for uwacom.c in the code of Bartosz Fabianowski.

My system is running a recent current btw:
# uname -a
FreeBSD Tablet.lan 8.0-CURRENT FreeBSD 8.0-CURRENT #2: Tue Nov  4
15:27:47 CET 2008     root@Tablet.lan:/usr/obj/usr/src/sys/TABLET
amd64

makefile.diff :
The Makefile of the port tries to create a link to the linuxwacom
archive in the work-dir of the port.
However, this archive is located in /usr/ports/distfiles/xorg/driver/
in stead of just /usr/ports/distfiles/ (or is this something specific
to current?)

uwacom.diff:
uwacom.c is missing the include <sys/clist.h>
Maybe this was indirectly included in a different header file in 6.x or 7.0?

After these changes the compile of uwacom.c still fails with:
=========
cc -O2 -pipe -fno-strict-aliasing -Werror -D_KERNEL -DKLD_MODULE
-std=c99 -nostdinc   -I. -I@ -I@/contrib/altq -finline-limit=8000
--param inline-unit-growth=100 --param large-function-growth=1000
-fno-common  -fno-omit-frame-pointer -mcmodel=kernel -mno-red-zone
-mfpmath=387 -mno-sse -mno-sse2 -mno-sse3 -mno-mmx -mno-3dnow
-msoft-float -fno-asynchronous-unwind-tables -ffreestanding
-fstack-protector -fstack-protector -Wall -Wredundant-decls
-Wnested-externs -Wstrict-prototypes  -Wmissing-prototypes
-Wpointer-arith -Winline -Wcast-qual  -Wundef -Wno-pointer-sign
-fformat-extensions -c uwacom.c
cc1: warnings being treated as errors
uwacom.c: In function 'uwacom_init':
uwacom.c:1072: warning: left shift count >= width of type
*** Error code 1

Stop in /usr/ports/x11-drivers/xf86-input-wacom/work/uwacom.
*** Error code 1
=========

Removing -Werror from the compile step silently ignores this warning,
but I guess that's not the right way to deal with this :)
I'm not familiar with the code at all and don't really know how to
solve this, but maybe you do.

Can't verify if it works though, I'm using it on a Tablet PC (Fujitsu
T4220) for which the wacom device should show up as a serial device.
(However, I'm still having some problems with that though)

As a sidenote. The tablets which use a serial port (e.g. most
tabletPCs according to the linuxwacom documentation) don't need the
the usb-part if I'm correct. So maybe it would be a good idea to split
the port into two parts? One just for the linuxwacom codebase, and one
for the kernel module? (which then depends on the linuxwacom port)


Kind regards,
Bruno

------=_Part_2475_7729976.1225913691452
Content-Type: application/octet-stream; name=makefile.diff
Content-Transfer-Encoding: base64
X-Attachment-Id: f_fn6cuf3w0
Content-Disposition: attachment; filename=makefile.diff

LS0tIE1ha2VmaWxlLm9yaWcJMjAwOC0xMS0wNSAyMDoxMDo1Mi4wMDAwMDAwMDAgKzAxMDAKKysr
IE1ha2VmaWxlCTIwMDgtMTEtMDQgMTc6MDY6MTYuMDAwMDAwMDAwICswMTAwCkBAIC0zNSw3ICsz
NSw3IEBACiAKIGRvLWNvbmZpZ3VyZToKIAlAY2QgJHtXUktESVJ9L2xpbnV4d2Fjb20gXAotCQkm
JiAke0xOfSAtcyAke0RJU1RESVJ9LyR7TElOVVhXQUNPTX0udGFyLmJ6MiBcCisJCSYmICR7TE59
IC1zICR7RElTVERJUn0veG9yZy9kcml2ZXIvJHtMSU5VWFdBQ09NfS50YXIuYnoyIFwKIAkJJiYg
Li9ydW5fY29uZmlndXJlCiAKIGRvLWJ1aWxkOgo=
------=_Part_2475_7729976.1225913691452
Content-Type: application/octet-stream; name=uwacom.diff
Content-Transfer-Encoding: base64
X-Attachment-Id: f_fn6cwbyl1
Content-Disposition: attachment; filename=uwacom.diff

LS0tIHV3YWNvbS5jLm9yaWcJMjAwOC0xMS0wNSAyMDoxNTo1MC4wMDAwMDAwMDAgKzAxMDAKKysr
IHV3YWNvbS5jCTIwMDgtMTEtMDUgMjA6MTY6MTIuMDAwMDAwMDAwICswMTAwCkBAIC04NSw2ICs4
NSw3IEBACiAjaW5jbHVkZSA8c3lzL3Vpby5oPgogI2luY2x1ZGUgPHN5cy9maWxpby5oPgogI2lu
Y2x1ZGUgPHN5cy9wb2xsLmg+CisjaW5jbHVkZSA8c3lzL2NsaXN0Lmg+CiAKICNpbmNsdWRlIDxk
ZXYvdXNiL3VzYi5oPgogI2luY2x1ZGUgPGRldi91c2IvdXNiZGkuaD4K
------=_Part_2475_7729976.1225913691452--



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?6e77fe4e0811051134l110b1aceu4b9b53dace370219>