linux-kernel - Re: [PATCH] s3c2410_udc: fix custom UDC command handling

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] [thread-next>] [day] [month] [year] [list]

Date:	Mon, 20 Jun 2011 18:36:47 +0300
From:	Felipe Balbi <balbi@...com>
To:	Skacore Systems <skacore.systems@...il.com>
Cc:	balbi@...com, Alan Stern <stern@...land.harvard.edu>,
	Viliam Mateicka <viliam.mateicka@...il.com>,
	Ben Dooks <ben-linux@...ff.org>,
	Kukjin Kim <kgene.kim@...sung.com>,
	David Brownell <dbrownell@...rs.sourceforge.net>,
	Greg Kroah-Hartman <gregkh@...e.de>,
	linux-arm-kernel@...ts.infradead.org, linux-usb@...r.kernel.org,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH] s3c2410_udc: fix custom UDC command handling

Hi,

On Mon, Jun 20, 2011 at 05:20:02PM +0200, Skacore Systems wrote:
> I've been exactly following
> http://www.kernel.org/doc/Documentation/SubmittingPatches:
> 
> To create a patch for a single file, it is often sufficient to do:
> 
> 	SRCTREE= linux-2.6
> 	MYFILE=  drivers/net/mydriver.c
> 
> 	cd $SRCTREE
> 	cp $MYFILE $MYFILE.orig
> 	vi $MYFILE	# make your change
> 	cd ..
> 	diff -up $SRCTREE/$MYFILE{.orig,} > /tmp/patch
> 
> And that's what I've done. What do you think is wrong with it ???

patch should be in 'p1' format to start with, not 'p0'!!

Your commit log shouldn't include your "regards" or anything like that,
or that will go into the git history.

> Subject: [PATCH] s3c2410_udc: fix custom UDC command handling

this should be something like:

[PATCH] usb: gadget: s3c2410_udc: fix custom UDC command handling

we generally put the subsystem on the subject to help maintainers
filter patches related to them.

> There is a bug in Samsung's UDC driver, which is completely disabling
> the USB device when a custom UDC command is used.
> Following patch seems to get the right?behavior (e.g. enabling pull-up
> instead of disabling then Vcc is applied).

"seems to get the right behavior" makes me wonder if you really tested
this properly. Either it fixes the problem, or it doesn't. Have you
tried with other possible commands to see if also helps ?

> Also I'm not in mailing list so please CC me.

this would go to commit log, shouldn't be here.

> Greetingz,
> Vilo

this shouldn't be here either.

> Signed-off-by: Viliam Mateicka <viliam.mateicka@...il.com>

there should be a tear line here. (---)

> --- drivers/usb/gadget/s3c2410_udc.c.orig ? ? ? 2011-05-19
> 06:06:34.000000000 +0200

this line break is bogus.

you could have looked at any mailing list archive for properly formatted
patches, but instead you chose to be unpolite with the persons who are
giving you a "service" free of charge.

At [1] you can find an example of a proper patch

[1] http://marc.info/?l=linux-usb&m=130833192703040&w=2

-- 
balbi

Download attachment "signature.asc" of type "application/pgp-signature" (491 bytes)