public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Lukasz Majewski <l.majewski@samsung.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] u-boot/master slight regression with trats
Date: Mon, 01 Oct 2012 12:25:34 +0200	[thread overview]
Message-ID: <20121001122534.2bbff03f@amdc308.digital.local> (raw)
In-Reply-To: <20121001112159.1bd9548c@amdc308.digital.local>

Hi all,

> 
> > 
> > On Sat, 29 Sep 2012 17:17:48 +0200
> > Albert ARIBAUD <albert.u.boot@aribaud.net> wrote:
> > 
> > > Hi all,
> > > 
> > > On Sat, 29 Sep 2012 12:32:36 +0200, Albert ARIBAUD
> > > <albert.u.boot@aribaud.net> wrote:
> > > 
> > > > Hello,
> > > > 
> > > > While merging u-boot-arm/master and u-boot/master I have noticed
> > > > one slight regression apparently coming from u-boot:
> > > > 
> > > > $ cat LOG/trats.ERR 
> > > > cmd_fat.c: In function ?do_fat_fswrite?:
> > > > cmd_fat.c:178:8: warning: unused variable
> > > > ?ep? [-Wunused-variable]
> > > > 
> > > > ep should have disappeared along with the code which used it,
> > > > but somehow resurfaced (I suspect it did in a merge).
> > > > 
> > > > This warning is not seen in u-boot-arm.
> > > > 
> > > > Amicalement,
> > > 
> > > Seems Heungjun Kim <riverful.kim@samsung.com> is now a dead
> > > address.
> > > 
> > > Anatolij, I see you've sent out a patch. Does it mean you would
> > > act as a maintainer for trats? 
> > > Otherwise the board should be made orphan.
> 
> Dear all, I was on a holidays :-). 
> 
> I can test if Trats is working. I posses one, so there is no problem
> with that :-).
> 
> I will fix Trats ASAP.

I've just tested TRATS with the newest origin/master 
(SHA1: 4668a086bb0a769b741e3a4ffab85f1c41c7cdb8 )

It builds without errors/warinigs

> 
> > 
> > I do not have a trats board, so I can't do tests and act as a
> > maintainer for it. Lukasz (on Cc) recently submitted some patches
> > for trats, maybe he knows who could become new maintainer for trats
> > board?
> > 




-- 
Best regards,

Lukasz Majewski

Samsung Poland R&D Center | Linux Platform Group

  parent reply	other threads:[~2012-10-01 10:25 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-09-29 10:32 [U-Boot] u-boot/master slight regression with trats Albert ARIBAUD
2012-09-29 10:52 ` [U-Boot] [PATCH] cmd_fat.c: fix build warning Anatolij Gustschin
2012-09-29 11:14   ` Anatolij Gustschin
2012-09-29 15:19     ` Albert ARIBAUD
2012-09-29 17:01   ` Tom Rini
2012-09-29 15:17 ` [U-Boot] u-boot/master slight regression with trats Albert ARIBAUD
2012-09-29 18:22   ` Anatolij Gustschin
2012-10-01  9:21     ` Lukasz Majewski
2012-10-01  9:31       ` Anatolij Gustschin
2012-10-01 10:25       ` Lukasz Majewski [this message]
2012-10-01 14:12         ` Tom Rini
2012-10-02  6:58           ` Lukasz Majewski
2012-10-02  7:03             ` Kyungmin Park
2012-10-02 16:32               ` [U-Boot] [PATCH] MAINTAINERS: New maintainer for Samsung's Trats development board Lukasz Majewski
2012-10-02 22:07                 ` [U-Boot] " Tom Rini
2012-10-04  2:41                   ` Minkyu Kang

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=20121001122534.2bbff03f@amdc308.digital.local \
    --to=l.majewski@samsung.com \
    --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