Hi all, Maxim, are you saying you prefer a pull request? Last time I did one, you required the patch to be sent on the list, so I am not sure what to do next time.
Thx, Christophe. On 27 February 2017 at 10:15, Maxim Uvarov <maxim.uva...@linaro.org> wrote: > for big serries of patches >20 it's good to provide some git link to > download all of that. That Might be Linaro's people git or github. > > Maxim. > > On 25 February 2017 at 01:03, Mike Holmes <mike.hol...@linaro.org> wrote: > >> I did make a ticket for IT to look into this >> >> On 24 February 2017 at 16:55, Bill Fischofer <bill.fischo...@linaro.org> >> wrote: >> >> > It looks like Christophe sent the original series to Mike, Forrest, >> > Sachin, Yi, Me, and the odp mailing list. I suspect that's why Mike >> > and I (and presumably the others on the direct mail list) have a >> > complete copy of the series. A number of the parts didn't make it >> > either to patchworks or the ODP mailing list archives, so I suspect >> > some sort of IT glitch. >> > >> > On Fri, Feb 24, 2017 at 6:15 AM, Mike Holmes <mike.hol...@linaro.org> >> > wrote: >> > > Christophe will be back next week and we can check into where they >> went. >> > I >> > > will ping Philip if I can prove I got them on CC and the list lost >> them. >> > > >> > > On Feb 24, 2017 3:31 AM, "Josep Puigdemont" < >> josep.puigdem...@linaro.org >> > > >> > > wrote: >> > >> >> > >> On Thu, Feb 23, 2017 at 10:51:27AM -0600, Bill Fischofer wrote: >> > >> > I see all the patches in this series on the mailing list, but it >> > appears >> > >> > that the patchwork series[1] is missing parts 14 and 16. >> > >> >> > >> and part 20. >> > >> >> > >> I don't have parts 14, 16, and 20 on my inbox either, and it doesn't >> > look >> > >> like they ever reached the list, since they are missing from the >> > >> archives too. Part 16/21 in the archives is a reply from you, Bill, so >> > >> you definitely got the email (maybe you were cc'd?). >> > >> >> > >> > >> > >> > --- >> > >> > [1] http://patches.opendataplane.org/project/lng-odp/list/? >> series=65 >> > >> > >> > >> > On Thu, Feb 23, 2017 at 4:16 AM, Yi He <yi...@linaro.org> wrote: >> > >> > >> > >> > > In patchwork for this series I saw 14, 16, 21 are missing, do you >> > see >> > >> > > the >> > >> > > same problem? >> > >> > > >> > >> > > Best Regards, Yi >> > >> > > >> > >> > > On 23 February 2017 at 07:32, Bill Fischofer >> > >> > > <bill.fischo...@linaro.org> >> > >> > > wrote: >> > >> > > >> > >> > >> This series compiles and runs/tests fine using gcc and clang on >> > both >> > >> > >> 64 >> > >> > >> and 32 bit systems. Many comments, mostly cosmetic, however some >> > >> > >> missing >> > >> > >> field initializations are noted as well as some suggestions for >> > >> > >> handling >> > >> > >> type conversions. >> > >> > >> >> > >> > >> On Wed, Feb 22, 2017 at 6:55 AM, Christophe Milard < >> > >> > >> christophe.mil...@linaro.org> wrote: >> > >> > >> >> > >> > >>> This patch series implements the driver interface, i.e. >> > >> > >>> enumerator class, enumerator, devio and drivers registration and >> > >> > >>> probing. >> > >> > >>> This interface is depicted in: >> > >> > >>> https://docs.google.com/document/d/1eCKPJF6uSlOllXi_sKDvRwUD >> > >> > >>> 2BXm-ZzxZoKT0nVEsl4/edit >> > >> > >>> The associated tests are testing these mechanisms. Note that >> these >> > >> > >>> tests >> > >> > >>> are testing staticaly linked modules only (hence avoiding the >> > >> > >>> module/platform/test debate). Also note that these tests are >> > >> > >>> gathering >> > >> > >>> all the elements (enumerators, enumerator classes, devio, >> drivers) >> > >> > >>> making >> > >> > >>> up the driver interface so as their interactions can be checked. >> > >> > >>> Real elements (pci enumerators, drivers...) will likely be >> written >> > >> > >>> in a >> > >> > >>> much >> > >> > >>> more stand-alone way. >> > >> > >>> >> > >> > >>> Christophe Milard (21): >> > >> > >>> drv: making parameter strings dynamically computable >> > >> > >>> linux-gen: drv: enumerator_class registration >> > >> > >>> test: drv: enumerator_class registration tests >> > >> > >>> linux-gen: drv: enumerator registration >> > >> > >>> test: drv: enumerator registration tests >> > >> > >>> drv: driver: change drv unbind function name and pass correct >> > >> > >>> parameter >> > >> > >>> drv: driver: add callback function for device destruction >> > >> > >>> linux-gen: drv: device creation and deletion >> > >> > >>> drv: driver: adding device query function >> > >> > >>> linux-gen: drv: driver: adding device querry function >> > >> > >>> test: drv: device creation and destruction >> > >> > >>> drv: driver: adding a probe and remove callback for devio >> > >> > >>> linux-gen: drv: devio registration >> > >> > >>> test: drv: devio creation and destruction >> > >> > >>> drv: adding driver remove function >> > >> > >>> drv: complement parameters to the driver probe() function >> > >> > >>> linux-gen: driver registration and probing >> > >> > >>> test: drv: driver registration and probing >> > >> > >>> drv: driver: adding functions to attach driver's data to the >> > >> > >>> device >> > >> > >>> linux-gen: adding functions to attach driver's data to the >> > device >> > >> > >>> test: drv: test for setting and retrieving driver's data >> > >> > >>> >> > >> > >>> include/odp/drv/spec/driver.h | 132 ++- >> > >> > >>> platform/linux-generic/Makefile.am | 1 + >> > >> > >>> platform/linux-generic/_modules.c | 4 + >> > >> > >>> platform/linux-generic/drv_driver.c | 1037 >> > >> > >>> +++++++++++++++++++- >> > >> > >>> .../linux-generic/include/drv_driver_internal.h | 22 + >> > >> > >>> platform/linux-generic/include/odp_internal.h | 5 + >> > >> > >>> platform/linux-generic/odp_init.c | 21 +- >> > >> > >>> test/common_plat/m4/configure.m4 | 1 + >> > >> > >>> test/common_plat/validation/drv/Makefile.am | 1 + >> > >> > >>> .../validation/drv/drvdriver/.gitignore | 5 + >> > >> > >>> .../validation/drv/drvdriver/Makefile.am | 60 ++ >> > >> > >>> .../validation/drv/drvdriver/drvdriver_device.c | 218 ++++ >> > >> > >>> .../validation/drv/drvdriver/drvdriver_device.h | 24 + >> > >> > >>> .../drv/drvdriver/drvdriver_device_main.c | 12 + >> > >> > >>> .../validation/drv/drvdriver/drvdriver_devio.c | 209 ++++ >> > >> > >>> .../validation/drv/drvdriver/drvdriver_devio.h | 24 + >> > >> > >>> .../drv/drvdriver/drvdriver_devio_main.c | 12 + >> > >> > >>> .../validation/drv/drvdriver/drvdriver_driver.c | 518 >> > >> > >>> ++++++++++ >> > >> > >>> .../validation/drv/drvdriver/drvdriver_driver.h | 24 + >> > >> > >>> .../drv/drvdriver/drvdriver_driver_main.c | 12 + >> > >> > >>> .../validation/drv/drvdriver/drvdriver_enumr.c | 303 >> ++++++ >> > >> > >>> .../validation/drv/drvdriver/drvdriver_enumr.h | 24 + >> > >> > >>> .../drv/drvdriver/drvdriver_enumr_class.c | 174 ++++ >> > >> > >>> .../drv/drvdriver/drvdriver_enumr_class.h | 24 + >> > >> > >>> .../drv/drvdriver/drvdriver_enumr_class_main.c | 12 + >> > >> > >>> .../drv/drvdriver/drvdriver_enumr_main.c | 12 + >> > >> > >>> test/linux-generic/Makefile.am | 5 + >> > >> > >>> 27 files changed, 2861 insertions(+), 35 deletions(-) >> > >> > >>> create mode 100644 >> > >> > >>> platform/linux-generic/include/drv_driver_internal.h >> > >> > >>> create mode 100644 >> > >> > >>> test/common_plat/validation/drv/drvdriver/.gitignore >> > >> > >>> create mode 100644 test/common_plat/validation/dr >> > >> > >>> v/drvdriver/Makefile.am >> > >> > >>> create mode 100644 test/common_plat/validation/dr >> > >> > >>> v/drvdriver/drvdriver_device.c >> > >> > >>> create mode 100644 test/common_plat/validation/dr >> > >> > >>> v/drvdriver/drvdriver_device.h >> > >> > >>> create mode 100644 test/common_plat/validation/dr >> > >> > >>> v/drvdriver/drvdriver_device_main.c >> > >> > >>> create mode 100644 test/common_plat/validation/dr >> > >> > >>> v/drvdriver/drvdriver_devio.c >> > >> > >>> create mode 100644 test/common_plat/validation/dr >> > >> > >>> v/drvdriver/drvdriver_devio.h >> > >> > >>> create mode 100644 test/common_plat/validation/dr >> > >> > >>> v/drvdriver/drvdriver_devio_main.c >> > >> > >>> create mode 100644 test/common_plat/validation/dr >> > >> > >>> v/drvdriver/drvdriver_driver.c >> > >> > >>> create mode 100644 test/common_plat/validation/dr >> > >> > >>> v/drvdriver/drvdriver_driver.h >> > >> > >>> create mode 100644 test/common_plat/validation/dr >> > >> > >>> v/drvdriver/drvdriver_driver_main.c >> > >> > >>> create mode 100644 test/common_plat/validation/dr >> > >> > >>> v/drvdriver/drvdriver_enumr.c >> > >> > >>> create mode 100644 test/common_plat/validation/dr >> > >> > >>> v/drvdriver/drvdriver_enumr.h >> > >> > >>> create mode 100644 test/common_plat/validation/dr >> > >> > >>> v/drvdriver/drvdriver_enumr_class.c >> > >> > >>> create mode 100644 test/common_plat/validation/dr >> > >> > >>> v/drvdriver/drvdriver_enumr_class.h >> > >> > >>> create mode 100644 test/common_plat/validation/dr >> > >> > >>> v/drvdriver/drvdriver_enumr_class_main.c >> > >> > >>> create mode 100644 test/common_plat/validation/dr >> > >> > >>> v/drvdriver/drvdriver_enumr_main.c >> > >> > >>> >> > >> > >>> -- >> > >> > >>> 2.7.4 >> > >> > >>> >> > >> > >>> >> > >> > >> >> > >> > > >> > >> >> >> >> -- >> Mike Holmes >> Engineering Director, Core Development >> Linaro.org <http://www.linaro.org/> *│ *Open source software for ARM SoCs >> "Work should be fun and collaborative, the rest follows" >>
