All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ayaka <ayaka@soulik.info>
To: Rob Herring <robh@kernel.org>
Cc: "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	Mark Rutland <mark.rutland@arm.com>,
	Russell King <linux@armlinux.org.uk>,
	"kgene@kernel.org" <kgene@kernel.org>,
	krzk@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>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 2/2] ARM: dts: add TOPEET itop elite based board
Date: Wed, 24 Aug 2016 16:29:07 +0800	[thread overview]
Message-ID: <4FC4C5C4-C71B-4CC3-9A4F-E03404CF571A@soulik.info> (raw)
In-Reply-To: <CAL_JsqKY=VEZ+U11kY+XbMWuXhaFDr6GpnR3AXKd1U=9_=8fgA@mail.gmail.com>



從我的 iPad 傳送

> Rob Herring <robh@kernel.org> 於 2016年8月24日 上午6:38 寫道:
> 
>> On Tue, Aug 23, 2016 at 1:36 PM, ayaka <ayaka@soulik.info> wrote:
>> 
>>> On 08/24/2016 02:14 AM, Rob Herring wrote:
>>> 
>>>> On Mon, Aug 22, 2016 at 10:18:09PM +0800, Randy Li wrote:
>>>> 
>>>> The TOPEET itop exynos 4412 have three versions base board. The
>>>> Elite version is the cheap one without too much peripheral devices
>>>> on it.
>>>> 
>>>> Currently supported are serial console, wired networking(USB),
>>>> USB OTG in peripheral mode, USB host, SD storage, GPIO buttons,
>>>> PWM beeper, ADC and LEDs.
> 
> [...]
> 
>>>> +/dts-v1/;
>>>> +#include "exynos4412-itop-scp-core.dtsi"
>>>> +
>>>> +/ {
>>>> +       model = "TOPEET iTop 4412 Elite board based on Exynos4412";
>>>> +       compatible = "topeet,itop4412-elite", "samsung,exynos4412",
>>>> "samsung,exynos4";
>>>> +
>>>> +       chosen {
>>>> +               bootargs ="console=ttySAC2,115200";
>>> 
>>> Don't need console set with...
>> 
>> But could I set something like rootfs here ?
>> As the bootloader(old u-boot) offered by the vendor is brick, it is
>> impossible to save any something in u-boot.
> 
> Yes, but then that is a user decision and shouldn't really be there
> either. In other words, ether the bootloader should set bootargs
> (which even a vendor u-boot should be able to do either directly or
> the zImage can take it from the ATAGS)
Yes, but you have to break the auto boot every time. I want to write a new u-boot for it, but I meet some problem with JTAG, it won't come soon.
> or a user can edit the dts and
> carry that change.
> 
> However, there are probably plenty of examples in the kernel that
> counter what I just said.
I see than you. May I ask if I specific the rootfs bootargs of in dts, should you accept such patch?
> 
> Rob

WARNING: multiple messages have this Message-ID (diff)
From: Ayaka <ayaka@soulik.info>
To: Rob Herring <robh@kernel.org>
Cc: Mark Rutland <mark.rutland@arm.com>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	"linux-samsung-soc@vger.kernel.org"
	<linux-samsung-soc@vger.kernel.org>,
	Russell King <linux@armlinux.org.uk>,
	krzk@kernel.org,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"kgene@kernel.org" <kgene@kernel.org>,
	"linux-arm-kernel@lists.infradead.org"
	<linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH 2/2] ARM: dts: add TOPEET itop elite based board
Date: Wed, 24 Aug 2016 16:29:07 +0800	[thread overview]
Message-ID: <4FC4C5C4-C71B-4CC3-9A4F-E03404CF571A@soulik.info> (raw)
In-Reply-To: <CAL_JsqKY=VEZ+U11kY+XbMWuXhaFDr6GpnR3AXKd1U=9_=8fgA@mail.gmail.com>



從我的 iPad 傳送

> Rob Herring <robh@kernel.org> 於 2016年8月24日 上午6:38 寫道:
> 
>> On Tue, Aug 23, 2016 at 1:36 PM, ayaka <ayaka@soulik.info> wrote:
>> 
>>> On 08/24/2016 02:14 AM, Rob Herring wrote:
>>> 
>>>> On Mon, Aug 22, 2016 at 10:18:09PM +0800, Randy Li wrote:
>>>> 
>>>> The TOPEET itop exynos 4412 have three versions base board. The
>>>> Elite version is the cheap one without too much peripheral devices
>>>> on it.
>>>> 
>>>> Currently supported are serial console, wired networking(USB),
>>>> USB OTG in peripheral mode, USB host, SD storage, GPIO buttons,
>>>> PWM beeper, ADC and LEDs.
> 
> [...]
> 
>>>> +/dts-v1/;
>>>> +#include "exynos4412-itop-scp-core.dtsi"
>>>> +
>>>> +/ {
>>>> +       model = "TOPEET iTop 4412 Elite board based on Exynos4412";
>>>> +       compatible = "topeet,itop4412-elite", "samsung,exynos4412",
>>>> "samsung,exynos4";
>>>> +
>>>> +       chosen {
>>>> +               bootargs ="console=ttySAC2,115200";
>>> 
>>> Don't need console set with...
>> 
>> But could I set something like rootfs here ?
>> As the bootloader(old u-boot) offered by the vendor is brick, it is
>> impossible to save any something in u-boot.
> 
> Yes, but then that is a user decision and shouldn't really be there
> either. In other words, ether the bootloader should set bootargs
> (which even a vendor u-boot should be able to do either directly or
> the zImage can take it from the ATAGS)
Yes, but you have to break the auto boot every time. I want to write a new u-boot for it, but I meet some problem with JTAG, it won't come soon.
> or a user can edit the dts and
> carry that change.
> 
> However, there are probably plenty of examples in the kernel that
> counter what I just said.
I see than you. May I ask if I specific the rootfs bootargs of in dts, should you accept such patch?
> 
> Rob


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

WARNING: multiple messages have this Message-ID (diff)
From: ayaka@soulik.info (Ayaka)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 2/2] ARM: dts: add TOPEET itop elite based board
Date: Wed, 24 Aug 2016 16:29:07 +0800	[thread overview]
Message-ID: <4FC4C5C4-C71B-4CC3-9A4F-E03404CF571A@soulik.info> (raw)
In-Reply-To: <CAL_JsqKY=VEZ+U11kY+XbMWuXhaFDr6GpnR3AXKd1U=9_=8fgA@mail.gmail.com>



??? iPad ??

> Rob Herring <robh@kernel.org> ? 2016?8?24? ??6:38 ???
> 
>> On Tue, Aug 23, 2016 at 1:36 PM, ayaka <ayaka@soulik.info> wrote:
>> 
>>> On 08/24/2016 02:14 AM, Rob Herring wrote:
>>> 
>>>> On Mon, Aug 22, 2016 at 10:18:09PM +0800, Randy Li wrote:
>>>> 
>>>> The TOPEET itop exynos 4412 have three versions base board. The
>>>> Elite version is the cheap one without too much peripheral devices
>>>> on it.
>>>> 
>>>> Currently supported are serial console, wired networking(USB),
>>>> USB OTG in peripheral mode, USB host, SD storage, GPIO buttons,
>>>> PWM beeper, ADC and LEDs.
> 
> [...]
> 
>>>> +/dts-v1/;
>>>> +#include "exynos4412-itop-scp-core.dtsi"
>>>> +
>>>> +/ {
>>>> +       model = "TOPEET iTop 4412 Elite board based on Exynos4412";
>>>> +       compatible = "topeet,itop4412-elite", "samsung,exynos4412",
>>>> "samsung,exynos4";
>>>> +
>>>> +       chosen {
>>>> +               bootargs ="console=ttySAC2,115200";
>>> 
>>> Don't need console set with...
>> 
>> But could I set something like rootfs here ?
>> As the bootloader(old u-boot) offered by the vendor is brick, it is
>> impossible to save any something in u-boot.
> 
> Yes, but then that is a user decision and shouldn't really be there
> either. In other words, ether the bootloader should set bootargs
> (which even a vendor u-boot should be able to do either directly or
> the zImage can take it from the ATAGS)
Yes, but you have to break the auto boot every time. I want to write a new u-boot for it, but I meet some problem with JTAG, it won't come soon.
> or a user can edit the dts and
> carry that change.
> 
> However, there are probably plenty of examples in the kernel that
> counter what I just said.
I see than you. May I ask if I specific the rootfs bootargs of in dts, should you accept such patch?
> 
> Rob

  reply	other threads:[~2016-08-24  8:56 UTC|newest]

Thread overview: 86+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-08-21 20:04 [PATCH 0/2] Add a new board for exynos 4412 Randy Li
2016-08-21 20:04 ` Randy Li
2016-08-21 20:04 ` Randy Li
2016-08-21 20:04 ` [PATCH 1/2] ARM: dts: Add TOPEET itop core board SCP package version Randy Li
2016-08-21 20:04   ` Randy Li
2016-08-22  6:50   ` Marek Szyprowski
2016-08-22  6:50     ` Marek Szyprowski
2016-08-22  8:28     ` Ayaka
2016-08-22  8:28       ` Ayaka
2016-08-22 14:45       ` Chanwoo Choi
2016-08-22 14:45         ` Chanwoo Choi
2016-08-22 14:49         ` ayaka
2016-08-22 14:49           ` ayaka
2016-08-22 14:18     ` [PATCH 0/2 v2] Add a new board TOPEET iTOP for exynos 4412 Randy Li
2016-08-22 14:18       ` Randy Li
2016-08-22 14:18       ` Randy Li
2016-08-22 14:18       ` [PATCH 1/2] ARM: dts: Add TOPEET itop core board SCP package version Randy Li
2016-08-22 14:18         ` Randy Li
2016-08-22 14:18       ` [PATCH 2/2] ARM: dts: add TOPEET itop elite based board Randy Li
2016-08-22 14:18         ` Randy Li
2016-08-22 18:59         ` kbuild test robot
2016-08-22 18:59           ` kbuild test robot
2016-08-22 18:59           ` kbuild test robot
2016-08-23 18:14         ` Rob Herring
2016-08-23 18:14           ` Rob Herring
2016-08-23 18:36           ` ayaka
2016-08-23 18:36             ` ayaka
2016-08-23 18:36             ` ayaka
2016-08-23 22:38             ` Rob Herring
2016-08-23 22:38               ` Rob Herring
2016-08-23 22:38               ` Rob Herring
2016-08-24  8:29               ` Ayaka [this message]
2016-08-24  8:29                 ` Ayaka
2016-08-24  8:29                 ` Ayaka
2016-08-28 11:21               ` [PATCH 0/4 v3] Add a new board TOPEET iTOP for exynos 4412 Randy Li
2016-08-28 11:21                 ` Randy Li
2016-08-28 11:21                 ` Randy Li
2016-08-28 11:21                 ` [PATCH 1/4] ARM: dts: Add TOPEET itop core board SCP package version Randy Li
2016-08-28 11:21                   ` Randy Li
2016-08-28 21:13                   ` Rask Ingemann Lambertsen
2016-08-28 21:13                     ` Rask Ingemann Lambertsen
2016-08-28 21:13                     ` Rask Ingemann Lambertsen
2016-08-30  7:16                   ` Krzysztof Kozlowski
2016-08-30  7:16                     ` Krzysztof Kozlowski
2016-08-31 12:58                     ` [RESEND PATCH 1/3] " Randy Li
2016-08-31 12:58                       ` Randy Li
2016-08-31 18:42                       ` Krzysztof Kozlowski
2016-08-31 18:42                         ` Krzysztof Kozlowski
2016-08-31 18:42                         ` Krzysztof Kozlowski
2016-08-31 19:49                         ` Ayaka
2016-08-31 19:49                           ` Ayaka
2016-08-31 19:49                           ` Ayaka
2016-09-01 13:03                       ` Bartlomiej Zolnierkiewicz
2016-09-01 13:03                         ` Bartlomiej Zolnierkiewicz
2016-09-01 13:03                         ` Bartlomiej Zolnierkiewicz
2016-09-01 16:12                         ` Krzysztof Kozlowski
2016-09-01 16:12                           ` Krzysztof Kozlowski
2016-09-01 16:12                           ` Krzysztof Kozlowski
2016-08-28 11:21                 ` [PATCH 2/4] ARM: dts: add TOPEET itop elite based board Randy Li
2016-08-28 11:21                   ` Randy Li
2016-08-30  7:32                   ` Krzysztof Kozlowski
2016-08-30  7:32                     ` Krzysztof Kozlowski
2016-08-31 12:59                     ` [RESEND PATCH 2/3] " Randy Li
2016-08-31 12:59                       ` Randy Li
2016-09-01 13:07                       ` Bartlomiej Zolnierkiewicz
2016-09-01 13:07                         ` Bartlomiej Zolnierkiewicz
2016-08-28 11:21                 ` [PATCH 3/4] ARM: dts: add fimc support in exynos4412 TOPEET iTop Elite board Randy Li
2016-08-28 11:21                   ` Randy Li
2016-08-30  7:33                   ` Krzysztof Kozlowski
2016-08-30  7:33                     ` Krzysztof Kozlowski
2016-08-31 13:00                     ` [PATCH 3/3 v3] devicetree: bindings: Add vendor prefix for Topeet Randy Li
2016-08-31 13:00                       ` Randy Li
2016-08-31 13:00                       ` Randy Li
2016-08-28 11:21                 ` [PATCH 4/4] ARM: dts: samsung: add analog audio codec on TOPEET iTop 4412 Elite Randy Li
2016-08-28 11:21                   ` Randy Li
2016-08-28 11:21                   ` Randy Li
2016-08-30  7:35                   ` Krzysztof Kozlowski
2016-08-30  7:35                     ` Krzysztof Kozlowski
2016-08-30  7:04                 ` [PATCH 0/4 v3] Add a new board TOPEET iTOP for exynos 4412 Krzysztof Kozlowski
2016-08-30  7:04                   ` Krzysztof Kozlowski
2016-08-30  7:04                   ` Krzysztof Kozlowski
2016-08-21 20:04 ` [PATCH 2/2] ARM: dts: add TOPEET itop elite based board Randy Li
2016-08-21 20:04   ` Randy Li
2016-08-21 20:04   ` Randy Li
2016-08-21 20:12   ` [PATCH 2/2 v2] " Randy Li
2016-08-21 20:12     ` Randy Li

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=4FC4C5C4-C71B-4CC3-9A4F-E03404CF571A@soulik.info \
    --to=ayaka@soulik.info \
    --cc=devicetree@vger.kernel.org \
    --cc=kgene@kernel.org \
    --cc=krzk@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-samsung-soc@vger.kernel.org \
    --cc=linux@armlinux.org.uk \
    --cc=mark.rutland@arm.com \
    --cc=robh@kernel.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.