From: David Miller <davem@davemloft.net>
To: joseph.kogut@gmail.com
Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] Drivers: net: 8139cp: Fixed 28 style errors, and 119 warnings.
Date: Thu, 15 Jul 2010 18:30:10 -0700 (PDT) [thread overview]
Message-ID: <20100715.183010.193726592.davem@davemloft.net> (raw)
In-Reply-To: <AANLkTikspw9LxFFsXJgWn30k2qCVu5zkOAq2FhWTpQvW@mail.gmail.com>
Besides the feedback given by Jeff Garzik, I would like to ask that
you please post your patches in a format other than as a binary
attachment. Plain text attachments are fine, for example.
That way your patch will be properly logged and tracked in
patchwork, and be easier for people to reply to and comment
upon.
Thank you.
prev parent reply other threads:[~2010-07-16 1:30 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-07-15 22:36 [PATCH] Drivers: net: 8139cp: Fixed 28 style errors, and 119 warnings Joseph Kogut
2010-07-16 0:57 ` Jeff Garzik
2010-07-16 1:30 ` David Miller [this message]
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=20100715.183010.193726592.davem@davemloft.net \
--to=davem@davemloft.net \
--cc=joseph.kogut@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=netdev@vger.kernel.org \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).