* TODO list for staging/r8188eu @ 2021-08-22 7:41 Fabio M. De Francesco 2021-08-22 12:36 ` Greg Kroah-Hartman 0 siblings, 1 reply; 7+ messages in thread From: Fabio M. De Francesco @ 2021-08-22 7:41 UTC (permalink / raw) To: Larry Finger, Phillip Potter, Greg Kroah-Hartman Cc: open list:STAGING SUBSYSTEM, open list, Michael Straube, Martin Kaiser Dear Larry, Philip, From what I understand how the development process works, drivers in staging should have a to-do list in the TODO file. Please read https://www.kernel.org/ doc/html/latest/process/2.Process.html?highlight=todo#staging-trees. Could you (as the maintainers of the r8188eu driver) please compile and provide the above mentioned list? Thanks, Fabio ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: TODO list for staging/r8188eu 2021-08-22 7:41 TODO list for staging/r8188eu Fabio M. De Francesco @ 2021-08-22 12:36 ` Greg Kroah-Hartman 2021-08-22 13:50 ` Fabio M. De Francesco 0 siblings, 1 reply; 7+ messages in thread From: Greg Kroah-Hartman @ 2021-08-22 12:36 UTC (permalink / raw) To: Fabio M. De Francesco Cc: Larry Finger, Phillip Potter, open list:STAGING SUBSYSTEM, open list, Michael Straube, Martin Kaiser On Sun, Aug 22, 2021 at 09:41:20AM +0200, Fabio M. De Francesco wrote: > Dear Larry, Philip, > > >From what I understand how the development process works, drivers in staging > should have a to-do list in the TODO file. Please read https://www.kernel.org/ > doc/html/latest/process/2.Process.html?highlight=todo#staging-trees. > > Could you (as the maintainers of the r8188eu driver) please compile and > provide the above mentioned list? Why don't you provide an initial list for people to work off of if you feel it is needed here? thanks, greg k-h ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: TODO list for staging/r8188eu 2021-08-22 12:36 ` Greg Kroah-Hartman @ 2021-08-22 13:50 ` Fabio M. De Francesco 2021-08-22 14:33 ` Greg Kroah-Hartman 2021-08-22 15:09 ` Phillip Potter 0 siblings, 2 replies; 7+ messages in thread From: Fabio M. De Francesco @ 2021-08-22 13:50 UTC (permalink / raw) To: Greg Kroah-Hartman Cc: Larry Finger, Phillip Potter, open list:STAGING SUBSYSTEM, open list, Michael Straube, Martin Kaiser On Sunday, August 22, 2021 2:36:57 PM CEST Greg Kroah-Hartman wrote: > On Sun, Aug 22, 2021 at 09:41:20AM +0200, Fabio M. De Francesco wrote: > > Dear Larry, Philip, > > > > >From what I understand how the development process works, drivers in staging > > > > should have a to-do list in the TODO file. Please read https:// www.kernel.org/ > > doc/html/latest/process/2.Process.html?highlight=todo#staging-trees. > > > > Could you (as the maintainers of the r8188eu driver) please compile and > > provide the above mentioned list? > > Why don't you provide an initial list for people to work off of if you > feel it is needed here? > > thanks, > > greg k-h Unfortunately I'm not able to tell what is needed to do to have a driver improved so that it can be moved off staging. This work should be better addressed by someone who is much more experienced. For example, I read from other drivers TODO lists that cfg80211 and lib80211 are required but I don't know what they are. Regards, Fabio ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: TODO list for staging/r8188eu 2021-08-22 13:50 ` Fabio M. De Francesco @ 2021-08-22 14:33 ` Greg Kroah-Hartman 2021-08-22 15:09 ` Phillip Potter 1 sibling, 0 replies; 7+ messages in thread From: Greg Kroah-Hartman @ 2021-08-22 14:33 UTC (permalink / raw) To: Fabio M. De Francesco Cc: Larry Finger, Phillip Potter, open list:STAGING SUBSYSTEM, open list, Michael Straube, Martin Kaiser On Sun, Aug 22, 2021 at 03:50:14PM +0200, Fabio M. De Francesco wrote: > On Sunday, August 22, 2021 2:36:57 PM CEST Greg Kroah-Hartman wrote: > > On Sun, Aug 22, 2021 at 09:41:20AM +0200, Fabio M. De Francesco wrote: > > > Dear Larry, Philip, > > > > > > >From what I understand how the development process works, drivers in > staging > > > > > > should have a to-do list in the TODO file. Please read https:// > www.kernel.org/ > > > doc/html/latest/process/2.Process.html?highlight=todo#staging-trees. > > > > > > Could you (as the maintainers of the r8188eu driver) please compile and > > > provide the above mentioned list? > > > > Why don't you provide an initial list for people to work off of if you > > feel it is needed here? > > > > thanks, > > > > greg k-h > > Unfortunately I'm not able to tell what is needed to do to have a driver > improved so that it can be moved off staging. This work should be better > addressed by someone who is much more experienced. > > For example, I read from other drivers TODO lists that cfg80211 and lib80211 > are required but I don't know what they are. Those are two things that are also required here as well, like all in-kernel wifi drivers. ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: TODO list for staging/r8188eu 2021-08-22 13:50 ` Fabio M. De Francesco 2021-08-22 14:33 ` Greg Kroah-Hartman @ 2021-08-22 15:09 ` Phillip Potter 2021-08-22 16:13 ` Fabio M. De Francesco 1 sibling, 1 reply; 7+ messages in thread From: Phillip Potter @ 2021-08-22 15:09 UTC (permalink / raw) To: Fabio M. De Francesco Cc: Greg Kroah-Hartman, Larry Finger, open list:STAGING SUBSYSTEM, open list, Michael Straube, Martin Kaiser On Sun, 22 Aug 2021 at 14:50, Fabio M. De Francesco <fmdefrancesco@gmail.com> wrote: > > On Sunday, August 22, 2021 2:36:57 PM CEST Greg Kroah-Hartman wrote: > > On Sun, Aug 22, 2021 at 09:41:20AM +0200, Fabio M. De Francesco wrote: > > > Dear Larry, Philip, > > > > > > >From what I understand how the development process works, drivers in > staging > > > > > > should have a to-do list in the TODO file. Please read https:// > www.kernel.org/ > > > doc/html/latest/process/2.Process.html?highlight=todo#staging-trees. > > > > > > Could you (as the maintainers of the r8188eu driver) please compile and > > > provide the above mentioned list? > > > > Why don't you provide an initial list for people to work off of if you > > feel it is needed here? > > > > thanks, > > > > greg k-h > > Unfortunately I'm not able to tell what is needed to do to have a driver > improved so that it can be moved off staging. This work should be better > addressed by someone who is much more experienced. > > For example, I read from other drivers TODO lists that cfg80211 and lib80211 > are required but I don't know what they are. > > Regards, > > Fabio > > > Dear Fabio, I believe Larry is working on hooking up CFG80211 again (he mentioned going quite for a few days to work on it). As for other stuff that needs doing, a few things come to mind: * Removal of dead code (whether or not this includes the debugging/printing macros is a matter of opinion, I would say yes personally) such as unusued functions, struct fields, etc. * Correction of code styling in the kernel (e.g. there is a lot of camel case). * Use of in-kernel functionality and removal of unnecessary wrappers where possible. * Removal of the HAL layer and migration of its functionality into the driver as a whole - this is an important one I would argue. There is likely to be lots of other items not on that list, but that would seem to be the basis of a TODO list if we need one? Regards, Phil ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: TODO list for staging/r8188eu 2021-08-22 15:09 ` Phillip Potter @ 2021-08-22 16:13 ` Fabio M. De Francesco 2021-08-23 0:00 ` Phillip Potter 0 siblings, 1 reply; 7+ messages in thread From: Fabio M. De Francesco @ 2021-08-22 16:13 UTC (permalink / raw) To: Phillip Potter Cc: Greg Kroah-Hartman, Larry Finger, open list:STAGING SUBSYSTEM, open list, Michael Straube, Martin Kaiser On Sunday, August 22, 2021 5:09:47 PM CEST Phillip Potter wrote: > On Sun, 22 Aug 2021 at 14:50, Fabio M. De Francesco > <fmdefrancesco@gmail.com> wrote: > > > > On Sunday, August 22, 2021 2:36:57 PM CEST Greg Kroah-Hartman wrote: > > > On Sun, Aug 22, 2021 at 09:41:20AM +0200, Fabio M. De Francesco wrote: > > > > Dear Larry, Philip, > > > > > > > > >From what I understand how the development process works, drivers in > > staging > > > > > > > > should have a to-do list in the TODO file. Please read https:// > > www.kernel.org/ > > > > doc/html/latest/process/2.Process.html?highlight=todo#staging-trees. > > > > > > > > Could you (as the maintainers of the r8188eu driver) please compile and > > > > provide the above mentioned list? > > > > > > Why don't you provide an initial list for people to work off of if you > > > feel it is needed here? > > > > > > thanks, > > > > > > greg k-h > > > > Unfortunately I'm not able to tell what is needed to do to have a driver > > improved so that it can be moved off staging. This work should be better > > addressed by someone who is much more experienced. > > > > For example, I read from other drivers TODO lists that cfg80211 and lib80211 > > are required but I don't know what they are. > > > > Regards, > > > > Fabio > > Dear Fabio, > > I believe Larry is working on hooking up CFG80211 again (he mentioned > going quite for a few days to work on it). As for other stuff that > needs doing, a few things come to mind: > > * Removal of dead code (whether or not this includes the > debugging/printing macros is a matter of opinion, I would say yes > personally) such as unused functions, struct fields, etc. > * Correction of code styling in the kernel (e.g. there is a lot of camel case). > * Use of in-kernel functionality and removal of unnecessary wrappers > where possible. > * Removal of the HAL layer and migration of its functionality into the > driver as a whole - this is an important one I would argue. > > There is likely to be lots of other items not on that list, but that > would seem to be the basis of a TODO list if we need one? > > Regards, > Phil Dear Philip, Thanks for this initial list. I suppose that every driver in staging should have it. If you like I could send a patch with the list you prepared, unless you prefer to make it yourself (since you wrote it). Please let me know what you prefer. We shouldn't forget that from time to time new people decide to enter kernel hacking through staging. I think that a list could help all future newcomers. Thanks, Fabio ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: TODO list for staging/r8188eu 2021-08-22 16:13 ` Fabio M. De Francesco @ 2021-08-23 0:00 ` Phillip Potter 0 siblings, 0 replies; 7+ messages in thread From: Phillip Potter @ 2021-08-23 0:00 UTC (permalink / raw) To: Fabio M. De Francesco Cc: Greg Kroah-Hartman, Larry Finger, open list:STAGING SUBSYSTEM, open list, Michael Straube, Martin Kaiser On Sun, 22 Aug 2021 at 17:13, Fabio M. De Francesco <fmdefrancesco@gmail.com> wrote: > > On Sunday, August 22, 2021 5:09:47 PM CEST Phillip Potter wrote: > > On Sun, 22 Aug 2021 at 14:50, Fabio M. De Francesco > > <fmdefrancesco@gmail.com> wrote: > > > > > > On Sunday, August 22, 2021 2:36:57 PM CEST Greg Kroah-Hartman wrote: > > > > On Sun, Aug 22, 2021 at 09:41:20AM +0200, Fabio M. De Francesco wrote: > > > > > Dear Larry, Philip, > > > > > > > > > > >From what I understand how the development process works, drivers in > > > staging > > > > > > > > > > should have a to-do list in the TODO file. Please read https:// > > > www.kernel.org/ > > > > > doc/html/latest/process/2.Process.html?highlight=todo#staging-trees. > > > > > > > > > > Could you (as the maintainers of the r8188eu driver) please compile and > > > > > provide the above mentioned list? > > > > > > > > Why don't you provide an initial list for people to work off of if you > > > > feel it is needed here? > > > > > > > > thanks, > > > > > > > > greg k-h > > > > > > Unfortunately I'm not able to tell what is needed to do to have a driver > > > improved so that it can be moved off staging. This work should be better > > > addressed by someone who is much more experienced. > > > > > > For example, I read from other drivers TODO lists that cfg80211 and lib80211 > > > are required but I don't know what they are. > > > > > > Regards, > > > > > > Fabio > > > > Dear Fabio, > > > > I believe Larry is working on hooking up CFG80211 again (he mentioned > > going quite for a few days to work on it). As for other stuff that > > needs doing, a few things come to mind: > > > > * Removal of dead code (whether or not this includes the > > debugging/printing macros is a matter of opinion, I would say yes > > personally) such as unused functions, struct fields, etc. > > * Correction of code styling in the kernel (e.g. there is a lot of camel case). > > * Use of in-kernel functionality and removal of unnecessary wrappers > > where possible. > > * Removal of the HAL layer and migration of its functionality into the > > driver as a whole - this is an important one I would argue. > > > > There is likely to be lots of other items not on that list, but that > > would seem to be the basis of a TODO list if we need one? > > > > Regards, > > Phil > > Dear Philip, > > Thanks for this initial list. I suppose that every driver in staging should have it. > If you like I could send a patch with the list you prepared, unless you prefer to > make it yourself (since you wrote it). Please let me know what you prefer. > > We shouldn't forget that from time to time new people decide to enter kernel > hacking through staging. I think that a list could help all future newcomers. > > Thanks, > > Fabio > > > > > > > Dear Fabio, By all means feel free to submit this - there will be other stuff people will probably want to add to it also, that is just what I thought of so far. Many thanks. Regards, Phil ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2021-08-23 0:00 UTC | newest] Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2021-08-22 7:41 TODO list for staging/r8188eu Fabio M. De Francesco 2021-08-22 12:36 ` Greg Kroah-Hartman 2021-08-22 13:50 ` Fabio M. De Francesco 2021-08-22 14:33 ` Greg Kroah-Hartman 2021-08-22 15:09 ` Phillip Potter 2021-08-22 16:13 ` Fabio M. De Francesco 2021-08-23 0:00 ` Phillip Potter
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).