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

Remove douled semicolons

Check the user submitted value for the rescaling method.

Fix the wording of the help message for the rescaling method.

Update the help message to make it clear that the quality argument needs
a parameter.

Let the user set the power mode via the switch '-p <power mode>'

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

cmake: disable optimizations in debug builds

Link to all ffmpeg libraries, not just to avdevice.

am7xxx: fix a typo, s/it's/its/

Merge branch 'unstable' into rettichschnidi

Purely cosmetic change: add missing newline

Use commas to separate different pico projector names.

examples: add a am7xxx-play example program

am7xxx-play is a simple player based on ffmpeg/libav which displays the
output on a am7xxx device. It can be used to capture the desktop screen,
or video from a v4l2 device, or from any other device supported by
libavdevice including video files.

However it is to note that am7xxx-play displays the output as fast as it
can so it is not reliable when used to display video files directly.

am7xxx: cosmetics, remove some useless double spaces

am7xxx: add  am7xxx_calc_scaled_image_dimensions()

This function is useful to calculate the dimensions of an image which
will be sent with am7xxx_send_image() in a way that keeps the original
image aspect ratio.

contrib: add PicoPix 1020 USB IDs to 55-am7xxx.rules

picoproj: move it to an example/ directory

We plan on adding some more useful examples in the future, so it is
better to have them all in their separate directory in order to make the
project structure easier to understand.

cmake: split out the maintenance targets to a new cmake module

This makes the top level CMakeLists.txt easier to follow.

doc: add an install target

For now only Unix-style destinations are supported, we will abstract
them out when the need raises.

am7xxx, doc: add Doxygen documentation for the public API

am7xxx: rename the 'size' argument of am7xxx_send_image() to 'image_size'

cosmetics: remove some unneeded white spaces

CmakeLists.txt: add some hardening options to CMAKE_C_FLAGS

These options (-fstack-protector --param=ssp-buffer-size=4) should not
affect the current code but it is useful to have them here for
reference.

am7xxx: introduce a new am7xxx_device_info type

This is in order to abstract the properties of an am7xxx device, for now
only native_width and native_height are supported.

HACKING.asciidoc: mention how to use valgrind

picoproj: add an option to set the log level

am7xxx: use the logging infrastructure

Also make debug_dump_devinfo_header() and trace_dump_buffer() stubbed
out at compile time when DEBUG is not defined, just to save some cycles
in the data sending routines when in production.

am7xxx: add a simple logging infrastructure

am7xxx: track the context in am7xxx_device

This will be used to retrieve the context in functions operating on
devices, in order to use the log calls which are going to be added.

am7xxx: pass the context to add_new_device() and find_device()

This is preparation for when we will track the context in am7xxx_device
as well, and as a side effect those functions now look more consistent
with the other ones.

am7xxx: silent a warning enabled by 'sparse' about an uninitialized variable

src/am7xxx.c:423:6: warning: ‘ret’ may be used uninitialized in this function [-Wuninitialized]

If libusb_get_device_list() returned 0 we didn't have 'ret' properly
initialized.

TODO: remove the entry about multi-device support

This is done now, even if not tested with actual hardware...

am7xxx: add multi-device support

Allow operating on more than one am7xxx device at the same time.

TODO: handle signals in picoproj and do the proper cleanup

am7xxx: support other devices which talk the same protocol

Add support for the Philips/Sagemcom PicoPix 1020, which has different
USB IDs.

cmake: fix libusb search

Adjust cmake test for libusb library so that the compatible libusb for
freebsd can be found. Since freebsd's libusb header are located in
a different place from libusb-1.0, the includes in the source code must
be adjusted using #include <libusb.h>. Actually those new include
statements are those that libusb-1.0 really expects to be used (see
libusb-1.0.pc cflags to verify this).

Taken from libfreenect commit 8219750df3a13501626a7f7f54d24afb8e64ec3f
https://github.com/OpenKinect/libfreenect/commit/8219750df3a13501626a7f7f54d24afb8e64ec3f

am7xxx: change am7xx_device definition, better buffer handling

Add a new data type for am7xxx devices and make it hold a buffer, this
way allocating a new buffer for each communication can be avoided; also
move this definition to am7xxx.c to avoid exposing details about libusb
in am7xxx.h.

am7xxx: initialize the 'transferred' variable before USB transfers

This makes sure the subsequent error messages print a sane value when
libusb fails without touching the variable.

README.asciidoc: Acer C120 is not based on AM7XXX

Dhanraj Rajput has one of these and he figured out that the
communication here is based on the SCSI Pass-Trough protocol, not on
plain USB bulk transfers.

README.asciidoc: rephrase the part about USB IDs

Those devices does not always use the Actions Micro Vendor ID in display
mode, some Philips ones don't for instance, so tell that the IDs
mentioned are just examples.

Also mention that usb-modeswitch perform the mode change automatically
now.

README.asciidoc: fix a typo s/trasfers/transfers/

README.asciidoc: mention Thomas Baquet's project and fix some style

contrib: add some udev rules to let normal users access the devices

Make any user in the "plugdev" group be able to use an am7xxx device,
not just the root user like it is now.

README.asciidoc: highlight USB IDs by using an unformatted style

Don't use fixed size integer types in the public header

The rationale behind this is that a user of the library is interested in
the meaning of the data passed, not on its storage size.

Make struct am7xxx_header and related types private

There is not need for struct am7xxx_header, am7xxx_packet_type, and the
other headers structs to be in the public am7xxx.h file.

Implement am7xxx_get_device_info()

Add support for packet type 0x01, which can be used to query device
information like the native width and native height.

Introduce also read_data(), read_header(), unserialize_header()
which are needed by am7xxx_get_device_info()

Add info about transfer direction in debug messages, add newline

picoproj: make the -f option mandatory

Passing an image file name is now mandatory, before that the user could
be induced into thinking that a command like the following was sending
an image:

 ./picoproj -W 800 -H 480 -F 1 some_image.jpg

while the actually intended command line was:

  ./picoproj -W 800 -H 480 -F 1 -f some_image.jpg
                                ^^
Note the missing -f in the first command line.

Implement am7xxx_set_power_mode()

The AM7XXX_PACKET_TYPE_POWER expects the power mode to be specified in
the header_data section of the header like a value between 0 and 4, but
with one bit per field, each field being a little-endian 32 bit integer.

Indent header_data fields when dumping headers

Document in_80chars() and remove reference_image_header[]

Dump the data only in DEBUG builds

Also use -Werror only in DEBUG builds, otherwise normal builds fail with:

src/am7xxx.c:42:13: error: ‘dump_header’ defined but not used [-Werror=unused-function]
src/am7xxx.c:73:13: error: ‘dump_buffer’ defined but not used [-Werror=unused-function]

Add a HACKING.asciidoc file

Give some info about how to make a debug build and mention the coding
style used in the project, so new contributors know how to behave.

Serialize struct am7xxx_header properly before sending it on the wire

That is in order to:

  1. keep data in the structs always in the host native byte order, this
     is more natural and less error prone as we might forget to use
     htole32() when setting struct fields or le32toh() when accessing
     them;

  2. be more portable: this way the buffer sent to the wire is
     independent of struct alignments or paddings introduced by
     compilers or required by a particular architecture.