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.

picoproj: exit with success when the -h option is used

Document usb_modeswitch command to change the device mode

Renamed AM7XXX_IMAGE_FORMAT_YUV_NV12 to AM7XXX_IMAGE_FORMAT_NV12.

Added support for imageformat YUV - NV12

Stop build process if a warning shows up

corrected some spelling mistakes

Relicense under GPLv2+

This could make sharing code with a linux kernel driver easier.

Add a TODO file

Add a README.asciidoc

Fix Copyright year

Add GPL-3 license text

Rename the header_len field to header_data_len

This matches better match its meaning, it is not the length of the whole
header, but only of the header_data section.

Make including am7xxx.h in C++ code safe

Use Cmake and make libam7xxx a shared library

Also move the source code to a src/ subdirectory in order to avoid
cluttering the project root dir.

Split am7xxx functions and definitions

This is the base for the libam7xxx shared library.

Add backup and changelog rules to Makefile

Add initial support for USB output communication

Add support for sending an actual JPEG image

Image size is always unsigned

Wrap lines when dumping big buffers

Add some command line options

This will make it easier to experiment with different image size and
format.

Minor Makefile cleanup

Rewrite to support different packet types

Fix a typo

Initial import