All of lore.kernel.org
 help / color / mirror / Atom feed
From: giometti@enneenne.com (Rodolfo Giometti)
To: linux-arm-kernel@lists.infradead.org
Subject: [RFC PATCH v2] ARM: at91: add support for Cosino board series by HCE Engineering
Date: Thu, 14 Nov 2013 11:20:56 +0100	[thread overview]
Message-ID: <20131114102055.GG10964@enneenne.com> (raw)
In-Reply-To: <52849021.4060809@atmel.com>

On Thu, Nov 14, 2013 at 09:56:01AM +0100, Nicolas Ferre wrote:
> On 13/11/2013 18:46, Rodolfo Giometti :
> >On Wed, Nov 13, 2013 at 06:34:30PM +0100, Nicolas Ferre wrote:
> >>From: Rodolfo Giometti <giometti@linux.it>
> >>
> >>This patch adds the Cosino at91sam9g35 based CPU module and the
> >>Cosino Mega 2560 extension board.
> >>
> >>Web site: http://www.cosino.it
> >>
> >>Signed-off-by: Rodolfo Giometti <giometti@linux.it>
> >>[nicolas.ferre at atmel.com: adapted to newer kernel, modified commit message]
> >>Signed-off-by: Nicolas Ferre <nicolas.ferre@atmel.com>
> >>---
> >>Rodolfo,
> >>
> >>Can you please review this adaptation of your previous patch to newer kernel.
> >>I hope to integrate your board in upcoming 3.14.
> >>
> >>You will see that I addressed the remarks by Jean-Christophe and Russell (about
> >>commit message). Please tell me if it is good on your side.
> >
> >I did some changes... attached are the correct patches.
> 
> Doing the diff between our patch I only seen a difference in nand binding:
> nand-on-flash-bbt;
> is present in my adaptation while it is not in yours but I do not
> think it is a problem to keep it.

Well, I tested Cosino's NAND flash without such parameter... however
if you think it could not be a problem you can leave it.

> >Here the changelog V1 -> V2:
> >
> >[Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>]
> >* prefix "at91-" added
> >* atmel,mux-mask stuff removed
> >* "compatible" stuff is now on the same line for easy-grep(TM)
> >* lcd binding dropped
> 
> Absolutely, I will add it to the commit log and integrate in the
> upcoming at91-3.14-dt branch.
> 
> Thanks for your quick feedback. Bye,

Great! Thanks. :)

Ciao,

Rodolfo

-- 

GNU/Linux Solutions                  e-mail: giometti at enneenne.com
Linux Device Driver                          giometti at linux.it
Embedded Systems                     phone:  +39 349 2432127
UNIX programming                     skype:  rodolfo.giometti
Freelance ICT Italia - Consulente ICT Italia - www.consulenti-ict.it

  reply	other threads:[~2013-11-14 10:20 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-11-06 16:19 [PATCH 1/1] arm mach-at91: add support for Cosino board series by HCE Engineering Rodolfo Giometti
2013-11-06 16:21 ` Russell King - ARM Linux
2013-11-06 16:25   ` Rodolfo Giometti
2013-11-07  6:06 ` Jean-Christophe PLAGNIOL-VILLARD
2013-11-13 17:34 ` [RFC PATCH v2] ARM: at91: " Nicolas Ferre
2013-11-13 17:46   ` Rodolfo Giometti
2013-11-14  8:56     ` Nicolas Ferre
2013-11-14 10:20       ` Rodolfo Giometti [this message]
2013-11-14 16:28   ` Jean-Christophe PLAGNIOL-VILLARD
2013-11-14 18:50     ` Rodolfo Giometti
2013-11-15 13:52       ` Jean-Christophe PLAGNIOL-VILLARD
2013-11-20 18:18         ` Rodolfo Giometti
2013-11-20 18:50           ` Jean-Christophe PLAGNIOL-VILLARD
2013-11-21  8:24             ` Nicolas Ferre

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=20131114102055.GG10964@enneenne.com \
    --to=giometti@enneenne.com \
    --cc=linux-arm-kernel@lists.infradead.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 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.