All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mauro Carvalho Chehab <mchehab@redhat.com>
To: Antonio Ospite <ospite@studenti.unina.it>
Cc: Jean-Francois Moine <moinejf@free.fr>, linux-media@vger.kernel.org
Subject: Re: [GIT PATCHES FOR 2.6.40] gspca for_v2.6.40
Date: Fri, 20 May 2011 11:43:15 -0300	[thread overview]
Message-ID: <4DD67E03.4070002@redhat.com> (raw)
In-Reply-To: <20110517113637.923e0da2.ospite@studenti.unina.it>

Em 17-05-2011 06:36, Antonio Ospite escreveu:
> On Tue, 17 May 2011 10:54:17 +0200
> Jean-Francois Moine <moinejf@free.fr> wrote:
> 
>> The following changes since commit
>> f9b51477fe540fb4c65a05027fdd6f2ecce4db3b:
>>
>>   [media] DVB: return meaningful error codes in dvb_frontend (2011-05-09 05:47:20 +0200)
>>
>> are available in the git repository at:
>>   git://linuxtv.org/jfrancois/gspca.git for_v2.6.40
>>
> [...]
> 
> Hi Jean-Francois, sometimes it is useful to add also a "why" section to
> commit messages so others can follow your thoughts, and even learn from
> them.
> 
> I have this very simple scheme: a summary of the "what" goes into the
> short commit message and the "why" and "how" go into the long commit
> message when they are not immediately trivial from the code; for
> instance the "why" of the USB trace changes in this series wasn't
> trivial to me.

Yeah, providing a good documentation is important to allow the others to know what's
happening at the driver.

Jean-Francois,

Could you please add more comments to your patch series? I'll discard this pull
request from my queue. So, feel free to re-base your tree if you need.

Thanks,
Mauro

  reply	other threads:[~2011-05-20 14:43 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-05-17  8:54 [GIT PATCHES FOR 2.6.40] gspca for_v2.6.40 Jean-Francois Moine
2011-05-17  9:36 ` Antonio Ospite
2011-05-20 14:43   ` Mauro Carvalho Chehab [this message]
  -- strict thread matches above, loose matches on Subject: below --
2011-05-23  9:20 Jean-Francois Moine
2011-06-01 11:29 ` Mauro Carvalho Chehab
2011-05-21  6:59 Jean-Francois Moine
2011-05-21  9:27 ` Antonio Ospite
2011-05-21 12:37 ` Mauro Carvalho Chehab
2011-04-19 18:20 Jean-Francois Moine
2011-04-19 19:54 ` Antonio Ospite
2011-04-19 20:34   ` Mauro Carvalho Chehab

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4DD67E03.4070002@redhat.com \
    --to=mchehab@redhat.com \
    --cc=linux-media@vger.kernel.org \
    --cc=moinejf@free.fr \
    --cc=ospite@studenti.unina.it \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.