public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Albert ARIBAUD <albert.u.boot@aribaud.net>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PULL] please pull u-boot-atmel/master
Date: Mon, 5 Nov 2012 14:09:52 +0100	[thread overview]
Message-ID: <20121105140952.69a10a88@lilith> (raw)
In-Reply-To: <1352110770-7198-1-git-send-email-andreas.devel@googlemail.com>

Hi Andreas,

On Mon,  5 Nov 2012 11:19:30 +0100, Andreas Bie?mann
<andreas.devel@googlemail.com> wrote:

> Dear Albert Aribaud,
> 
> The following changes since commit 46d626d3926cf6b9f9c477782c5a0ee620a424cd:
> 
>   socfpga/spl: Remove malloc.h (2012-11-04 16:23:23 +0100)
> 
> are available in the git repository at:
> 
>   git://git.denx.de/u-boot-atmel.git master
> 
> for you to fetch changes up to 15915a45d438b9e65e03c3052c3dbab1374927bc:
> 
>   at91: move get_chip_id() and get_extension_chip_id() to cpu.c, which can share with all at91 board. (2012-11-05 11:15:12 +0100)
> 
> ----------------------------------------------------------------
> Bo Shen (1):
>       video: atmel: implement lcd_setcolreg funtion

What is the point of adding this function? This does not seem to be a
strong override of a weak symbol, and therefore, it is dead code unless
this patch is a dependency of one or several other patches -- in which
case, please submit the other patch(es) along this one in your PR.

> Wu, Josh (1):
>       at91: move get_chip_id() and get_extension_chip_id() to cpu.c, which can share with all at91 board.

This one, despite being a cleaner rewrite, also seems to be a
preparatory patch waiting to be used.

The whole PR is dead/dormant code. Please resubmit when dependent
patches are added (or consider rolling back your master until you get
these dependent patches).

Amicalement,
-- 
Albert.

  reply	other threads:[~2012-11-05 13:09 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-11-05 10:19 [U-Boot] [PULL] please pull u-boot-atmel/master Andreas Bießmann
2012-11-05 13:09 ` Albert ARIBAUD [this message]
2012-11-06  9:19   ` Andreas Bießmann
  -- strict thread matches above, loose matches on Subject: below --
2013-05-21  9:59 Andreas Bießmann
2013-05-22  9:59 ` Albert ARIBAUD
2013-05-12 15:01 Andreas Bießmann
2013-05-13  9:25 ` Albert ARIBAUD
2013-04-05 20:22 Andreas Bießmann
2013-04-05 20:53 ` Albert ARIBAUD
2012-12-09 21:54 [U-Boot] [PULL] Please " Andreas Bießmann
2013-01-09 19:03 ` Albert ARIBAUD
2012-10-23 18:21 [U-Boot] [PULL] please " Andreas Bießmann
2012-10-26  7:58 ` Albert ARIBAUD
2012-09-04 21:04 [U-Boot] =?true?q?=5BPULL=5D=20Please=20pull=20u-boot-atmel/master?= Andreas Bießmann
2012-09-05  6:45 ` [U-Boot] [PULL] Please pull u-boot-atmel/master Albert ARIBAUD

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=20121105140952.69a10a88@lilith \
    --to=albert.u.boot@aribaud.net \
    --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