Commit by Antonio Ospite <ospite@studenti.unina.it> on master :: r247e268536 libam7xxx/README.asciidoc: (link)

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.

Commit by Antonio Ospite <ospite@studenti.unina.it> on master :: r5f35f6df51 libam7xxx/README.asciidoc: (link)

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.

Commit by Antonio Ospite <ospite@studenti.unina.it> on master :: r35ea676b59 libam7xxx/README.asciidoc: (link)

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

Commit by Antonio Ospite <ospite@studenti.unina.it> on master :: r1b664da3a5 libam7xxx/README.asciidoc: (link)

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

Commit by Antonio Ospite <ospite@studenti.unina.it> on master :: rad08509274 libam7xxx/contrib/55-am7xxx.rules: (link)

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.

Commit by Antonio Ospite <ospite@studenti.unina.it> on master :: rec30ccd48e libam7xxx/README.asciidoc: (link)

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

Commit by Antonio Ospite <ospite@studenti.unina.it> on master :: r9056bbf801 libam7xxx/src/ (am7xxx.h picoproj.c): (link)

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.

Commit by Antonio Ospite <ospite@studenti.unina.it> on master :: r120b63ecba libam7xxx/src/ (am7xxx.c am7xxx.h): (link)

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.

Commit by Antonio Ospite <ospite@studenti.unina.it> on master :: r48aab112af libam7xxx/src/ (am7xxx.c am7xxx.h picoproj.c): (link)

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()

Commit by Antonio Ospite <ospite@studenti.unina.it> on master :: rb9376b1fd2 libam7xxx/src/am7xxx.c: (link)

Add info about transfer direction in debug messages, add newline

Commit by Antonio Ospite <ospite@studenti.unina.it> on master :: r0a0245dfbf libam7xxx/src/picoproj.c: (link)

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.

Commit by Antonio Ospite <ospite@studenti.unina.it> on master :: rad32c9df36 libam7xxx/src/ (am7xxx.c am7xxx.h picoproj.c): (link)

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.

Commit by Antonio Ospite <ospite@studenti.unina.it> on master :: rb162f59656 libam7xxx/src/am7xxx.c: (link)

Indent header_data fields when dumping headers

Commit by Antonio Ospite <ospite@studenti.unina.it> on master :: rbe42db2992 libam7xxx/src/am7xxx.c: (link)

Document in_80chars() and remove reference_image_header[]

Commit by Antonio Ospite <ospite@studenti.unina.it> on master :: ra083bf6cc5 libam7xxx/ (CMakeLists.txt src/am7xxx.c): (link)

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]

Commit by Antonio Ospite <ospite@studenti.unina.it> on master :: r6d8691adaf libam7xxx/HACKING.asciidoc: (link)

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.

Commit by Antonio Ospite <ospite@studenti.unina.it> on master :: r5e7d217e7f libam7xxx/src/ (5 files): (link)

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.

Commit by Antonio Ospite <ospite@studenti.unina.it> on master :: rc4cda76c6f libam7xxx/src/picoproj.c: (link)

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

Commit by Antonio Ospite <ospite@studenti.unina.it> on master :: r2834def502 libam7xxx/README.asciidoc: (link)

Document usb_modeswitch command to change the device mode