linux-kernel - Re: [PATCH] usb: musb: tweak musb_read

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-Id: <200902112201.12902.david-b@pacbell.net>
Date:	Wed, 11 Feb 2009 22:01:12 -0800
From:	David Brownell <david-b@...bell.net>
To:	Mike Frysinger <vapier.adi@...il.com>
Cc:	me@...ipebalbi.com, Bryan Wu <cooloney@...nel.org>,
	felipe.balbi@...ia.com, linux-usb@...r.kernel.org,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH] usb: musb: tweak musb_read_fifo() to avoid unused  warnings

On Friday 06 February 2009, Mike Frysinger wrote:
> > Is this patch avoiding a warning about these variables being unused ?
> >
> > Would be nice to get this info to the patch body ;-)
> 
> the subject already says that

Not enough.

Mike, why is it that patches from ADI generally don't include
patch descriptions?   It's not like folk haven't asked for them...

In the case of "avoid a warning" patches, the convention is to
have the patch summary include the warnings being avoided.
(Or at least a few, if they're too numerous.)



--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/