From mboxrd@z Thu Jan  1 00:00:00 1970
From: Hyungwon Hwang <human.hwang@samsung.com>
Date: Thu, 11 Dec 2014 11:11:29 +0900
Subject: [U-Boot] [PATCH v11 0/3] Adds support for Exynos5422 odroid xu3
 board
In-Reply-To: <7h1to7nwbn.fsf@deeprootsystems.com>
References: <1418092779-20025-1-git-send-email-human.hwang@samsung.com>
	<7h1to8qtun.fsf@deeprootsystems.com>
	<20141210095834.2f4f9bb2@hwh-ubuntu>
	<7h1to7nwbn.fsf@deeprootsystems.com>
Message-ID: <20141211111129.612554fc@hwh-ubuntu>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Dear Kevin,

On Wed, 10 Dec 2014 11:23:08 -0800
Kevin Hilman <khilman@kernel.org> wrote:

> Hyungwon Hwang <human.hwang@samsung.com> writes:
> 
> > Dear Kevin,
> >
> > On Tue, 09 Dec 2014 15:36:00 -0800
> > Kevin Hilman <khilman@kernel.org> wrote:
> >
> >> Hyungwon Hwang <human.hwang@samsung.com> writes:
> >> 
> >> > This is v11 of the patchset adding support Odroud XU3 board.
> >> 
> >> I finally got around to testing this on top of v2015.01-rc3 on my
> >> XU3.
> >> 
> >> As I mentioned earlier, I had to enable the USB and networking
> >> options so I could dhcp/tftp but after that it works for me.
> >> 
> >> Feel free to add:
> >> 
> >> Tested-by: Kevin Hilman <khilman@linaro.org>
> >
> > Thanks for yout review. Sjoerd is waiting for his patch merged
> > (title: Exynos: Move down common USB
> > configuration). So the features related USB and networking will be
> > enabled after this patchset and his patch are merged.
> 
> OK, good.
> 
> >> 
> >> [...]
> >> 
> >> > Note: If you use micro SD card for your test you have to apply
> >> > the below patch additionally. This patch is needed, because
> >> > micro sd card is recognized as MMC1 instead of MMC0. Additional
> >> > work is needed to make it work regardless of device id.
> >> 
> >> FYI, with or without your MMC ID patch, I wasn't able to save the
> >> environment to the SD card I'm booting from:
> >> 
> >> ODROID-XU3 # saveenv
> >> Saving Environment to MMC...
> >> dwmci_send_cmd: Timeout.
> >> MMC init failed
> >> 
> >
> > Actually I just tested it again. But it works for me.
> >
> > Saving Environment to MMC...
> > Writing to MMC(1)... done
> >
> > I applied my patchset and MMC ID patch to commit
> > 38cd8c4253013ccdd4052ee021f6066fe9a52551 in
> > http://git.denx.de/u-boot-samsung.git (branch: master).
> >
> > I don't know why it does't work for you. Please feel free to need my
> > help for this, if you need.
> 
> Curious wh you're using u-boot-samsung.git and not mainline.  Can you
> test this using mainline u-boot v2015.01-rc3?

I am sorry for the previous email. It was sent mistakenly by the bug
of pressing ctrl in my multi-screen program. As I know, basically the
merge is done by Minkyu Kang who manages the u-boot-samsung, and he
send pull request to mainline. So I have done this work based on
u-boot-samsung. How about use u-boot-samsung until it is merged to
mainline, even though there is a explicit bug (You can fix it with
https://patchwork.ozlabs.org/patch/415635/)?

Also, I tested saveenv in the
mainline u-boot(http://git.denx.de/u-boot.git). But it works.

Which compiler do you use? I use gcc version 4.8.2 (Ubuntu/Linaro
4.8.2-16ubuntu4). Can you tell me what you use or test it using another
compiler?

> 
> Kevin
>