Don't free anything in PreInit, provide an UnInit instead.

Also fixes:
- leaking priv->buffer
- fd closure bug

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

Dont call xf86DeleteInput on PreInit failure.

The server calls this for us once PreInit returns with an error,
calling it during PreInit means the server continues to use
already free'd memory.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

Always set the type name.

The server assigns "UNKNOWN", override it with something more specific.

Introduced in f85c4b580c074f7054eac98753d1f4e91f08305e.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

Assign local->private after allocating.

It is detrimental to the user experience when the driver tries to derefernce
null pointers.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

Remove pointless checks on is_core_pointer.

Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Daniel Stone <daniel@fooishbar.org>
Signed-off-by: Cyril Brulebois <kibi@debian.org>

Remove checks on the ABI now that 12 or higher is required.

Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Daniel Stone <daniel@fooishbar.org>
Signed-off-by: Cyril Brulebois <kibi@debian.org>

Drop support for pre-input ABI 12 servers.

Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Daniel Stone <daniel@fooishbar.org>
Signed-off-by: Cyril Brulebois <kibi@debian.org>

Fix “initialization from incompatible pointer type” for default_options.

InputDriverRec's last field is a char**, not a const char**, fix this
warning accordingly:
|   CC     acecad.lo
| acecad.c:132:1: warning: initialization from incompatible pointer type [enabled by default]
| acecad.c:132:1: warning: (near initialization for 'ACECAD.default_options') [enabled by default]

Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Daniel Stone <daniel@fooishbar.org>
Signed-off-by: Cyril Brulebois <kibi@debian.org>

Silence gcc: report_{x, y} are getting initialized.

Get us rid of:
|   CC     acecad.lo
| acecad.c: In function 'USBReadInput':
| acecad.c:1052:43: warning: 'report_y' may be used uninitialized in this function [-Wuninitialized]
| acecad.c:1052:43: warning: 'report_x' may be used uninitialized in this function [-Wuninitialized]

Both code paths (with XORG_BOTCHED_INPUT or not) will lead report_x and
report_y to be set, but make sure the compiler stops guessing.

Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Daniel Stone <daniel@fooishbar.org>
Signed-off-by: Cyril Brulebois <kibi@debian.org>

Avoid “'ReverseConvertProc' defined but not used” warning.

Move this function next to its unique caller (AceCadPreInit), and only
declare/define it when the input ABI is strictly less than 12.

Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Daniel Stone <daniel@fooishbar.org>
Signed-off-by: Cyril Brulebois <kibi@debian.org>

acecad 1.5.0

Signed-off-by: Cyril Brulebois <kibi@debian.org>

The driver doesn't need libXext

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Gaetan Nadon <memsize@videotron.ca>

Bump to 1.4.99

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

Add mode field to InitValuatorAxisStruct

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

Adjust to new PreInit for ABI 12.

Most of the init is now handled by the server, not by the driver.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

Use a local variable for history_size.

Field was removed from InputInfoRec in ABI and the server hasn't actually
accessed this field for quite a while.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

Purge close_proc, hasn't been called by the server for ages.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

Move default_options up, preparation work for ABI 12.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

Remove libc wrapper usage for xcalloc, xfree, etc.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

Replace LocalDevicePtr with InputInfoPtr.

LocalDevicePtr was removed with input ABI 12, but InputInfoPtr has been
around for years anyway.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

Remove usage of XI86_POINTER_CAPABLE.

Flag was a write-only flag in both the server and the driver has now been
removed from the server.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

Remove refcnt field from InputDriverRec.

Wasn't used in the server and is now removed.

Fixes compiler error:
"acecad.c", line 111: too many struct/union initializers

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

config: requires RANDR through xf86.h

The xf86.h file contains external declarations specific to RANDR
Will not compile if this extension is not installed.
Because this is an optional extension, XORG_DRIVER_CHECK_EXT
must be used.

Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

config: requires xext, kb and input extensions through xf86.h

The xf86.h file includes headers from these protocols.
Will not compile if these extensions are not installed.
Because these extensions are always defined in the server, XORG_DRIVER_CHECK_EXT
should not be used.

Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

config: use AC_PROG_INSTALL now supplied by XORG_DEFAULT_OPTIONS

It depends on util-macros 1.8
The existing statement can now be removed from the configuration file.

Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

config: upgrade to util-macros 1.8 for additional man page support

Use MAN_SUBST now supplied in XORG_MANPAGE_SECTIONS
The value of MAN_SUBST is the same for all X.Org packages.

Use AC_PROG_SED now supplied by XORG_DEFAULT_OPTIONS
The existing statement can now be removed from the configuration file.

Use Automake recommened $() for makefile variables.

Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

config: acecad does not use the input protocol

There is no need to PKG_CHECK_MODULES inputproto

Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

config: AC_PROG_SED is required explicitly on Solaris

It sets the SED env variable with an appropriate sed program path

Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

man: Use Autoconf provided $(AM_V_GEN)$(SED)

Enables silent rule and use platform appropriate version of sed.

Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

config: optional extension check for inputproto not required

The check should be in PKG_CHECK_MODULES permanently.

The XORG_DRIVER_CHECK_EXT will add inputproto to PKG_CHECK_MODULES
only if the extension is available. By definition, this extension
is always available and the acecad driver cannot function without it.

This is evidenced by the absence of XINPUT conditional code.

Acked-by: Peter Hutterer <peter.hutterer@who-t.net>
Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

config: remove extension check for unused RANDR extension

The randr protocol is not used, no randr*.h are included.

Acked-by: Peter Hutterer <peter.hutterer@who-t.net>
Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

config: fix checking for extensions section

A hidden dependency on statement order was discovered.
The REQUIRE_MODULES variable was removed but needed.
Comments added to clarify the situation.

Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

config: remove AC_PROG_CC as it overrides AC_PROG_C_C99

XORG_STRICT_OPTION from XORG_DEFAULT_OPTIONS calls
AC_PROG_C_C99. This sets gcc with -std=gnu99.
If AC_PROG_CC macro is called afterwards, it resets CC to gcc.

Acked-by: Peter Hutterer <peter.hutterer@who-t.net>
Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

config: fix warnings and layout

Minor upgrades. Layout and comment statements.
Remove AC_PROG_CC as it overrides AC_PROG_C_C99 from
XORG_STRICT_OPTION.

Acked-by: Peter Hutterer <peter.hutterer@who-t.net>
Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

config: replace deprecated AM_CONFIG_HEADER with AC_CONFIG_HEADERS

Regroup AC statements at the top.

Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

config: update AC_PREREQ statement to 2.60

Unrelated to the previous patches, the new value simply reflects
the reality that the minimum level for autoconf to configure
all x.org modules is 2.60 dated June 2006.

ftp://ftp.gnu.org/gnu/autoconf/autoconf-2.60.tar.gz

Acked-by: Peter Hutterer <peter.hutterer@who-t.net>
Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

config: simplify configuration regarding sysfs headers and library.

Replace deprecated AC_HEADER_CHECK with AC_CHECK_HEADERS
AC_CHECK_LIB ensures sysfs library is present and adds -lsysfs
AC_CHECK_HEADERS verifies sysfs and linux input headers are present

Acked-by: Peter Hutterer <peter.hutterer@who-t.net>
Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

config: add conditional inclusion of config.h

Acked-by: Peter Hutterer <peter.hutterer@who-t.net>
Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

config: remove AH_TOP autoheader statement

The generated config.h does not need to include xorg-server.h
for the content it provides.
Add #include <xorg-server.h> in .[hc] files as needed.

Acked-by: Peter Hutterer <peter.hutterer@who-t.net>
Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

config: move CWARNFLAGS from configure.ac to Makefile.am

Compiler warning flags should be explicitly set in the makefile
rather than being merged with other packages compiler flags.

Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

Update Sun license notices to current X.Org standard form

Signed-off-by: Alan Coopersmith <alan.coopersmith@sun.com>

configure.ac: remove unused sdkdir=$(pkg-config...) statement

The sdkdir variable isn't use, so remove the statement.

Acked-by: Dan Nicholson <dbn.lists@gmail.com>
Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

Makefile.am: add ChangeLog and INSTALL on MAINTAINERCLEANFILES

Now that the INSTALL file is generated.
Allows running make maintainer-clean.

INSTALL, NEWS, README or AUTHORS files are missing/incorrect #24206

Automake 'foreign' option is specified in configure.ac.
Remove from Makefile.am

INSTALL, NEWS, README or AUTHORS files are missing/incorrect #24206

Add missing INSTALL file. Use standard GNU file on building tarball
README may have been updated
Remove AUTHORS file as it is empty and no content available yet.
Remove NEWS file as it is empty and no content available yet.

Several driver modules do not have a ChangeLog target in Makefile.am #23814

The git generated ChangeLog replaces the hand written one.
Update configure.ac to xorg-macros level 1.3.
Use XORG_DEFAULT_OPTIONS which replaces four XORG_* macros
Update Makefile.am to add ChangeLog target if missing
Remove ChangeLog from EXTRA_DIST or *CLEAN variables
This is a pre-req for the INSTALL_CMD

.gitignore: use common defaults with custom section # 24239

Using common defaults will reduce errors and maintenance.
Only the very small or inexistent custom section need periodic maintenance
when the structure of the component changes. Do not edit defaults.

Remove some unused crap from configure

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

acecad 1.4.0

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

Require xorg-macros 1.3 for XORG_DEFAULT_OPTIONS

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

Remove redefinition of read()

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

Remove unused code from TearDownProc

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

unifdef XFree86LOADER, this isn't optional anymore anyway.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

Remove RCS tags

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

remove un-used _ACECAD_C_ define

xf86-input-acecad: Remove unused .cvsignore file #23776

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

Cope with ABI_XINPUT_VERSION 7 - requires button/axes labels.

We don't have an atom for relative pressure, so if the device is set to
relative, we export the axis as REL_Z instead.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

Fill in COPYING file, add SubmittingPatches URL to README

acecad 1.3.0

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

Janitor: Correct make distcheck and minor configure.ac cleanup.

Add README with pointers to mailing list, bugzilla & git repos

Remove xorgconfig & xorgcfg from See Also list in man page

Correct wrong check for Linux libsysfs.

  This patch was the original one, and the previous test was a wrong
attempt to simplify the change.
  Triggered by tinderbox build.

Dont dlopen libsysfs.so, instead, link with it when available.

Check for XINPUT ABI 3.

Compile warning fixes.

Don't call xf86usleep, just call usleep directly.
Remove unused variables.

Signed-off-by: Peter Hutterer <peter@cs.unisa.edu.au>

Bump to 1.2.2

acecad: fix blotchy input in Inkscape and the Gimp by not pretending to be the core pointer by default

acecad: improve handling of botched input versions

acecad: fake device limits == screen limits in xserver 1.4 for xinput

xserver 1.4 does not do automatic coordinate scaling, so we have to do
it manually. But this breaks xinput clients that do their own scaling
using the device limits as reference. Pretend that they are the same as the
screen limits to work around this.

acecad: set type_name to XI_TABLET

acecad: do our own scaling with USB device since it's not done in the kernel and ConverProc() isn't called automatically by X

acecad: don't crash when xf86IsCorePointer is not defined

Bump to 1.2.1

Make compatible with new input ABI

Don't crash X when the tablet wasn't found.

X would crash when switching to console if the acecad module was loaded but the tablet had not been configured.

Fix by properly freeing structures before returning NULL in PreInit()

fix build on BSD systems.

Add .gitignore

Off-by-one (and comment) fix.

Just a quick patch:

Bump to 1.2.0.

Simplify flag setup.

More whitespace/coding style cleanups.

Centralize the success code for the probe routine.

Keep track of device availability.

Don't spam the log with errors about the device not being available, just inform the user when the device is available again.

NOTAVAIL macro to check if errno means device not available.

Define (and use) a NOTAVAIL macro that checks errno against ENODEV, ENXIO and ENOENT, instead of checking manually against ENODEV only.

Make acecadAutoDev into a flag.

Introduce AUTODEV_FLAG for the flags field in AceCadPrivateRec to tell if we are autodetecting devices or not, instead of having a separate acecadAutoDev field.

Make acecadUSB into a flag.

Introduce USB_FLAG for the flags field in AceCadPrivateRec to tell if we have a USB device or not, instead of having a separate acecadUSB field.

More whitespace cleanup.

More coding style cleanups.

Add missing newlines in a couple of messages.

Whitespace cleanup.

De-spammify AceCadAutoDevProbe().

Demote X_ERRORs in AceCadAutoDevProbe() to X_WARNINGs, and add an X-style
verbosity parameter. This is set to 0 during initial probe, and to 4 during
runtime probes.

Re-probe on device not found on input if auto-dev is active.

If USBReadInput() fails because the device is not being found, and auto-dev is
active, try re-probing for the device. The only downside of this procedure is
that the log may get spammed a lot as long as no device can be found.

Re-probe for device if DeviceOn fails and auto-dev is active.

If DeviceOn does not find the device and auto-dev is active, run the probe
again to try to find the new location of the device (if present).

Device autodetection via sysfs.

If support for sysfs is compiled in and the libsysfs library is found, the
acecad driver will now try to autodetect the device via sysfs at first. If this
fails, it will probe /dev/input/eventX devices.

Currently the Name option is ignored during sysfs detection: the first possibly
valid candidate is accepted without further checks.

"Name" option to configure autodev detection string.

The string we look for when probing event devices can now be configured with
the "Name" option.

Merge branch 'master' into autodev

Add autogen.sh and autogenerate ChangeLog to dist

Get rid of the manually maintained ChangeLog and put in some rules to
autogenerate it from git logs, following the xorg/lib/libXi example.

Also add ChangeLog and autogen.sh to the dist files.

Add autoconf check for linux/input.h

Since X went modular, we cannot rely on LINUX_INPUT being defined even when the
Linux input subsystem is available. Fix: add an autoconf rule that checks for
linux/input.h and define LINUX_INPUT if the header is found.

Initial support for device autodetection.

When the "Device" option is missing or set to "auto-dev" the acecad module will
attempt autodetection of the correct device.

Currently, it will probe all /dev/input/eventX nodes until one is found that
reports a device name that begins with "ACECAD".