On 02/27/17 12:54, Christophe Milard wrote: > 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.
I mean that you can include pull request link to cover letter. If somebody will miss some patch he can download it with that link. Maxim. > > 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" >>>
