On Thu, Jan 02, 2014 at 05:13:30PM +0100, Krzysztof Opasiak wrote: > Dear Matt, > > Please excuse me my passivity after discussuon about libusbg some time ago. > I had to close some other issues before taking up this one. > > Recently I looked into code of libusbg, build it and found some errors > which are fixed in attached patches. Moreover I have done some clean up > and minor refactoring. > > I hope that you will accept enclosed patched and update the repository.
Hi Krzysztof, Thanks, these all look pretty good. A few things: 1) This v3 series didn't get marked as [PATCH v3 ...] so it was a bit confusing when I was applying to test. 2) In the future, please mention any dependencies in your cover letter. In this case, you have some dependencies on Stanislaw's series: http://www.spinics.net/lists/linux-usb/msg96859.html There's some open comments on that series atm that I need addressed. 3) The series will need to be rebased on the current libusbg. It was necessary to do a flag day api change early on to get rid of the libgadget-induced naming. It should be an easy rebase on top of the tip of libusbg, now at git://github.com/libusbg/libusbg.git Thanks, Matt > Changes since v2: > - replace // comments with /* */ > - add missing parenthesis in commit message > - fix minor spelling mistakes in commit messages > > Changes since v1: > - replace memcpy with direct structure assignment > - Change goto lables on more meaningful in example > - Remove additional check in gadget_read_string > > > Krzysztof Opasiak (12): > libusbg: Surround header with include guards. > libusbg: Add missing return statement in non-void functions. > libusbg: Fix gadget-acm-ecm example to cleanup at exit. > libusbg: Move directory creation before writing attributes. > libusbg: Fix memory leak when unable to create directory. > libusbg: Add error handling to gadget_read_string(). > libusbg: Add missing config attrs parsing while new config creation. > libusbg: Separate parsing gadget attributes and strings. > libusbg: Initialize gadget attributes and strings while gadget > creation. > libusbg: Move symlink creation after memory allocation. > libusbg: Replace memcpy with structure assignment. > libusbg: Replace directory names with defines. > > examples/gadget-acm-ecm.c | 20 ++++----- > include/gadget/gadget.h | 4 ++ > src/gadget.c | 110 > +++++++++++++++++++++++++++++---------------- > 3 files changed, 86 insertions(+), 48 deletions(-) > > -- > 1.7.9.5 > -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html