Re: patch "staging: dgnc: remove item from TODO list" added to staging-next

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sun, Mar 12, 2017 at 05:24:38PM +0100, gregkh@xxxxxxxxxxxxxxxxxxx wrote:
> 
> This is a note to let you know that I've just added the patch titled
> 
>     staging: dgnc: remove item from TODO list
> 
> to my staging git tree which can be found at
>     git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git
> in the staging-next branch.
> 
> The patch will show up in the next release of the linux-next tree
> (usually sometime within the next 24 hours during the week.)
> 
> The patch will also be merged in the next major kernel release
> during the merge window.
> 
> If you have any questions about this process, please let me know.
> 
> 
> From 2437a9e023fe1df77c053b222762357bb73212e8 Mon Sep 17 00:00:00 2001
> From: "Tobin C. Harding" <me@xxxxxxxx>
> Date: Tue, 7 Mar 2017 20:57:04 +1100
> Subject: staging: dgnc: remove item from TODO list
> 
> TODO file contains task to verify and correct function return
> sites. Need to check for and implement correct usage of goto's when
> there is work to be done before returning.
> 
> Remove task from TODO list after already having completed audit of
> directory drivers/staging/dgnc.
> 
> Signed-off-by: Tobin C. Harding <me@xxxxxxxx>
> Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
> ---
>  drivers/staging/dgnc/TODO | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/drivers/staging/dgnc/TODO b/drivers/staging/dgnc/TODO
> index e26d1d6a5941..d3806bebbcb5 100644
> --- a/drivers/staging/dgnc/TODO
> +++ b/drivers/staging/dgnc/TODO
> @@ -1,7 +1,6 @@
>  * remove unnecessary comments
>  * remove unnecessary error messages. Example kzalloc() has its
>    own error message. Adding an extra one is useless.
> -* use goto statements for error handling when appropriate
>  * there is a lot of unnecessary code in the driver. It was
>    originally a standalone driver. Remove unneeded code.
>  
> -- 
> 2.12.0
> 
> 

Greg, sorry to start your Monday with unnecessary complications. This
patch series has been applied, it was v2 of a series. I had a v3 done
on Friday that I did not get to sending. It has improvements based on
a few back and forth comments with Dan. I would like to still get his
suggestions implemented. My fear is that the patch set that will apply
on top of staging-next now is going to seem a bit trivial if viewed
without the context of the original patch set. Also there will be no
checkpatch warnings to justify the changes.

Did I do something wrong that led to this series being applied before
it was totally ready? Was there something that I should have done to
let you know that there was still work to be done on top of version 2?

What is the easiest way forward for you? 

Series in question is

patch "staging: dgnc: remove item from TODO list" added to staging-next
patch "staging: dgnc: audit goto's in dgnc_tty" added to staging-next
patch "staging: dgnc: audit goto's in dgnc_mgmt" added to staging-next
patch "staging: dgnc: audit goto's in dgnc_driver" added to staging-next


thanks,
Tobin.
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel



[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux