am7xxx: add support for setting the power mode on the PicoPix 2055

am7xxx: dump fields of unknown packet types

Allow dumping the payload fields as generic header_data, this is useful
when the packet type cannot be handled explicitly because the meaning of
the data is not known.

am7xxx: factor out a send_command() function

Some packet types do not carry any header_data and thus they can be seen
as commands issued to the device.

Add a send_command() function to handle such packet types, others of
them are going to be added in future commits.

am7xxx: switch from device quirks to device ops

Define device specific operations in order to allow supporting different
implementation of a given operation by different device models.

For instance this will be used to add an alternative implementation of
the 'set_power_mode' operation for the Philips/SagemCom PicoPix 2055
projector.

For now device specific callbacks are used only for the 'set_power_mode'
and 'set_zoom_mode' operations.

debian/control: drop the am7xxx metapackage

Recommend libam7xxx0.1-bin for libam7xxx0.1 and recommend
libam7xxx0.1-doc for libam7xxx-dev.

debian/control: specify the debian package branch in Vcs-Git

This is what is suggested in
http://www.debian.org/doc/debian-policy/ch-controlfields.html#s-f-VCS-fields

debian/control: bump Standards-Version to 3.9.5

debian/gbp.conf: explicitly disable pristine-tar

This fixes build when users have pristine-tar set to True in the global
configuration file.

debian/control: drop the version on the asciidoc dependency

Also drop the dependency on docbook-xsl as it will be brought in by
xmlto.

doc/lsusb_dumps: add lsusb_Philips-PicoPix-2055.log

doc/lsusb_dumps: strip trailing spaces

This makes it easier to compare different files with diff.

doc/man: misc fixes to man pages

The help text in particular was copied from the output of the '-h'
option which was referring to "this help message", this was OK on the
command line but it sounds a little off in a man page.

debian/patches: add 0003-CMakeLists.txt-define-DEBUG-1-only-when-CMAKE_BUILD_.patch

DEBUG=1 should not be enabled for Debian packages which are built with
CMAKE_BUILD_TYPE=RelWithDebInfo.

CMakeLists.txt: define DEBUG=1 only when CMAKE_BUILD_TYPE=Debug

When building with CMAKE_BUILD_TYPE=RelWithDebInfo the development debug
code should not be enabled, so don't define DEBUG=1 in DEBUG_FLAGS as
this will end up being both in CMAKE_C_FLAGS_DEBUG and
CMAKE_C_FLAGS_RELWITHDEBINFO.

Instead define DEBUG=1 only for CMAKE_C_FLAGS_DEBUG so it gets used only
when CMAKE_BUILD_TYPE=Debug.

debian/control: remove versions from libav dependencies

HACKING.asciidoc: fix the url of the avcodec_encode_video2 patch

am7xxx-play: cosmetics, fix coding style

debian/changelog: release Debian package 0.1.4-2

Git-Dch: ignore

debian/patches: add 0002-am7xxx-add-quirks-for-Philips-Sagemcom-PicoPix-2055.patch

With this patch PicoPix 2055 works more reliably.

am7xxx: add quirks for Philips/Sagemcom PicoPix 2055

The Philips/Sagemcom PicoPix 2055 does not support some operations:
  - setting the power mode confuses the firmware
  - setting the zoom mode has no effect whatsoever

Disable these operations.

Thanks-to: Luca Bompani

NEWS: fix a typo s/mode/more/

debian/changelog: package upstream version v0.1.4

Git-Dch: ignore

debian/control: update Standards-Version to 3.9.4

debian/control: fix a not-binnmuable-any-depends-all lintian error

E: libam7xxx source: not-binnmuable-any-depends-all am7xxx -> libam7xxx-doc

debian/control: update the list of supported devices

debian/libam7xxx0.1.symbols: add the am7xxx_send_image_async symbol

debian/patches: refresh patches for v0.1.4

  - Drop 0001-doc-add-a-man-page-for-am7xxx_mode_switch.patch it has
    been merged upstream
  - Rename 0002-Revert-am7xxx-play-switch-to-avcodec_encode_video2.patch
    to 0001-Revert-am7xxx-play-switch-to-avcodec_encode_video2.patch as
    it's now the first patch
  - Refresh patch to apply cleanly to upstream version 0.1.4
    0001-Revert-am7xxx-play-switch-to-avcodec_encode_video2.patch

Merge tag 'v0.1.4' into debian

Release version 0.1.4

Release version 0.1.4

contrib: add some benchmarking data about am7xxx_send_image_async

am7xxx-play: fix a crash when a packet cannot be encoded

am7xxx-play: don't initialize variables when not needed

TODO: mention that atoi() must go away

picoproj: get rid of exit(), return more meaningful values to userspace

Maybe the values returned by picoproj will never be used, but it is
better to use a clean style in example code.

HACKING.asciidoc: add commands to compile with clang

HACKING.asciidoc: add an example of testing am7xxx-play with valgrind

This will make testing easier as the line can be copied and pasted.

doc: update Doxyfile.in

contrib: add a udev rule to invoke am7xxx-modeswitch

This will be useful on embedded systems like RaspberryPi or BeagleBone
where users may not want the whole usb-modeswitch package to be
installed just for libam7xxx.

Rename am7xxx_mode_switch to am7xxx-modeswitch

The new name matches more closely the name of the binary used in recent
usb-modeswitch versions.

TODO: mention that data types could be improved in the API

picoporj: fix another -Wshorten-64-to-32 warning from clang

  ../examples/picoproj.c:196:8: error: implicit conversion loses integer precision: 'size_t'
        (aka 'unsigned long') to 'int' [-Werror,-Wshorten-64-to-32]
          ret = fread(image, size, 1, image_fp);
              ~ ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

picoproj: silence a -Wshorten-64-to-32 warning from clang

  ../examples/picoproj.c:187:12: error: implicit conversion loses integer precision: '__off_t' (aka 'long')
        to 'unsigned int' [-Werror,-Wshorten-64-to-32]
          size = st.st_size;
               ~ ~~~^~~~~~~

CMakeLists.txt: disable -Wsign-conversion warnings

am7xxx: fix a clang warning

  ../src/am7xxx.c:685:16: error: implicit conversion loses integer precision: 'ssize_t' (aka 'long') to 'int'
        [-Werror,-Wshorten-64-to-32]
          num_devices = libusb_get_device_list(ctx->usb_context, &list);
                    ~ ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

libusb_get_device_list() returns ssize_t, just use it for the num_devices
variable.

examples: silence a couple of clang warnings

Silence a couple of warnings given when compiling with clang:

  ../examples/am7xxx-play.c:764:28: warning: will never be executed [-Wunreachable-code]
                                          AM7XXX_ZOOM_ORIGINAL, AM7XXX_ZOOM_TEST);
                                                                ^~~~~~~~~~~~~~~~
  ../examples/am7xxx-play.c:748:24: warning: will never be executed [-Wunreachable-code]
                                          AM7XXX_POWER_OFF, AM7XXX_POWER_TURBO);
  2 warnings generated.

  ../examples/picoproj.c:142:28: warning: will never be executed [-Wunreachable-code]
                                          AM7XXX_ZOOM_ORIGINAL, AM7XXX_ZOOM_TEST);
                                                                ^~~~~~~~~~~~~~~~
  ../examples/picoproj.c:127:24: warning: will never be executed [-Wunreachable-code]
                                          AM7XXX_POWER_OFF, AM7XXX_POWER_TURBO);
  2 warnings generated.

The warnings refer to the default case of a switch statement over an
enum variable, clang fails to understand that the default case can still
be reached because any integer value can be assigned to the enum
variables via the atoi() function. Work around the warning.

The behavior is also documented here:
http://lists.cs.uiuc.edu/pipermail/cfe-dev/2012-September/024174.html

In general assigning arbitrary values to enum variables will just be
weird; using enums in read mode is always safe tho.

doc: mention the Top-Height/TEC PP700 in the Doxygen documentation

doc: add some lsusb dumps for reference

am7xxx-play: use am7xxx_send_image_async()

Woot, this change alone doubles the frame rate :)

am7xxx: implement am7xxx_send_image_async()

Implement am7xxx_send_image_async() the non-blocking version of
am7xxx_send_image().

This way user programs can more easily overlap encoding and
communication and achieve higher framerates.

am7xxx: fix a typo in a comment s/a am7xxx device/an am7xxx device/

CMakeLists.txt: enable two new compiler warnings

Enable -Wsign-conversion as a strict compilation check for both gcc and
clang.

For clang also enable -Wshorten-64-to-32 as a strict compilation check.

am7xxx: add quirks for devices not supporting some operations

For instance Philips/SagemoCm PicoPix PPX 2330 does not support power
modes and zoom modes, and the device even gets confused when trying to
use them; so for reliable operation it is better to just skip these
operations on such devices.

Thanks-to: Grégory Lemesre

picoproj: remove an unreachable break statement

am7xxx-play: uniform coding style

Uniform the coding style of error paths to the one use throughout in the
file.

CMakeLists.txt: add support for clang and isolate gcc-only options

HACKING.asciidoc: mention the patch needed for older libav/ffmpeg

libav/ffmpeg 0.8.X series do not have avcodec_encode_video2() which the
code uses, so for these older versions a patch is needed to undo the
changes which added support for that call.

README.asciidoc: mention the TEC PP700 projector as supported

Mention the Top-Height/TEC PP700 amongst the supported devices, it has
been verified to be work with libam7xxx (thanks to the user nicknamed
MHz).

The projector looks exactly like an Acer C112 from the software point of
view.

picoproj: remove an unneeded blank line

examples: print the usage message when a required option is missing

While at it specify which option is missing when picoproj fails..

CMakeLists.txt: fix enabling verbose debug output

In 024af793fa6d6af644c225804d140cc13aa56307 the definition -DDEBUG=1 got
lost, add it back.

am7xxx: use the symbolic constant for libusb log level

picoproj: show the image resolution when image does not fit the native one

This may help spotting a missing or wrong dimension parameter on the
command line.

picoproj: remove one of two consecutive blank lines

doc, contrib: add PicoPix 2330 to the list of supported devices

Thanks: Varlei Everton Menconi <varlei@dge.inpe.br>

am7xxx: add support for Philips/Sagemcom PicoPix 2330

Merge branch 'per-device-usb-config'

am7xxx: make the supported_device array const

am7xxx: improve setting USB configuration and interface_number

There are devices out there with different bConfigurationValue or
bInterfaceNumber, having the ability to specify these setting on
a per-device basis makes it easier to add support for them.

am7xxx: reference am7xxx_usb_device_descriptor in struct _am7xxx_device

This makes it possible to retrieve info from the am7xxx device
descriptor starting from an am7xxx_device.

am7xxx: fail if USB configuration or interface are not right

Check the return values of libusb_set_configuration() and
libusb_claim_interface(), this way it is easier to spot devices which
require a different setting.

am7xxx_mode_switch: release interface only if claimed

debian/changelog: mention the latest change about libav versioned deps

Git-Dch: ignore

debian/control: restrict the version also for the other libav libraries

doc: add a man page for am7xxx_mode_switch

debian/changelog: mention patch 002

Git-Dch: ignore

debian/patches: add 0002-Revert-am7xxx-play-switch-to-avcodec_encode_video2.patch

Make build succeed with the libavcodec version in Wheezy: for now we
depend on libavcodec-dev (<< 6:9) to be sure the old API is still there
and enabled.

debian/changelog: adjust entries and mark the release for unstable

Git-Dch: ignore

Add a NEWS file

debian: add a changelog entry for the new upstream release

Git-Dch: ignore

debian: add a package for debug symbols

debian/control: update the list of supported devices

debian/patches: add patch 0001 for a missing man page

Provide the missing man page for am7xxx_mode_switch as a patch until
this is applied upstream.

debian/libam7xxx0.1.symbols: add the am7xxx_set_zoom_mode symbol

Merge tag 'v0.1.3' into debian

Release version 0.1.3

Release version 0.1.3

am7xxx-play: switch to avcodec_encode_video2()

The code can now compile with -Werror=deprecated-declarations

am7xxx-play: remove an unreachable break

am7xxx-play: rename 'packet' to 'in_packet'

This is in preparation to use an 'out_packet' for the conversion to
avcodec_encode_video2().

am7xxx: fix coding style

Keep logical operators on the current line when continuing to another line.

contrib: add the am7xxx-play-window.sh script

Merge branch 'fix-devinfo-for-PicoPix'

am7xxx: make libam7xxx work with Philips/Sagemcom PPX projectors

Philips/Sagemcom PicoPix projectors require that the DEVINFO packet is
the first one to be sent to the device in order for it to successfully
return the correct device information.

Call am7xxx_get_device_info() from am7xxx_open_device() to fulfill the
requirement from above. The device info will be cached by
am7xxx_get_device_info() and further calls to the latter will return the
correct data too.

am7xxx: cache device info in am7xxx_get_device_info()

Request the actual device info from the device only when there is not
a cached version of it already.

This serves two purposes:

  1. Cover the case when some firmwares provide useful device info
     only under some conditions and spit out bogus data instead when
     these conditions are not met.

  2. Speed up am7xxx_get_device_info() as the USB communication is
     performed only once. AFAWN the device info does not change
     throughout the life of an am7xxx_device.

am7xxx: detect unexpected responses to AM7XXX_PACKET_TYPE_DEVINFO requests

Under some conditions, some devices like Philips/Sagemcom PPX projectors
return a strangely small packet (e.g. an AM7XXX_PACKET_TYPE_POWER with
header_data_len == 2) in reply to an AM7XXX_PACKET_TYPE_DEVINFO, hence
NOT providing useful device info.

Fix an error when compiling with both -O0 and -Wp,-D_FORTIFY_SOURCE=2

Fix the following error happening with the gcc shipped on some Ubuntu
versions:

In file included from /usr/include/stdio.h:27:0,
                 from /home/pedro/lib/libam7xxx/src/am7xxx.c:19:
/usr/include/features.h:330:4: error: #warning _FORTIFY_SOURCE
requires compiling with optimization (-O) [-Werror=cpp]
cc1: all warnings being treated as errors
make[2]: *** [src/CMakeFiles/am7xxx.dir/am7xxx.c.o] Error 1
make[1]: *** [src/CMakeFiles/am7xxx.dir/all] Error 2
make: *** [all] Error 2

Reported-by: Petr Certik <petr@certik.cz>

am7xxx: don't mention AM7XXX_DIRECTION_OUT in read_header()

The validity check in read_header() looks for:

h->direction == AM7XXX_DIRECTION_IN

When this is false (direction is _not_equal_ to AM7XXX_DIRECTION_IN) it
cannot be said for sure that the direction field value is going to
really be AM7XXX_DIRECTION_OUT, technically the device may have put
anything there.

So, just report what happens don't try to overguess.

am7xxx: add a note on the symmetry of read_header() and send_header()

In read_header we do:
  1. read data
  2. unserialize header
  3. validity check
  4. dump header

In send_header:
  1. dump header
  2. validity check (we skip it, actually)
  3. serialize header
  4. send data

am7xxx: assign device_list next to its first use

This makes the code a little more readable.

am7xxx: print text description of the 'direction' field

Print "IN" or "OUT" (or "UNKNOWN") when dumping the direction field of
the header, this makes it easier to follow the communication in the
debug output.

picoproj: clarify that when AM7XXX_ZOOM_TEST is set no image gets sent