From: Kevin Hilman <khilman@linaro.org>
To: Olof Johansson <olof@lixom.net>
Cc: Tushar Behera <tushar.behera@linaro.org>,
Arnd Bergmann <arnd@arndb.de>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>,
"linux-samsung-soc@vger.kernel.org"
<linux-samsung-soc@vger.kernel.org>,
Kukjin Kim <kgene.kim@samsung.com>,
Doug Anderson <dianders@chromium.org>,
Patch Tracking <patches@linaro.org>
Subject: Re: [PATCH] ARM: EXYNOS: Update defconfig for Arndale and Origen board
Date: Wed, 29 May 2013 11:23:52 -0700 [thread overview]
Message-ID: <87sj1563wn.fsf@linaro.org> (raw)
In-Reply-To: <CAOesGMh50Dzr_jczHZC5_txfW=6a+7EgFKvgZgj9MhUcVUTJ1Q@mail.gmail.com> (Olof Johansson's message of "Tue, 28 May 2013 22:31:41 -0700")
Olof Johansson <olof@lixom.net> writes:
> On Tue, May 28, 2013 at 8:59 PM, Tushar Behera <tushar.behera@linaro.org> wrote:
>
>> The patches are at [1].
>
> FWIW, a cgit/gitweb link is easier to follow when you're reading an
> email. Anyway, found the patches.
>
>> There are a total of 6 patches on top of
>> v3.10-rc3. 3 of them (a, b, d) are queued for 3.10-rc4 and another patch
>> (c) is just a defconfig hack to get Arndale booting. Remaining 2 patches
>> (e, f) are required to reset the hub during EHCI initialization.
>
> Huh, I thought you said that (c) wasn't needed when I posted the
> defconfig update. It'd be nice to see the code fixed to handle this
> case instead of Linaro carrying a patch like this though. I.e. make it
> able to deselect the uart, or make it tied to DEBUG_LL like on other
> platforms instead of having a separate config for this.
I agree, that defconfig hack is awful to have to carry.
I also noticed that correct setting of CONFIG_S3C_LOWLEVEL_UART_PORT is
needed even without DEBUG_LL which seems broken.
Kevin
WARNING: multiple messages have this Message-ID (diff)
From: khilman@linaro.org (Kevin Hilman)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] ARM: EXYNOS: Update defconfig for Arndale and Origen board
Date: Wed, 29 May 2013 11:23:52 -0700 [thread overview]
Message-ID: <87sj1563wn.fsf@linaro.org> (raw)
In-Reply-To: <CAOesGMh50Dzr_jczHZC5_txfW=6a+7EgFKvgZgj9MhUcVUTJ1Q@mail.gmail.com> (Olof Johansson's message of "Tue, 28 May 2013 22:31:41 -0700")
Olof Johansson <olof@lixom.net> writes:
> On Tue, May 28, 2013 at 8:59 PM, Tushar Behera <tushar.behera@linaro.org> wrote:
>
>> The patches are at [1].
>
> FWIW, a cgit/gitweb link is easier to follow when you're reading an
> email. Anyway, found the patches.
>
>> There are a total of 6 patches on top of
>> v3.10-rc3. 3 of them (a, b, d) are queued for 3.10-rc4 and another patch
>> (c) is just a defconfig hack to get Arndale booting. Remaining 2 patches
>> (e, f) are required to reset the hub during EHCI initialization.
>
> Huh, I thought you said that (c) wasn't needed when I posted the
> defconfig update. It'd be nice to see the code fixed to handle this
> case instead of Linaro carrying a patch like this though. I.e. make it
> able to deselect the uart, or make it tied to DEBUG_LL like on other
> platforms instead of having a separate config for this.
I agree, that defconfig hack is awful to have to carry.
I also noticed that correct setting of CONFIG_S3C_LOWLEVEL_UART_PORT is
needed even without DEBUG_LL which seems broken.
Kevin
WARNING: multiple messages have this Message-ID (diff)
From: Kevin Hilman <khilman@linaro.org>
To: Olof Johansson <olof@lixom.net>
Cc: Tushar Behera <tushar.behera@linaro.org>,
Arnd Bergmann <arnd@arndb.de>,
"linux-kernel\@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"linux-arm-kernel\@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>,
"linux-samsung-soc\@vger.kernel.org"
<linux-samsung-soc@vger.kernel.org>,
Kukjin Kim <kgene.kim@samsung.com>,
Doug Anderson <dianders@chromium.org>,
Patch Tracking <patches@linaro.org>
Subject: Re: [PATCH] ARM: EXYNOS: Update defconfig for Arndale and Origen board
Date: Wed, 29 May 2013 11:23:52 -0700 [thread overview]
Message-ID: <87sj1563wn.fsf@linaro.org> (raw)
In-Reply-To: <CAOesGMh50Dzr_jczHZC5_txfW=6a+7EgFKvgZgj9MhUcVUTJ1Q@mail.gmail.com> (Olof Johansson's message of "Tue, 28 May 2013 22:31:41 -0700")
Olof Johansson <olof@lixom.net> writes:
> On Tue, May 28, 2013 at 8:59 PM, Tushar Behera <tushar.behera@linaro.org> wrote:
>
>> The patches are at [1].
>
> FWIW, a cgit/gitweb link is easier to follow when you're reading an
> email. Anyway, found the patches.
>
>> There are a total of 6 patches on top of
>> v3.10-rc3. 3 of them (a, b, d) are queued for 3.10-rc4 and another patch
>> (c) is just a defconfig hack to get Arndale booting. Remaining 2 patches
>> (e, f) are required to reset the hub during EHCI initialization.
>
> Huh, I thought you said that (c) wasn't needed when I posted the
> defconfig update. It'd be nice to see the code fixed to handle this
> case instead of Linaro carrying a patch like this though. I.e. make it
> able to deselect the uart, or make it tied to DEBUG_LL like on other
> platforms instead of having a separate config for this.
I agree, that defconfig hack is awful to have to carry.
I also noticed that correct setting of CONFIG_S3C_LOWLEVEL_UART_PORT is
needed even without DEBUG_LL which seems broken.
Kevin
next prev parent reply other threads:[~2013-05-29 18:23 UTC|newest]
Thread overview: 42+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-05-10 6:49 [PATCH] ARM: EXYNOS: Update defconfig for Arndale and Origen board Tushar Behera
2013-05-10 6:49 ` Tushar Behera
2013-05-10 15:31 ` Doug Anderson
2013-05-10 15:31 ` Doug Anderson
2013-05-14 0:06 ` Kevin Hilman
2013-05-14 0:06 ` Kevin Hilman
2013-05-14 3:07 ` Tushar Behera
2013-05-14 3:07 ` Tushar Behera
2013-05-28 23:17 ` Olof Johansson
2013-05-28 23:17 ` Olof Johansson
2013-05-29 3:59 ` Tushar Behera
2013-05-29 3:59 ` Tushar Behera
2013-05-29 5:31 ` Olof Johansson
2013-05-29 5:31 ` Olof Johansson
2013-05-29 8:50 ` Tushar Behera
2013-05-29 8:50 ` Tushar Behera
2013-05-29 9:33 ` Jingoo Han
2013-05-29 9:33 ` Jingoo Han
2013-05-29 18:46 ` Olof Johansson
2013-05-29 18:46 ` Olof Johansson
2013-05-30 3:37 ` Tushar Behera
2013-05-30 3:37 ` Tushar Behera
2013-05-30 7:05 ` Jingoo Han
2013-05-30 7:05 ` Jingoo Han
2013-05-30 13:10 ` Vivek Gautam
2013-05-30 13:10 ` Vivek Gautam
2013-05-30 13:10 ` Vivek Gautam
[not found] ` <CAHbNUh0kWxqDMvLwE-_PKQVassDJvbgiCiDDNSHn9AW3md=A5A@mail.gmail.com>
2013-05-30 15:28 ` Tushar Behera
2013-05-30 15:28 ` Tushar Behera
2013-05-29 12:03 ` Tushar Behera
2013-05-29 12:03 ` Tushar Behera
2013-05-29 18:23 ` Kevin Hilman [this message]
2013-05-29 18:23 ` Kevin Hilman
2013-05-29 18:23 ` Kevin Hilman
2013-06-04 12:13 ` Mark Brown
2013-06-04 12:13 ` Mark Brown
2013-05-29 18:34 ` Kevin Hilman
2013-05-29 18:34 ` Kevin Hilman
2013-05-28 23:24 ` Olof Johansson
2013-05-28 23:24 ` Olof Johansson
2013-05-29 0:07 ` Kukjin Kim
2013-05-29 0:07 ` Kukjin Kim
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=87sj1563wn.fsf@linaro.org \
--to=khilman@linaro.org \
--cc=arnd@arndb.de \
--cc=dianders@chromium.org \
--cc=kgene.kim@samsung.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-samsung-soc@vger.kernel.org \
--cc=olof@lixom.net \
--cc=patches@linaro.org \
--cc=tushar.behera@linaro.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.