debian/changelog: improve the description of one patch

Gbp-Dch: ignore

debian/changelog: release package version 0.1.6-2

Gbp-Dch: ignore

debian/libam7xxx0.1-doc.lintian-overrides: drop unused-overrides

debian/patches: add am7xxx-bump-_POSIX_C_SOURCE-to-200112L-for-lroundf.patch

This should fix this warning from the QA site:
  W implicit-declaration /??PKGBUILDDIR??/src/am7xxx.c:1216 ...

debian/patches: add a patch to make the build of Doxygen docs reproducible

The patch disables HTML_TIMESTAMPS as suggested here:
https://wiki.debian.org/ReproducibleBuilds/TimestampsInDocumentationGeneratedByDoxygen

debian/changelog: release package version 0.1.6-1

Gbp-Dch: ignore

debian/control: increase Standards-Version to 3.9.6

debian/patches: drop all patches, they have all been applied upstream

debian/changelog: prepare for the 0.1.6-1 release

Gbp-Dch: ignore

Merge tag 'v0.1.6' into debian

Release version 0.1.6

Release version 0.1.6

Refresh contrib/performance/0001-Instrument-code-with-fps-meter.patch

Strip trailing spaces here and there

am7xxx-play: fix warning about deprecated fields in the output codec

  .../examples/am7xxx-play.c:238:2:
  warning: ‘lmin’ is deprecated [-Wdeprecated-declarations]
    output_codec_ctx->mb_lmin    = output_codec_ctx->lmin = output_codec_ctx->qmin * FF_QP2LAMBDA;
    ^
  .../examples/am7xxx-play.c:239:2:
  warning: ‘lmax’ is deprecated [-Wdeprecated-declarations]
    output_codec_ctx->mb_lmax    = output_codec_ctx->lmax = output_codec_ctx->qmax * FF_QP2LAMBDA;
    ^

am7xxx-play: don't dump the last frame unconditionally

Stop dumping the last frame unconditionally in DEBUG mode, it's not the
expected behavior; instead add an option to explicitly enable dumping
the last frame but still leave this active only in DEBUG mode.

am7xxx-play: fix runtime warnings from ffmpeg

ffmpeg emits warnings at runtime about some AVFrame fields not being
set:

  [mjpeg @ 0x669040] AVFrame.format is not set
  [mjpeg @ 0x669040] AVFrame.width or height is not set

This happens in a loop so the message is repeated over and over and
becomes annoying, fix this by setting the missing values.

TODO: add some ideas for minor enhancements

am7xxx-play: fix an error message, we now use am7xxx_send_image_async()

Fix some format string warnings from clang

Building with clang gives some format string warnings as below, fix them
by using the correct format string when printing out the values.

  .../libam7xxx/src/am7xxx.c:804:38: warning:
  format specifies type 'unsigned char' but the argument has type 'int'
  [-Wformat]
                        (*dev)->desc->configuration, current_configuration);
                                                     ^~~~~~~~~~~~~~~~~~~~~
  .../libam7xxx/src/am7xxx.c:65:85: note:
  expanded from macro 'debug'
  #define debug(ctx, ...)   log_message(ctx, AM7XXX_LOG_DEBUG, __func__, 0, __VA_ARGS__)
                                                                            ^
  1 warning generated.

  [...]

  .../libam7xxx/examples/am7xxx-modeswitch.c:75:5:
  warning: format specifies type 'unsigned char' but the argument has type
  'int' [-Wformat]
                                  AM7XXX_STORAGE_CONFIGURATION);
                                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
  .../libam7xxx/examples/am7xxx-modeswitch.c:26:38:
  note: expanded from macro 'AM7XXX_STORAGE_CONFIGURATION'
  #define AM7XXX_STORAGE_CONFIGURATION 1
                                       ^
  .../libam7xxx/examples/am7xxx-modeswitch.c:87:4:
  warning: format specifies type 'unsigned char' but the argument has type
  'int' [-Wformat]
                          AM7XXX_STORAGE_INTERFACE);
                          ^~~~~~~~~~~~~~~~~~~~~~~~
  .../libam7xxx/examples/am7xxx-modeswitch.c:27:38:
  note: expanded from macro 'AM7XXX_STORAGE_INTERFACE'
  #define AM7XXX_STORAGE_INTERFACE     0
                                       ^
  .../libam7xxx/examples/am7xxx-modeswitch.c:104:4:
  warning: format specifies type 'unsigned char' but the argument has type
  'int' [-Wformat]
                          AM7XXX_STORAGE_CONFIGURATION, current_configuration);
                          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
  .../libam7xxx/examples/am7xxx-modeswitch.c:26:38:
  note: expanded from macro 'AM7XXX_STORAGE_CONFIGURATION'
  #define AM7XXX_STORAGE_CONFIGURATION 1
                                       ^
  .../libam7xxx/examples/am7xxx-modeswitch.c:104:34:
  warning: format specifies type 'unsigned char' but the argument has type
  'int' [-Wformat]
                          AM7XXX_STORAGE_CONFIGURATION, current_configuration);
                                                        ^~~~~~~~~~~~~~~~~~~~~
  4 warnings generated.

am7xxx-play: Replace deprecated FFmpeg API symbol PIX_FMT_NV12

PIX_FMT_NV12 is not going to be available in ffmpeg 2.9 so prepare for
that by using AV_PIX_FMT_NV12, which BTW has been available for quite
some time already.

am7xxx: put some spaces around operators

am7xxx-modeswitch: fix the Length field in the switch command

The switch_command is a USB storage message, the bulk_cb_wrap structure
from include/linux/usb/storage.h in the Linux kernel shows the fields
meanings:

struct bulk_cb_wrap {
__le32 Signature; /* contains 'USBC' */
__u32 Tag; /* unique per command id */
__le32 DataTransferLength; /* size of data */
__u8 Flags; /* direction in bit 0 */
__u8 Lun; /* LUN normally 0 */
__u8 Length; /* length of the CDB */
__u8 CDB[16]; /* max command */
};

In switch_command the data in CDB is 16 bytes long (0x10), so using 0x0c
was wrong, it worked on some devices but it did not on others.

Use the correct value.

Thanks-to: Balasubramanian S <sbala214@gmail.com>

am7xxx: remove a redundant debug message in open_device()

The calling function will show an error for the same condition.

am7xxx: use debug() instead of fatal() in add_new_device()

We are guaranteed that ctx is non-null by a previous check.

am7xxx: rename the "function" argument of log_message() to "function_name"

This express better the meaning of the variable.

am7xxx: split declaration and initialization of the "transferred" variable

This makes it clearer than we mean to initialize the variable to
0 before _each_ libusb_bulk_transfer() call.

am7xxx-modeswitch: add a blank line for readability

am7xxx-modeswitch: set USB configuration following libusb documentation

For the details see: http://libusb.sourceforge.net/api-1.0/caveats.html

am7xxx-modeswitch: reorder declarations of variables

am7xxx-modeswitch: remove unneeded assignment

am7xxx-modeswitch: improve messages relative to libusb failures

am7xxx-modeswitch: use LIBUSB_LOG_LEVEL_INFO instead of a magic number

am7xxx-modeswitch: use libusb_set_auto_detach_kernel_driver()

am7xxx: print an error message when libusb_init fails

am7xxx: improve the comment about setting a new configuration

Explain better why it is not needed to detach all kernel drivers in most
situations.

am7xxx: clarify a comment about copying data in send_data_async()

am7xxx: release the interface when needed in open_device()

Release the interface when bailing out of open_device() after the
interface has been claimed.

am7xxx: check the return value of libusb_get_configuration()

libusb_get_configuration() can still fail after the device has been
opened, so check its actual return value before trusting the returned
current_configuration.

am7xxx: fix the error path of the second configuration check

Return a negative error value when the configuration check done after
claiming the interface fails, and also fix the output messages.

am7xxx: add comment about detaching kernel drivers before setting configuration

am7xxx: add some spaces in a comment

am7xxx: fix the coding style used to represent pointers

am7xxx: use libusb_error_name to print the return value of libusb calls

This makes the debug output more useful.

am7xxx: detach kernel driver before claiming the interface

This is needed on some systems like Ubuntu 14.04, but apparently not on
Debian ones.

Thanks-to: Andrea Console <andreaconsole@gmail.com>

am7xxx: use a saner approach when setting the USB configuration

Follow the advices from
http://libusb.sourceforge.net/api-1.0/caveats.html

am7xxx: when listing possible return values use "or" in place of "and"

am7xxx: mention open_device() in the comment about scan_devices()

am7xxx: clean up the exit path in open_device()

am7xxx: comment on the loop termination in SCAN_OP_OPEN_DEVICE mode

am7xxx: factor out an open_device() function from scan_devices()

This reduces the indentation level and makes the code more readable, now
it will be easier to improve open_device().

am7xxx: make a comment clearer about SCAN_OP_OPEN_DEVICE

TODO: add another item about the style used in cmake files

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

examples: fix handling optional libraries in target_link_libraries()

am7xxx-play can compile just fine without XCB, but cmake was being
overly strict when XCB was missing.

am7xxx-play: improve a comment

am7xxx-play: fix a possible case of use before initialization

The got_packet variable is assigned inside an internal "if" of the while
loop, but it is also used in the checks at the end of the while loop,
and it is possible to get to this part before having ever assigned the
variable, so in order to behave correctly initialize the variable before
the loop starts.

debian/changelog: explain why asciidoc and xmlto were moved to Build-Depends

Git-Dch: ignore

debian/changelog: release package version 0.1.5-3

Git-Dch: ignore

debian/control: move asciidoc and xmlto to Build-Depends

This fixes the problem of the missing man pages in libam7xxx0.1-bin when
building only the binary packages.

Building the man pages depends on asciidoc, and the man pages need to be
built along with libam7xxx0.1-bin which is a binary package; however
dependencies in Build-Depends-Indep are not brought in when doing
a binary only build.

xmlto is needed as well because the a2x program from asciidoc needs it.

doc: update Doxyfile.in

In particular this fixes the following warnings from doxygen:

Warning: Tag `XML_SCHEMA' at line 235 of file `Doxyfile.in' has become obsolete.
         This tag has been removed.
Warning: Tag `XML_DTD' at line 236 of file `Doxyfile.in' has become obsolete.
         This tag has been removed.

debian/changelog: release package version 0.1.5-2

Git-Dch: ignore

debian/libam7xxx0.1-doc.examples: move the C example programs to libam7xxx-dev

debian/libam7xxx0.1-bin.install: distribute some files from contrib/

debian/copyright: document the license of some scripts from contrib/

debian/patches: add a patch to fix the license of the example programs

examples: the examples were always meant to be all under GPL-3+

debian/libam7xxx0.1-doc.lintian-overrides: update one override

debian/gbp.conf: fix typo s/prinstine-tar/pristine-tar/g

debian/changelog: release package version 0.1.5-1

Git-Dch: ignore

debian/control: fix cme warnings "Synopsis is too long."

debian/control: use a single dash in short descriptions

Some packages use two dashes to separate the generic description from
the package specific one, but the majority of packages use only one.

debian/control: fix installation over old versions

debian/watch: detect new versions from gitweb

debian/patches: drop all patches

Some patches were fixes from upstream which are now in the orig
codebase.

Some other patches were reverts of upstream commits needed to build with
older libav and libusb versions shipped in Debian distributions, drop
these as well as they are not needed anymore to build the package in
Unstable; possibly they can come back in a backport branch if needed.

debian/control: rename the libam7xxx-bin package to libam7xxx0.1-bin

This fixes a lintian error:

  E: libam7xxx source: version-substvar-for-external-package libam7xxx0.1 -> libam7xxx0.1-bin

debian/libam7xxx0.1-doc.lintian-overrides: update overrides

debian/control: rename the libam7xxx-doc package to libam7xxx0.1-doc

This fixes a lintian error:

  E: libam7xxx source: version-substvar-for-external-package libam7xxx-dev -> libam7xxx0.1-doc

debian/rules: install the NEWS file as the upstream changelog

debian/copyright: update contact info and copyright years

debian/control: update Maintainer email address

debian/changelog: prepare for package version 0.1.5-1

Git-Dch: ignore

Merge tag 'v0.1.5' into debian

Release version 0.1.5

Release version 0.1.5

HACKING.asciidoc: update Windows cross-build section to use libav10

am7xxx-play: port to libav10

Update email address and copyright years

contrib/performance/README: fix some typos

cosmetics: remove some trailing spaces

README.asciidoc: update instructions to test libam7xxx on Windows

HACKING.asciidoc: update the build instructions for Windows

am7xxx: don't use partial designated initializers

Compilers may warn about missing fields in initializers when more checks
are enabled with STRICT_COMPILATION_CHECKS=ON:

  .../src/am7xxx.c:1131:31: error: missing field 'direction' initializer [-Werror,-Wmissing-field-initializers]
          struct am7xxx_header h = { 0 };
                                       ^
  1 error generated.

am7xxx: add a portable_endian.h

Add a portable_endian.h to define platform specific implementations for
the functions found on most unix-like systems in endian.h.

Merge branch 'zoom-tele-picopix'

am7xxx-play: allow setting the AM7XXX_ZOOM_TELE zoom mode

picoproj: allow setting the AM7XXX_ZOOM_TELE zoom mode

am7xxx: implement the set_zoom_mode() operation for the PicoPix 2055

Apparently AM7XXX_ZOOM_ORIGINAL and AM7XXX_ZOOM_TELE are the only
supported zoom modes on the PicoPix 2055.

am7xxx: add support for a new AM7XXX_ZOOM_TELE zoom mode

Some PicoPix projectors support a different zoom mode than the Acer
C110, in this mode the image uses a different lens distortion but the
frame aspect ratio does not change.

am7xxx: add a msleep() implementation in tools.[ch]

debian/changelog: prepare for 0.1.4-3

Git-Dch: ignore

debian/patches: add 0004-Revert-am7xxx-use-the-symbolic-constant-for-libusb-l.patch

Some old libusb version without the LIBUSB_LOG_LEVEL_* defines is still
around in Debian distros (like Raspbian). So add a patch to be able to
build packages for them.