Re: What's wrong with my patch?
Thank you for your reply :) > 29 Feb is only one week old, > your patch is in queue, just wait. > > 2016-03-08 6:31 UTC+01:00, Navy Cheng : > > Hi, > > I have sent a patch on 29 Feb, but no reply. I want to know what's wrong > > with my patch so that I can improve it and myself. my patch is like this: > > > >> Date: Mon, 29 Feb 2016 14:41:34 +0800 > >> From: Navy Cheng > >> To: Greg Kroah-Hartman > >> Cc: Lidza Louina , kernelnewbies > >> > >> Subject: [PATCH] staging: dgnc: Remove useless and deadly judgment > >> User-Agent: Mutt/1.5.23 (2014-03-12) > >> > >> pci_unregister_driver() should be used once dgnc module exit. It has > >> nothing to do with dgnc_NumBoards. Remove the judgment of dgnc_NumBoards > >> to > >> avoid pci_unregister_driver() is not used when dgnc_NumBoards is 0. > >> > >> Signed-off-by: Navy Cheng > >> --- > >> drivers/staging/dgnc/dgnc_driver.c | 3 +-- > >> 1 file changed, 1 insertion(+), 2 deletions(-) > >> > >> diff --git a/drivers/staging/dgnc/dgnc_driver.c > >> b/drivers/staging/dgnc/dgnc_driver.c > >> index fc6d298..20b0c3b 100644 > >> --- a/drivers/staging/dgnc/dgnc_driver.c > >> +++ b/drivers/staging/dgnc/dgnc_driver.c > >> @@ -156,8 +156,7 @@ static void dgnc_cleanup_module(void) > >> > >> dgnc_tty_post_uninit(); > >> > >> - if (dgnc_NumBoards) > >> - pci_unregister_driver(&dgnc_driver); > >> + pci_unregister_driver(&dgnc_driver); > >> } > >> > >> /* > > > > Thank you :) > > > > > > ___ > > Kernelnewbies mailing list > > Kernelnewbies@kernelnewbies.org > > http://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies > > > > > -- > « On ne résout pas un problème avec les modes de pensée qui l’ont engendré. » > « You cannot solve current problems with current thinking. Current > problems are the result of current thinking » > > ___ > Kernelnewbies mailing list > Kernelnewbies@kernelnewbies.org > http://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies ___ Kernelnewbies mailing list Kernelnewbies@kernelnewbies.org http://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies
Re: What's wrong with my patch?
29 Feb is only one week old, your patch is in queue, just wait. 2016-03-08 6:31 UTC+01:00, Navy Cheng : > Hi, > I have sent a patch on 29 Feb, but no reply. I want to know what's wrong > with my patch so that I can improve it and myself. my patch is like this: > >> Date: Mon, 29 Feb 2016 14:41:34 +0800 >> From: Navy Cheng >> To: Greg Kroah-Hartman >> Cc: Lidza Louina , kernelnewbies >> >> Subject: [PATCH] staging: dgnc: Remove useless and deadly judgment >> User-Agent: Mutt/1.5.23 (2014-03-12) >> >> pci_unregister_driver() should be used once dgnc module exit. It has >> nothing to do with dgnc_NumBoards. Remove the judgment of dgnc_NumBoards >> to >> avoid pci_unregister_driver() is not used when dgnc_NumBoards is 0. >> >> Signed-off-by: Navy Cheng >> --- >> drivers/staging/dgnc/dgnc_driver.c | 3 +-- >> 1 file changed, 1 insertion(+), 2 deletions(-) >> >> diff --git a/drivers/staging/dgnc/dgnc_driver.c >> b/drivers/staging/dgnc/dgnc_driver.c >> index fc6d298..20b0c3b 100644 >> --- a/drivers/staging/dgnc/dgnc_driver.c >> +++ b/drivers/staging/dgnc/dgnc_driver.c >> @@ -156,8 +156,7 @@ static void dgnc_cleanup_module(void) >> >> dgnc_tty_post_uninit(); >> >> - if (dgnc_NumBoards) >> - pci_unregister_driver(&dgnc_driver); >> + pci_unregister_driver(&dgnc_driver); >> } >> >> /* > > Thank you :) > > > ___ > Kernelnewbies mailing list > Kernelnewbies@kernelnewbies.org > http://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies > -- « On ne résout pas un problème avec les modes de pensée qui l’ont engendré. » « You cannot solve current problems with current thinking. Current problems are the result of current thinking » ___ Kernelnewbies mailing list Kernelnewbies@kernelnewbies.org http://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies
What's wrong with my patch?
Hi, I have sent a patch on 29 Feb, but no reply. I want to know what's wrong with my patch so that I can improve it and myself. my patch is like this: > Date: Mon, 29 Feb 2016 14:41:34 +0800 > From: Navy Cheng > To: Greg Kroah-Hartman > Cc: Lidza Louina , kernelnewbies > > Subject: [PATCH] staging: dgnc: Remove useless and deadly judgment > User-Agent: Mutt/1.5.23 (2014-03-12) > > pci_unregister_driver() should be used once dgnc module exit. It has > nothing to do with dgnc_NumBoards. Remove the judgment of dgnc_NumBoards to > avoid pci_unregister_driver() is not used when dgnc_NumBoards is 0. > > Signed-off-by: Navy Cheng > --- > drivers/staging/dgnc/dgnc_driver.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/staging/dgnc/dgnc_driver.c > b/drivers/staging/dgnc/dgnc_driver.c > index fc6d298..20b0c3b 100644 > --- a/drivers/staging/dgnc/dgnc_driver.c > +++ b/drivers/staging/dgnc/dgnc_driver.c > @@ -156,8 +156,7 @@ static void dgnc_cleanup_module(void) > > dgnc_tty_post_uninit(); > > - if (dgnc_NumBoards) > - pci_unregister_driver(&dgnc_driver); > + pci_unregister_driver(&dgnc_driver); > } > > /* Thank you :) ___ Kernelnewbies mailing list Kernelnewbies@kernelnewbies.org http://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies