Thread (1 message) 1 message, 1 author, 2012-10-26

Re: [PATCH] can/sja1000: Add miniPCIe/cPCI PC/104Plus PCI/104e PEAK-System boards

From: Marc Kleine-Budde <mkl@pengutronix.de>
Date: 2012-10-26 08:11:27

Possibly related (same subject, not in this thread)

On 10/26/2012 10:01 AM, Stephane Grosjean wrote:
Hi Wolfgang,
Hi everybody,
Well the "push PATCH" method is bit too formal, it should exist a field
where to say "Hello guys, how are you since last time ? Sorry to disturb
you again, but I'd like to push..." ;-)
:D

If you want to add some prose text to the patch, that should not go into
the commit message you simply add it between the "---" and the diffstat:
Signed-off-by: Stephane Grosjean <redacted>
---
...here...
 drivers/net/can/sja1000/peak_pci.c |   12 +++++++++++-
 1 files changed, 11 insertions(+), 1 deletions(-)
Le 24/10/2012 08:29, Wolfgang Grandegger a écrit :
quoted
Hi Stephane,

On 10/23/2012 12:37 PM, Stephane Grosjean wrote:
    MODULE_AUTHOR("Wolfgang Grandegger [off-list ref]");
+MODULE_AUTHOR("Stephane Grosjean [off-list ref]");
Hm, does adding a second MODULE_AUTHOR work? Anyway, feel free to remove
my line.
Yes it does:

$ modinfo peak_pci.ko
filename:       peak_pci.ko
license:        GPL v2
description:    Socket-CAN driver for PEAK PCAN PCI family cards
author:         Stephane Grosjean [off-list ref]
author:         Wolfgang Grandegger [off-list ref]
srcversion:     72C217A41A3CA4ABBFEAFDB
alias:          pci:v0000001Cd00000002sv*sd*bc*sc*i*
alias:          pci:v0000001Cd00000004sv*sd*bc*sc*i*
alias:          pci:v0000001Cd00000007sv*sd*bc*sc*i*
alias:          pci:v0000001Cd00000006sv*sd*bc*sc*i*
alias:          pci:v0000001Cd00000008sv*sd*bc*sc*i*
alias:          pci:v0000001Cd00000005sv*sd*bc*sc*i*
alias:          pci:v0000001Cd00000003sv*sd*bc*sc*i*
alias:          pci:v0000001Cd00000001sv*sd*bc*sc*i*
depends:        sja1000,i2c-algo-bit
vermagic:       3.0.0-14-generic-pae SMP mod_unload modversions 686

(well, don't ask me why my name comes before yours ;-) Maybe the
alphabetic order is used instead of the declaration order...)

So, for me there is no need (and I don't want) to remove yours.
I've replaced you as the author and the patch is already in David
Millers net tree.

Marc

-- 
Pengutronix e.K.                  | Marc Kleine-Budde           |
Industrial Linux Solutions        | Phone: +49-231-2826-924     |
Vertretung West/Dortmund          | Fax:   +49-5121-206917-5555 |
Amtsgericht Hildesheim, HRA 2686  | http://www.pengutronix.de   |

Attachments

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help