From: Ladislav Michl <Ladislav.Michl@seznam.cz>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 0/5] NetStar updates
Date: Mon, 8 Feb 2010 19:39:37 +0100 [thread overview]
Message-ID: <20100208183937.GA12806@localhost.localdomain> (raw)
In-Reply-To: <0554BEF07D437848AF01B9C9B5F0BC5D98A9CA1D@dlee01.ent.ti.com>
On Mon, Feb 08, 2010 at 10:55:04AM -0600, Paulraj, Sandeep wrote:
> Patches 1, 4 and 5 returned checkpatch errors.
I used scripts/checkpatch.pl from linux-2.6.33-rc5 and for patch 5 it reports
patch "has no obvious style problems and is ready for submission". For others
two it reports lines over 80 characters.
> Please fix them and resend the entire series to the list again.
Sorry, I'm unable to fix it and keep coding style consistent as you would
probably notice yourself looking at files in question. However I'm fine
with sending additional patches to make lines up to 80 characters long.
Or other way, first send bunch of patches making formatting changes
and rebasing functional changes on top of them. But given the fact
reaction comes more than one week after sending patches and Wolfgang's
wish to have everything pulled till 10th, I'm all for sooner way of
proceeding with patches as functionality matter a lot to me...
Anyway, it is your decision, just let me know which way do you prefer.
Thanks,
ladis
next prev parent reply other threads:[~2010-02-08 18:39 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-01-27 23:16 [U-Boot] [PATCH 0/5] NetStar updates Ladislav Michl
2010-01-27 23:18 ` [U-Boot] [PATCH 1/5] NetStar: eeprom - undefined reference to `memset' Ladislav Michl
2010-01-27 23:19 ` [U-Boot] [PATCH 2/5] NetStar: eeprom - be less verbose Ladislav Michl
2010-01-27 23:20 ` [U-Boot] [PATCH 3/5] NetStar: eeprom - fix linker error Ladislav Michl
2010-01-27 23:21 ` [U-Boot] [PATCH 4/5] NetStar: fix default environment Ladislav Michl
2010-01-27 23:22 ` [U-Boot] [PATCH 5/5] NetStar: make mtdparts default ready for recent kernels Ladislav Michl
2010-02-08 16:55 ` [U-Boot] [PATCH 0/5] NetStar updates Paulraj, Sandeep
2010-02-08 18:39 ` Ladislav Michl [this message]
2010-02-08 19:21 ` Paulraj, Sandeep
2010-02-08 21:41 ` Ladislav Michl
2010-02-08 20:45 ` Paulraj, Sandeep
2010-02-09 9:12 ` [U-Boot] [PATCH] netstar.h: do not exceed 80 columns Ladislav Michl
2010-02-18 2:22 ` Paulraj, Sandeep
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=20100208183937.GA12806@localhost.localdomain \
--to=ladislav.michl@seznam.cz \
--cc=u-boot@lists.denx.de \
/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