[prev in list] [next in list] [prev in thread] [next in thread] 

List:       linux-can
Subject:    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 8:11:22
Message-ID: 508A45AA.3080105 () pengutronix ! de
[Download RAW message or body]

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 <s.grosjean@peak-system.com>
> ---

...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 :
>> Hi Stephane,
>>
>> On 10/23/2012 12:37 PM, Stephane Grosjean wrote:
>>     MODULE_AUTHOR("Wolfgang Grandegger <wg@grandegger.com>");
>> +MODULE_AUTHOR("Stephane Grosjean <s.grosjean@peak-system.com>");
>> 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 <s.grosjean@peak-system.com>
> author:         Wolfgang Grandegger <wg@grandegger.com>
> 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   |


["signature.asc" (application/pgp-signature)]
--
To unsubscribe from this list: send the line "unsubscribe linux-can" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[prev in list] [next in list] [prev in thread] [next in thread]