Message ID | 1490194110-40168-1-git-send-email-christophe.milard@linaro.org |
---|---|
Headers | show |
Series | driver items registration and probing | expand |
Hi, team Today in odp cloud meeting we talked about the DDF status, before the new LNG colleague joining to take over the DDF works, I'll take the ownership of the DDF and continue to move this patch series forward and related bugs, comments. This Thursday François would like to have a call (1:30 hours) to introduce knowledge, background and discussions he have had with Christophe around DDF, all are welcome if you feel interested in DDF topics and please reply before Wednesday earlier afternoon and I'll send meeting invitation to you. thanks and best regards, Yi On 22 March 2017 at 22:48, Christophe Milard <christophe.milard@linaro.org> wrote: > This patch series can be pulled from: > https://git.linaro.org/people/christophe.milard/odp.git/log/ > ?h=drv_framework_v2 > > Since V1: Fixes following Bill's comments. > > Note: I am not really sure this is still in phase with what was discussed > at connect, since I couldn't attend. But, at least I did the changes > following > the comments I recieved. Hope that still makes sence. > Also, I am aware that patch 1 generates a warning: I copied this part from > the > north API so I assume this is an agreed decision. > > 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_ > sKDvRwUD2BXm-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 (23): > drv: adding compiler hints in the driver interface > linux-gen: adding compiler hints in the driver interface > 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 ++- > include/odp/drv/spec/hints.h | 119 +++ > include/odp_drv.h | 1 + > platform/Makefile.inc | 1 + > platform/linux-generic/Makefile.am | 1 + > platform/linux-generic/_modules.c | 4 + > platform/linux-generic/drv_driver.c | 1051 > +++++++++++++++++++- > .../linux-generic/include/drv_driver_internal.h | 22 + > platform/linux-generic/include/odp/drv/hints.h | 34 + > 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 + > 31 files changed, 3030 insertions(+), 35 deletions(-) > create mode 100644 include/odp/drv/spec/hints.h > create mode 100644 platform/linux-generic/include/drv_driver_internal.h > create mode 100644 platform/linux-generic/include/odp/drv/hints.h > create mode 100644 test/common_plat/validation/drv/drvdriver/.gitignore > create mode 100644 test/common_plat/validation/drv/drvdriver/Makefile.am > create mode 100644 test/common_plat/validation/drv/drvdriver/drvdriver_ > device.c > create mode 100644 test/common_plat/validation/drv/drvdriver/drvdriver_ > device.h > create mode 100644 test/common_plat/validation/drv/drvdriver/drvdriver_ > device_main.c > create mode 100644 test/common_plat/validation/ > drv/drvdriver/drvdriver_devio.c > create mode 100644 test/common_plat/validation/ > drv/drvdriver/drvdriver_devio.h > create mode 100644 test/common_plat/validation/ > drv/drvdriver/drvdriver_devio_main.c > create mode 100644 test/common_plat/validation/drv/drvdriver/drvdriver_ > driver.c > create mode 100644 test/common_plat/validation/drv/drvdriver/drvdriver_ > driver.h > create mode 100644 test/common_plat/validation/drv/drvdriver/drvdriver_ > driver_main.c > create mode 100644 test/common_plat/validation/ > drv/drvdriver/drvdriver_enumr.c > create mode 100644 test/common_plat/validation/ > drv/drvdriver/drvdriver_enumr.h > create mode 100644 test/common_plat/validation/ > drv/drvdriver/drvdriver_enumr_class.c > create mode 100644 test/common_plat/validation/ > drv/drvdriver/drvdriver_enumr_class.h > create mode 100644 test/common_plat/validation/ > drv/drvdriver/drvdriver_enumr_class_main.c > create mode 100644 test/common_plat/validation/ > drv/drvdriver/drvdriver_enumr_main.c > > -- > 2.7.4 > >
On 04/11/17 17:06, Yi He wrote: > Hi, team > > Today in odp cloud meeting we talked about the DDF status, before the new > LNG colleague joining to take over the DDF works, I'll take the ownership > of the DDF and continue to move this patch series forward and related bugs, > comments. > > This Thursday François would like to have a call (1:30 hours) to introduce > knowledge, background and discussions he have had with Christophe around > DDF, all are welcome if you feel interested in DDF topics and please reply > before Wednesday earlier afternoon and I'll send meeting invitation to you. > > thanks and best regards, Yi > this can go as pull request to github. In that case we will not loose that patches and base for which these patches were done. Maxim. > On 22 March 2017 at 22:48, Christophe Milard <christophe.milard@linaro.org> > wrote: > >> This patch series can be pulled from: >> https://git.linaro.org/people/christophe.milard/odp.git/log/ >> ?h=drv_framework_v2 >> >> Since V1: Fixes following Bill's comments. >> >> Note: I am not really sure this is still in phase with what was discussed >> at connect, since I couldn't attend. But, at least I did the changes >> following >> the comments I recieved. Hope that still makes sence. >> Also, I am aware that patch 1 generates a warning: I copied this part from >> the >> north API so I assume this is an agreed decision. >> >> 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_ >> sKDvRwUD2BXm-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 (23): >> drv: adding compiler hints in the driver interface >> linux-gen: adding compiler hints in the driver interface >> 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 ++- >> include/odp/drv/spec/hints.h | 119 +++ >> include/odp_drv.h | 1 + >> platform/Makefile.inc | 1 + >> platform/linux-generic/Makefile.am | 1 + >> platform/linux-generic/_modules.c | 4 + >> platform/linux-generic/drv_driver.c | 1051 >> +++++++++++++++++++- >> .../linux-generic/include/drv_driver_internal.h | 22 + >> platform/linux-generic/include/odp/drv/hints.h | 34 + >> 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 + >> 31 files changed, 3030 insertions(+), 35 deletions(-) >> create mode 100644 include/odp/drv/spec/hints.h >> create mode 100644 platform/linux-generic/include/drv_driver_internal.h >> create mode 100644 platform/linux-generic/include/odp/drv/hints.h >> create mode 100644 test/common_plat/validation/drv/drvdriver/.gitignore >> create mode 100644 test/common_plat/validation/drv/drvdriver/Makefile.am >> create mode 100644 test/common_plat/validation/drv/drvdriver/drvdriver_ >> device.c >> create mode 100644 test/common_plat/validation/drv/drvdriver/drvdriver_ >> device.h >> create mode 100644 test/common_plat/validation/drv/drvdriver/drvdriver_ >> device_main.c >> create mode 100644 test/common_plat/validation/ >> drv/drvdriver/drvdriver_devio.c >> create mode 100644 test/common_plat/validation/ >> drv/drvdriver/drvdriver_devio.h >> create mode 100644 test/common_plat/validation/ >> drv/drvdriver/drvdriver_devio_main.c >> create mode 100644 test/common_plat/validation/drv/drvdriver/drvdriver_ >> driver.c >> create mode 100644 test/common_plat/validation/drv/drvdriver/drvdriver_ >> driver.h >> create mode 100644 test/common_plat/validation/drv/drvdriver/drvdriver_ >> driver_main.c >> create mode 100644 test/common_plat/validation/ >> drv/drvdriver/drvdriver_enumr.c >> create mode 100644 test/common_plat/validation/ >> drv/drvdriver/drvdriver_enumr.h >> create mode 100644 test/common_plat/validation/ >> drv/drvdriver/drvdriver_enumr_class.c >> create mode 100644 test/common_plat/validation/ >> drv/drvdriver/drvdriver_enumr_class.h >> create mode 100644 test/common_plat/validation/ >> drv/drvdriver/drvdriver_enumr_class_main.c >> create mode 100644 test/common_plat/validation/ >> drv/drvdriver/drvdriver_enumr_main.c >> >> -- >> 2.7.4 >> >>
Yes, Maxim I'll start by going through this patch series and convert it into a github PR. Best Regards, Yi On 12 April 2017 at 03:34, Maxim Uvarov <maxim.uvarov@linaro.org> wrote: > On 04/11/17 17:06, Yi He wrote: > > Hi, team > > > > Today in odp cloud meeting we talked about the DDF status, before the new > > LNG colleague joining to take over the DDF works, I'll take the ownership > > of the DDF and continue to move this patch series forward and related > bugs, > > comments. > > > > This Thursday François would like to have a call (1:30 hours) to > introduce > > knowledge, background and discussions he have had with Christophe around > > DDF, all are welcome if you feel interested in DDF topics and please > reply > > before Wednesday earlier afternoon and I'll send meeting invitation to > you. > > > > thanks and best regards, Yi > > > > this can go as pull request to github. In that case we will not loose > that patches and base for which these patches were done. > > Maxim. > > > On 22 March 2017 at 22:48, Christophe Milard < > christophe.milard@linaro.org> > > wrote: > > > >> This patch series can be pulled from: > >> https://git.linaro.org/people/christophe.milard/odp.git/log/ > >> ?h=drv_framework_v2 > >> > >> Since V1: Fixes following Bill's comments. > >> > >> Note: I am not really sure this is still in phase with what was > discussed > >> at connect, since I couldn't attend. But, at least I did the changes > >> following > >> the comments I recieved. Hope that still makes sence. > >> Also, I am aware that patch 1 generates a warning: I copied this part > from > >> the > >> north API so I assume this is an agreed decision. > >> > >> 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_ > >> sKDvRwUD2BXm-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 (23): > >> drv: adding compiler hints in the driver interface > >> linux-gen: adding compiler hints in the driver interface > >> 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 ++- > >> include/odp/drv/spec/hints.h | 119 +++ > >> include/odp_drv.h | 1 + > >> platform/Makefile.inc | 1 + > >> platform/linux-generic/Makefile.am | 1 + > >> platform/linux-generic/_modules.c | 4 + > >> platform/linux-generic/drv_driver.c | 1051 > >> +++++++++++++++++++- > >> .../linux-generic/include/drv_driver_internal.h | 22 + > >> platform/linux-generic/include/odp/drv/hints.h | 34 + > >> 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 + > >> 31 files changed, 3030 insertions(+), 35 deletions(-) > >> create mode 100644 include/odp/drv/spec/hints.h > >> create mode 100644 platform/linux-generic/ > include/drv_driver_internal.h > >> create mode 100644 platform/linux-generic/include/odp/drv/hints.h > >> create mode 100644 test/common_plat/validation/ > drv/drvdriver/.gitignore > >> create mode 100644 test/common_plat/validation/ > drv/drvdriver/Makefile.am > >> create mode 100644 test/common_plat/validation/ > drv/drvdriver/drvdriver_ > >> device.c > >> create mode 100644 test/common_plat/validation/ > drv/drvdriver/drvdriver_ > >> device.h > >> create mode 100644 test/common_plat/validation/ > drv/drvdriver/drvdriver_ > >> device_main.c > >> create mode 100644 test/common_plat/validation/ > >> drv/drvdriver/drvdriver_devio.c > >> create mode 100644 test/common_plat/validation/ > >> drv/drvdriver/drvdriver_devio.h > >> create mode 100644 test/common_plat/validation/ > >> drv/drvdriver/drvdriver_devio_main.c > >> create mode 100644 test/common_plat/validation/ > drv/drvdriver/drvdriver_ > >> driver.c > >> create mode 100644 test/common_plat/validation/ > drv/drvdriver/drvdriver_ > >> driver.h > >> create mode 100644 test/common_plat/validation/ > drv/drvdriver/drvdriver_ > >> driver_main.c > >> create mode 100644 test/common_plat/validation/ > >> drv/drvdriver/drvdriver_enumr.c > >> create mode 100644 test/common_plat/validation/ > >> drv/drvdriver/drvdriver_enumr.h > >> create mode 100644 test/common_plat/validation/ > >> drv/drvdriver/drvdriver_enumr_class.c > >> create mode 100644 test/common_plat/validation/ > >> drv/drvdriver/drvdriver_enumr_class.h > >> create mode 100644 test/common_plat/validation/ > >> drv/drvdriver/drvdriver_enumr_class_main.c > >> create mode 100644 test/common_plat/validation/ > >> drv/drvdriver/drvdriver_enumr_main.c > >> > >> -- > >> 2.7.4 > >> > >> > >
-----Original Message----- From: lng-odp [mailto:lng-odp-bounces@lists.linaro.org] On Behalf Of Yi He Sent: 11 April 2017 19:36 To: lng-odp <lng-odp@lists.linaro.org> Subject: Re: [lng-odp] [API-NEXT PATCHv2 00/23] driver items registration and probing Hi, team Today in odp cloud meeting we talked about the DDF status, before the new LNG colleague joining to take over the DDF works, I'll take the ownership of the DDF and continue to move this patch series forward and related bugs, comments. This Thursday François would like to have a call (1:30 hours) to introduce knowledge, background and discussions he have had with Christophe around DDF, all are welcome if you feel interested in DDF topics and please reply before Wednesday earlier afternoon and I'll send meeting invitation to you. I would be interested to join same. However I am based out of India. So please see if it would be feasible to schedule meeting to accommodate India time zone. Thanks Shally thanks and best regards, Yi On 22 March 2017 at 22:48, Christophe Milard <christophe.milard@linaro.org> wrote: > This patch series can be pulled from: > https://git.linaro.org/people/christophe.milard/odp.git/log/ > ?h=drv_framework_v2 > > Since V1: Fixes following Bill's comments. > > Note: I am not really sure this is still in phase with what was > discussed at connect, since I couldn't attend. But, at least I did the > changes following the comments I recieved. Hope that still makes > sence. > Also, I am aware that patch 1 generates a warning: I copied this part > from the north API so I assume this is an agreed decision. > > 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_ > sKDvRwUD2BXm-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 (23): > drv: adding compiler hints in the driver interface > linux-gen: adding compiler hints in the driver interface > 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 ++- > include/odp/drv/spec/hints.h | 119 +++ > include/odp_drv.h | 1 + > platform/Makefile.inc | 1 + > platform/linux-generic/Makefile.am | 1 + > platform/linux-generic/_modules.c | 4 + > platform/linux-generic/drv_driver.c | 1051 > +++++++++++++++++++- > .../linux-generic/include/drv_driver_internal.h | 22 + > platform/linux-generic/include/odp/drv/hints.h | 34 + > 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 + > 31 files changed, 3030 insertions(+), 35 deletions(-) create mode > 100644 include/odp/drv/spec/hints.h create mode 100644 > platform/linux-generic/include/drv_driver_internal.h > create mode 100644 platform/linux-generic/include/odp/drv/hints.h > create mode 100644 > test/common_plat/validation/drv/drvdriver/.gitignore > create mode 100644 > test/common_plat/validation/drv/drvdriver/Makefile.am > create mode 100644 > test/common_plat/validation/drv/drvdriver/drvdriver_ > device.c > create mode 100644 > test/common_plat/validation/drv/drvdriver/drvdriver_ > device.h > create mode 100644 > test/common_plat/validation/drv/drvdriver/drvdriver_ > device_main.c > create mode 100644 test/common_plat/validation/ > drv/drvdriver/drvdriver_devio.c create mode 100644 > test/common_plat/validation/ drv/drvdriver/drvdriver_devio.h create > mode 100644 test/common_plat/validation/ > drv/drvdriver/drvdriver_devio_main.c > create mode 100644 > test/common_plat/validation/drv/drvdriver/drvdriver_ > driver.c > create mode 100644 > test/common_plat/validation/drv/drvdriver/drvdriver_ > driver.h > create mode 100644 > test/common_plat/validation/drv/drvdriver/drvdriver_ > driver_main.c > create mode 100644 test/common_plat/validation/ > drv/drvdriver/drvdriver_enumr.c create mode 100644 > test/common_plat/validation/ drv/drvdriver/drvdriver_enumr.h create > mode 100644 test/common_plat/validation/ > drv/drvdriver/drvdriver_enumr_class.c > create mode 100644 test/common_plat/validation/ > drv/drvdriver/drvdriver_enumr_class.h > create mode 100644 test/common_plat/validation/ > drv/drvdriver/drvdriver_enumr_class_main.c > create mode 100644 test/common_plat/validation/ > drv/drvdriver/drvdriver_enumr_main.c > > -- > 2.7.4 > >
Hi Yi, >This Thursday François would like to have a call (1:30 hours) to introduce knowledge, background and discussions he have had with >Christophe around DDF, all are welcome if you feel interested in DDF topics and please reply before Wednesday earlier afternoon and I'll >send meeting invitation to you. I would like to participate in this meeting and am from India. Please see if it is feasible to schedule this meeting to accommodate us. Regards, Mahipal ________________________________ From: Verma, Shally Sent: Wednesday, April 12, 2017 10:38 AM To: Yi He; lng-odp Cc: Narayana, Prasad Athreya; Challa, Mahipal Subject: RE: [lng-odp] [API-NEXT PATCHv2 00/23] driver items registration and probing -----Original Message----- From: lng-odp [mailto:lng-odp-bounces@lists.linaro.org] On Behalf Of Yi He Sent: 11 April 2017 19:36 To: lng-odp <lng-odp@lists.linaro.org> Subject: Re: [lng-odp] [API-NEXT PATCHv2 00/23] driver items registration and probing Hi, team Today in odp cloud meeting we talked about the DDF status, before the new LNG colleague joining to take over the DDF works, I'll take the ownership of the DDF and continue to move this patch series forward and related bugs, comments. This Thursday François would like to have a call (1:30 hours) to introduce knowledge, background and discussions he have had with Christophe around DDF, all are welcome if you feel interested in DDF topics and please reply before Wednesday earlier afternoon and I'll send meeting invitation to you. I would be interested to join same. However I am based out of India. So please see if it would be feasible to schedule meeting to accommodate India time zone. Thanks Shally thanks and best regards, Yi On 22 March 2017 at 22:48, Christophe Milard <christophe.milard@linaro.org> wrote: > This patch series can be pulled from: > https://git.linaro.org/people/christophe.milard/odp.git/log/ > ?h=drv_framework_v2 > > Since V1: Fixes following Bill's comments. > > Note: I am not really sure this is still in phase with what was > discussed at connect, since I couldn't attend. But, at least I did the > changes following the comments I recieved. Hope that still makes > sence. > Also, I am aware that patch 1 generates a warning: I copied this part > from the north API so I assume this is an agreed decision. > > 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_ > sKDvRwUD2BXm-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 (23): > drv: adding compiler hints in the driver interface > linux-gen: adding compiler hints in the driver interface > 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 ++- > include/odp/drv/spec/hints.h | 119 +++ > include/odp_drv.h | 1 + > platform/Makefile.inc | 1 + > platform/linux-generic/Makefile.am | 1 + > platform/linux-generic/_modules.c | 4 + > platform/linux-generic/drv_driver.c | 1051 > +++++++++++++++++++- > .../linux-generic/include/drv_driver_internal.h | 22 + > platform/linux-generic/include/odp/drv/hints.h | 34 + > 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 + > 31 files changed, 3030 insertions(+), 35 deletions(-) create mode > 100644 include/odp/drv/spec/hints.h create mode 100644 > platform/linux-generic/include/drv_driver_internal.h > create mode 100644 platform/linux-generic/include/odp/drv/hints.h > create mode 100644 > test/common_plat/validation/drv/drvdriver/.gitignore > create mode 100644 > test/common_plat/validation/drv/drvdriver/Makefile.am > create mode 100644 > test/common_plat/validation/drv/drvdriver/drvdriver_ > device.c > create mode 100644 > test/common_plat/validation/drv/drvdriver/drvdriver_ > device.h > create mode 100644 > test/common_plat/validation/drv/drvdriver/drvdriver_ > device_main.c > create mode 100644 test/common_plat/validation/ > drv/drvdriver/drvdriver_devio.c create mode 100644 > test/common_plat/validation/ drv/drvdriver/drvdriver_devio.h create > mode 100644 test/common_plat/validation/ > drv/drvdriver/drvdriver_devio_main.c > create mode 100644 > test/common_plat/validation/drv/drvdriver/drvdriver_ > driver.c > create mode 100644 > test/common_plat/validation/drv/drvdriver/drvdriver_ > driver.h > create mode 100644 > test/common_plat/validation/drv/drvdriver/drvdriver_ > driver_main.c > create mode 100644 test/common_plat/validation/ > drv/drvdriver/drvdriver_enumr.c create mode 100644 > test/common_plat/validation/ drv/drvdriver/drvdriver_enumr.h create > mode 100644 test/common_plat/validation/ > drv/drvdriver/drvdriver_enumr_class.c > create mode 100644 test/common_plat/validation/ > drv/drvdriver/drvdriver_enumr_class.h > create mode 100644 test/common_plat/validation/ > drv/drvdriver/drvdriver_enumr_class_main.c > create mode 100644 test/common_plat/validation/ > drv/drvdriver/drvdriver_enumr_main.c > > -- > 2.7.4 > >
Hi, Bala, Shally and Mahipal yes, I'll send the invitation booking François' 10:00AM, India/shanghai afternoon time, can update the invitation if you have any suggestion. Best Regards, Yi On 12 April 2017 at 13:38, Bala Manoharan <bala.manoharan@linaro.org> wrote: > Hi Yi, > > I am also interested in the meeting. Do send me an invite also. > > Regards, > Bala > > > On 12 April 2017 at 10:46, Challa, Mahipal <Mahipal.Challa@cavium.com> > wrote: > > Hi Yi, > > > >>This Thursday François would like to have a call (1:30 hours) to > introduce knowledge, background and discussions he have had with > >Christophe around DDF, all are welcome if you feel interested in DDF > topics and please reply before Wednesday earlier afternoon and I'll >send > meeting invitation to you. > > > > > > I would like to participate in this meeting and am from India. Please > see if it is feasible to schedule this meeting to accommodate us. > > > > > > Regards, > > > > Mahipal > > > > > > ________________________________ > > From: Verma, Shally > > Sent: Wednesday, April 12, 2017 10:38 AM > > To: Yi He; lng-odp > > Cc: Narayana, Prasad Athreya; Challa, Mahipal > > Subject: RE: [lng-odp] [API-NEXT PATCHv2 00/23] driver items > registration and probing > > > > > > > > -----Original Message----- > > From: lng-odp [mailto:lng-odp-bounces@lists.linaro.org] On Behalf Of Yi > He > > Sent: 11 April 2017 19:36 > > To: lng-odp <lng-odp@lists.linaro.org> > > Subject: Re: [lng-odp] [API-NEXT PATCHv2 00/23] driver items > registration and probing > > > > Hi, team > > > > Today in odp cloud meeting we talked about the DDF status, before the > new LNG colleague joining to take over the DDF works, I'll take the > ownership of the DDF and continue to move this patch series forward and > related bugs, comments. > > > > This Thursday François would like to have a call (1:30 hours) to > introduce knowledge, background and discussions he have had with Christophe > around DDF, all are welcome if you feel interested in DDF topics and please > reply before Wednesday earlier afternoon and I'll send meeting invitation > to you. > > > > I would be interested to join same. However I am based out of India. So > please see if it would be feasible to schedule meeting to accommodate India > time zone. > > Thanks > > Shally > > > > thanks and best regards, Yi > > > > On 22 March 2017 at 22:48, Christophe Milard < > christophe.milard@linaro.org> > > wrote: > > > >> This patch series can be pulled from: > >> https://git.linaro.org/people/christophe.milard/odp.git/log/ > >> ?h=drv_framework_v2 > >> > >> Since V1: Fixes following Bill's comments. > >> > >> Note: I am not really sure this is still in phase with what was > >> discussed at connect, since I couldn't attend. But, at least I did the > >> changes following the comments I recieved. Hope that still makes > >> sence. > >> Also, I am aware that patch 1 generates a warning: I copied this part > >> from the north API so I assume this is an agreed decision. > >> > >> 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_ > >> sKDvRwUD2BXm-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 (23): > >> drv: adding compiler hints in the driver interface > >> linux-gen: adding compiler hints in the driver interface > >> 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 ++- > >> include/odp/drv/spec/hints.h | 119 +++ > >> include/odp_drv.h | 1 + > >> platform/Makefile.inc | 1 + > >> platform/linux-generic/Makefile.am | 1 + > >> platform/linux-generic/_modules.c | 4 + > >> platform/linux-generic/drv_driver.c | 1051 > >> +++++++++++++++++++- > >> .../linux-generic/include/drv_driver_internal.h | 22 + > >> platform/linux-generic/include/odp/drv/hints.h | 34 + > >> 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 + > >> 31 files changed, 3030 insertions(+), 35 deletions(-) create mode > >> 100644 include/odp/drv/spec/hints.h create mode 100644 > >> platform/linux-generic/include/drv_driver_internal.h > >> create mode 100644 platform/linux-generic/include/odp/drv/hints.h > >> create mode 100644 > >> test/common_plat/validation/drv/drvdriver/.gitignore > >> create mode 100644 > >> test/common_plat/validation/drv/drvdriver/Makefile.am > >> create mode 100644 > >> test/common_plat/validation/drv/drvdriver/drvdriver_ > >> device.c > >> create mode 100644 > >> test/common_plat/validation/drv/drvdriver/drvdriver_ > >> device.h > >> create mode 100644 > >> test/common_plat/validation/drv/drvdriver/drvdriver_ > >> device_main.c > >> create mode 100644 test/common_plat/validation/ > >> drv/drvdriver/drvdriver_devio.c create mode 100644 > >> test/common_plat/validation/ drv/drvdriver/drvdriver_devio.h create > >> mode 100644 test/common_plat/validation/ > >> drv/drvdriver/drvdriver_devio_main.c > >> create mode 100644 > >> test/common_plat/validation/drv/drvdriver/drvdriver_ > >> driver.c > >> create mode 100644 > >> test/common_plat/validation/drv/drvdriver/drvdriver_ > >> driver.h > >> create mode 100644 > >> test/common_plat/validation/drv/drvdriver/drvdriver_ > >> driver_main.c > >> create mode 100644 test/common_plat/validation/ > >> drv/drvdriver/drvdriver_enumr.c create mode 100644 > >> test/common_plat/validation/ drv/drvdriver/drvdriver_enumr.h create > >> mode 100644 test/common_plat/validation/ > >> drv/drvdriver/drvdriver_enumr_class.c > >> create mode 100644 test/common_plat/validation/ > >> drv/drvdriver/drvdriver_enumr_class.h > >> create mode 100644 test/common_plat/validation/ > >> drv/drvdriver/drvdriver_enumr_class_main.c > >> create mode 100644 test/common_plat/validation/ > >> drv/drvdriver/drvdriver_enumr_main.c > >> > >> -- > >> 2.7.4 > >> > >> >