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

doc: add a man page for am7xxx_mode_switch

Add a NEWS file

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

am7xxx: add support for Philips/SagemCom PicoPix PPX 2055

examples: support multiple devices

Expose a '-d <index>' option to set the device index, this way multiple
devices can be used.

Tested-by: Konstantin Lohmann <konstl@konstl.com>

doc: update Doxygen configuration

Use "doxygen -s -u doc/Doxyfile.in"; the update fixes some warnings
when building the documentation with Doxygen 1.8:

warning: Tag `SHOW_DIRECTORIES' at line 79 of file Doxyfile has become obsolete.
To avoid this warning please remove this line from your configuration file or upgrade it using "doxygen -u"
warning: Tag `HTML_ALIGN_MEMBERS' at line 149 of file Doxyfile has become obsolete.
To avoid this warning please remove this line from your configuration file or upgrade it using "doxygen -u"
warning: Tag `USE_INLINE_TREES' at line 176 of file Doxyfile has become obsolete.
To avoid this warning please remove this line from your configuration file or upgrade it using "doxygen -u"

Merge branch 'am7xxx_set_zoom_mode'

contrib: add a test image to show how zoom modes work

The image can be converted to a suitable JPEG with 'convert' from
imagemagick:

  $ convert libam7xxx_test_image_800x480.svg libam7xxx_test_image_800x480.jpg

am7xxx-play: make the help about power mode more consistent

Use the same terminology used in the Doxygen documentation, "power mode"
instead of "power level".

Print values using "%d" instead of "%x".

picoproj: make the help about power mode more consistent

Use the same terminology used in the Doxygen documentation, "power mode"
instead of "power level".

Print values using "%d" instead of "%x".

am7xxx: update signature and documentation of am7xxx_set_power_mode()

In the signature, rename the 'mode' parameter  to 'power' in order to
show better what it is about, after all in am7xxx_set_zoom_mode() the
correspondent parameter has been called zoom.

In the documentation tells about the new guesses about what the expected
behavior after AM7XXX_POWER_OFF is.

am7xxx-play: support setting the zoom mode

picoproj: support setting the zoom mode

am7xxx: implement support for the AM7XXX_PACKET_TYPE_ZOOM

The info has been guessed from the windows drivers, it may be imprecise
and incomplete.

Merge remote-tracking branch 'origin/rettichschnidi'

Add missing break.

Fix typo.

contrib: add an example of how to start displaying images automatically

Add an example of how an am7xxx-autodisplay functionality might be
implemented: some udev rules call a script which resizes the screen and
then call am7xxx-play.

contrib: add other supported devices to 55-am7xxx.rules

doc: update the list of supported devices

am7xxx: rename am7xxx_header.unknown0 to am7xxx_header.direction

This field should indicate the direction of the communication, according to
the USB dumps it is 0 for outgoing packets and 1 for incoming packets.

README.asciidoc: add Aiptek PocketCinema T25 to the AM7XXX devices list

am7xxx: add support for Aiptek PocketCinema T25

README.asciidoc: add info about running am7xxx-play.exe on Windows

HACKING.asciidoc: add info about compiling am7xxx-play for Windows

am7xxx-play: check if strtok_r is available

On Windows systems, where strtok_r is not available, strtok_s could be
used but this needs the C Run-Time library (msvcrt.dll or msvcr80.dll)
and we can't distribute it.

So for now, when strtok_r is not available, just print a message to warn
the user that the '-o' options is not available.

am7xxx-play: check if sigaction is available

If sigaction is not available just stub out the set_signal_handler()
function.

am7xxx-play: add a fallback definition for ENOTSUP

am7xxx-play: get the framerate from the video stream

When setting the output codec get the time_base (which indicates the
framerate) from the video stream, not from the input codec context, this
is the correct way to do it.

On some systems (Windows) the time_base from the input codec context is
set different from the one of the video stream, and timebase.num ends up
being 0 which results in the output codec initialization failing with
the message "framerate not set".

mingw_cross_toolchain.cmake: set the MINGW variable to True

Setting MINGW to True is important to let cmake set some variables such
as CMAKE_FIND_LIBRARY_PREFIXES and CMAKE_FIND_LIBRARY_SUFFIXES from
Platform/Windows-GNU.cmake

I wasted a lot of time on this with find_library() not finding libav
libraries compiled for MingW whith the ".dll.a" suffix.

HACKING.asciidoc: add info about getting and building libam7xxx

Some Ubuntu user asked for a step by step guide.

README.asciidoc: add Royaltek PJU-2100 to the AM7XXX based devices list

The drivers for this device look like the same "EZ Display" bundle as
some other models.

am7xxx: add support for Acer C112

README.asciidoc document how to get libam7xx running on MS Windows

Add a simple usb_mode_switch clone for am7xxx devices

Add am7xxx_mode_switch, this is will be used on systems where
usb_mode_switch is not easily available (e.g. MS Windows).

TODO: mention the plan about GStreamer

Merge branch 'mingw-port'

HACKING.asciidoc: add a section to explain Windows cross compilation

Add a CMAKE_TOOLCHAIN_FILE to compile with MinGW

This is inspired by this blog post by Sylvain Beucler:
http://blog.beuc.net/posts/Cross-compiling_with_CMake/

Don't set -pedantic-errors in CMAKE_C_FLAGS, it breaks check_symbol_exists()

Set -pedantic-errors in more specific variables like
CMAKE_C_FLAGS_DEBUG, CMAKE_C_FLAGS_RELEASE and
CMAKE_C_FLAGS_RELWITHDEBINFO.

See also http://public.kitware.com/Bug/view.php?id=13208

picoproj: replace mmap() with more portable file stream operations

This makes picoproj compile for Windows, for instance.

picoproj: fix a typo

We have no am7xxx_get_info(), maybe this is a leftover of some older
unreleased API version.

picoproj: issue a warning when passing "-f" more than once

am7xxx: don't look for the math library when compiling for Windows

am7xxx: MinGW does not have endian.h, provide fallbacks

Provide fallbacks only for the functions used in the code, and assume
Windows is always using little endian.

am7xxx: fix setting the USB configuration

libusb_set_configuration() expects the bConfigurationValue of the
configuration we wish to activate as the second parameter.

The am7xxx devices we know have bConfigurationValue=2 for the first and
only configuration available in the descriptor, use this value.

While the libusb linux back-end looks forgiving on this one, without this
change the WinUSB back-end will fail with an error like:

C:\Documents and Settings\Administrator\Desktop\win>picoproj.exe -f image.jpg -F 1 -l 5 -W 800 -H 480
JPEG format
scan_devices: am7xxx device found, index: 0, name: Acer C110
[timestamp] [threadID] facility level [function call] <message>
--------------------------------------------------------------------------------

[ 0.000000] [00000eb0] libusbx: warning [winusb_submit_control_transfer] cannot set configuration other than the default one
[ 0.031250] [00000eb0] libusbx: warning [winusb_submit_control_transfer] cannot set configuration other than the default one
[ 0.046875] [00000eb0] libusbx: error [winusb_submit_bulk_transfer] unable to match endpoint to an open interface - cancelling transfer
read_data[281]: ret: -5 transferred: 0 (expected 24)
am7xxx_get_info: No such file or directory

am7xxx: use hex notation for USB endpoints

This is for consistency with other uses.

picoproj: use MAP_PRIVATE in the mmap call

There is no need to allow sharing the mapped region with other
processes.

HACKING.asciidoc: fix cmake invocation examples

Put the path to the source dir last, like the cmake manual says:

  cmake [options] <path-to-source>

Also put a space between the -D option and the configuration setting it
modifies, this is more readable.

Merge remote-tracking branch 'origin/rettichschnidi' into mingw-port

picoproj: add a note about image dimensions and native resolution

Also print a warning when the user supplies a big image, picoproj does
not perform any rescale so the device may just display a wrong picture
or even hang.

Fix usage of FIND_PATH, allow $FFMPEG_DIR to be used.

Stop CMake if function avformat_open_input not available,
print an error message. Prevents compile errors later on.

Make example programs optional.

Release version 0.1.2

This fixes the handling of DESTDIR and should finally make packaging
easier.

doc: fix the "custom install targets" to handle DESTDIR

Stuff has to be installed/copied in an actual "install" target for Cmake
to handle DESTDIR sanely, a custom target is not enough.

So the story now is:

  - add a component-local install rule

  - make the "custom install target" invoke it

  - make sure the custom target has been executed at install time

Maybe all this is not even worth the effort anymore and the "custom
install targets" could go away altogether, their point was to have
_them_ invoke at install time end exploit the dependencies chain.

Release version 0.1.1

This release adds some improvements which should ease packaging, we also
ship a changelog (generated with "make changelog") because some
distributions like to ship some form of upstream history in their
packages.

doc: use ${DOC_OUTPUT_PATH} in the targets once we have it defined

examples: keep the -h option as the last one in am7xxx-play

This is the Unix tradition AFAICT. Also, options in the geopt() loop
usually follow the order in which they appear in the usage message.

examples: add power level setting to picoproj

TODO: remove the entry about signals and picoproj

picoproj is a minimal example and it is just meant to illustrate the API
and run and exit quickly, it is not worth adding signal handling in it,
that would just make the code more noisy.

cosmetics: remove some trailing spaces

am7xxx: control shared library symbols visibility

Export as public symbols only those really needed, without this change
some symbols from serialize.c (get_8, get_le32, put_8, put_le32) were
public too.

Use -fvisibility=hidden so we don't need to explicitly mark symbols as
local.

doc: make sure docs have been generated when installing

This is done in three steps:

 - first a "custom install target" is created for each custom target
   which builds documentation;

 - then dependencies between those custom install targets and the
   correspondent "build" targets are added;

  - finally some code is added to ensure those "custom install targets"
    are called when the builtin install target is executed.

Increase project number to 0.1.0

Version 0.1.0 is going to be the first libam7xxx stable release.

Merge branch 'unstable'

TODO: remove the entry about documenting the API with Doxygen

doc: add generation of man pages from asciidoc sources

doc: add a link to the public API on the main page

examples: rephrase picoproj example description

This is to avoid repeating 'show' two times in the same sentence.

examples: make picoproj usage look more like am7xxx-play one

doc: add examples to the Doxygen documentation

sm7xxx-play: add missing newline on some error messages

am7xxx: round scaled_height and scaled_width

Round when converting from float to int, this is more like what the user
expects about image dimensions.

cmake: make stricter compilation checks conditional

On some systems building with options like -Werror is just impractical.

Merge remote-tracking branch 'origin/rettichschnidi' into unstable

Conflicts:
examples/am7xxx-play.c

examples: set proper return codes in am7xxx-play

Set the return codes explicitly on the error path of functions not
returning an int error value.

examples: add a -l option to am7xxx-play

Allow setting the verbosity level of log messages.

Fix typo: dimesions -> dimensions

All modes above LOW need both connectors to be plugged in.

Fix typo: architechtures -> architectures