Re: [PATCH 07/10] staging: mt7621-mmc: Remove error message in debug

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

 



On Fri, Apr 06 2018, Christian Lütke-Stetzkamp wrote:

> On Fri, Apr 06, 2018 at 11:51:24AM +0300, Dan Carpenter wrote:
>> On Fri, Apr 06, 2018 at 10:36:57AM +0200, Christian Lütke-Stetzkamp wrote:
>> > If the debug file in proc fs is not successfully created current code
>> > prints an error message, this is removed.
>> > 
>> > Reported-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
>> > Signed-off-by: Christian Lütke-Stetzkamp <christian@xxxxxxxx>
>> > ---
>> >  drivers/staging/mt7621-mmc/dbg.c | 4 ----
>> >  1 file changed, 4 deletions(-)
>> > 
>> > diff --git a/drivers/staging/mt7621-mmc/dbg.c b/drivers/staging/mt7621-mmc/dbg.c
>> > index abf92a84a417..9ad330729300 100644
>> > --- a/drivers/staging/mt7621-mmc/dbg.c
>> > +++ b/drivers/staging/mt7621-mmc/dbg.c
>> > @@ -340,10 +340,6 @@ static const struct file_operations msdc_debug_fops = {
>> >  int msdc_debug_proc_init(void)
>> >  {
>> >  	struct proc_dir_entry *de = proc_create("msdc_debug", 0667, NULL, &msdc_debug_fops);
>> > -
>> > -	if (!de || IS_ERR(de))
>> > -		printk("!! Create MSDC debug PROC fail !!\n");
>> > -
>> 
>> The warning is that the IS_ERR() can be removed...  You can remove the
>> NULL check too though, I don't care.  But then, please remove the whole
>> "struct proc_dir_entry *de = " assignment as well?
>
> Thank, I overlooked it, will remove the assignment and resend a v2
> tomorrow.

While you are at it, maybe change msdc_debug_proc_init() to return
'void' as the return value is never used.

Patches look good to me, and my gnubee still boots and works, so
  Reviewed-by: NeilBrown <neil@xxxxxxxxxx>

for the series.

Thanks,
NeilBrown

Attachment: signature.asc
Description: PGP signature

_______________________________________________
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