From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 0/2] update sunxi-boards and sunxi-tools for pcDuino board
Date: Wed, 11 Dec 2013 22:53:36 +0100 [thread overview]
Message-ID: <20131211225336.489ff1cc@skate> (raw)
In-Reply-To: <1386721499.2817.17.camel@kokopelli>
Dear Brian Beattie,
On Tue, 10 Dec 2013 16:24:59 -0800, Brian Beattie wrote:
> In preparation for adding support for for the pcduino board, update the
> sunxi-boards and sunxi-tools to the commit recommended for pcduino.
>
> These packages are also used by the Cubieboard so if somebody with a
> Cubieboard could test this that would be good.
>
> Brian Beattie (2):
> update sunxi-boards to newer commit as recomended for pcDuino board
> update sunxi-boards to newer commit as recomended for pcDuino board
Thanks for your patches! They look good, except for a few minor things:
* Their title should be something like:
<package name>: <what is happening to them>
so typically:
sunxi-boards: bump version, to support the pcDuino board
sunxi-tools: bump version, to support the pcDuino board
* The title of your second patch was wrong: it bumps the version of
sunxi-tools, not sunxi-boards.
* You should sent your patches with 'git send-email'. You've sent them
with Evolution, but Evolution has line-wrapped the patches, which
prevents them from being applied. Also, it will ensure that your
patches are sent as replies to your cover letter, all automatically.
* Your patches should have a Signed-off-by line.
See
http://buildroot.org/downloads/manual/manual.html#_contributing_to_buildroot
for more details.
Could you resend a version 2 of your patch series with these minor
issues fixed?
Also, since you're using the pcDuino board, could you submit a simple
defconfig that builds a minimal system for this platform (i.e toolchain
+ bootloader + kernel + minimal rootfs) ?
Thanks!
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
next prev parent reply other threads:[~2013-12-11 21:53 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-12-11 0:24 [Buildroot] [PATCH 0/2] update sunxi-boards and sunxi-tools for pcDuino board Brian Beattie
2013-12-11 21:53 ` Thomas Petazzoni [this message]
2014-04-20 21:04 ` Thomas Petazzoni
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=20131211225336.489ff1cc@skate \
--to=thomas.petazzoni@free-electrons.com \
--cc=buildroot@busybox.net \
/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.