Re: [PATCH 1/1] usbserial: cp210x - icount support for parity error checking

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

 



Greg Kroah-Hartman wrote on 6/21/20 11:55 AM:
I read it, but still not sure what exactly was wrong? Yes, I wrapped lines
of description to 80 colums and now I noticed that only 75 columns is
allowed but I doubt that it is all?
That is one thing, but also the "This patch..." should not be in a
changelog, right?  Look at the other changes sent to the list for
examples of how to do this.
Yes, I looked at another messages here and there are a lot of things which I don't understand. For example two dash -- marker at the end (bellow patch) with some strange number (2.7.4). I didn't find anything about that in documentation.

And documentation request diff -up
https://www.kernel.org/doc/html/v4.17/process/submitting-patches.html#diff-up
but patches here use another settings because diff -up never give me line like
index 86638c1..f1b46b5 100644
before file names but put me file date and time next to filename. So what version of diff should I use? I have diff (GNU diffutils) 3.7

Signed-off-by: Jaromir Skorpil <Jerry@xxxxxx>
This does not match your From: line :(
I supposed that only mail address in From line matter?
I understand that real name is mandatory only for Signed-off-by field?
It has to match the From: line of your email to ensure that this really
is the same person.
Really?
I looked at another message as you advised and it seems that even YOUR name often changes?
https://marc.info/?l=linux-usb&m=159257306831535
https://marc.info/?l=linux-usb&m=159256948030250

Why is a name so important when you can't verify it? Typing the same text twice doesn't prove anything. In fact my real name can't be written in ascii because of diacritics marks and I doubt that it will work here correctly if I use unicode...
thanks,

greg k-h
Jerry



[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux