All of lore.kernel.org
 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 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.