From: Antony Pavlov <antonynpavlov@gmail.com>
To: Lucas Stach <dev@lynxeye.de>
Cc: barebox@lists.infradead.org
Subject: Re: [PATCH 1/7] tegra: move address map to tegra20-silicon.h
Date: Mon, 4 Mar 2013 23:26:20 +0400 [thread overview]
Message-ID: <CAA4bVAG9rkedNQmhT2EWMSSNG4Zc1s-6q+OYBe-AmpcuOoHfnA@mail.gmail.com> (raw)
In-Reply-To: <1362266753.2127.14.camel@antimon>
On 3 March 2013 03:25, Lucas Stach <dev@lynxeye.de> wrote:
> Am Freitag, den 01.03.2013, 14:17 +0100 schrieb Jean-Christophe
> PLAGNIOL-VILLARD:
>> On 10:22 Fri 01 Mar , Lucas Stach wrote:
>> > The address map is specific to the Tegra20 SoC. Move it to a
>> > file with an appropriate name. While at it clarify and fix
>> > the definitions.
>>
>> folloow the kernel
>
> The Tegra kernel is fully device tree enabled and doesn't need any of
> those defines, as the hardware mapping is encoded in the DT. While I'm
> certainly aiming for the same thing in barebox, it's just not the right
> thing to do at this state of development.
Is there any benefit in the renaming of the iomap.h file to tegra20-silicon.h?
I think no. The name iomap.h is used in the linux kernel so it is
natural that just the same file will have the same name it barebox.
It is more comfortable for kernel developer see the kernel header
files in bootloader with the same kernel name.
Anyway the number of kernel Tegra developers is much more than number
of barebox Tegra developers :)))
>> >
>> > Signed-off-by: Lucas Stach <dev@lynxeye.de>
>> > ---
>> > [...]
>
>
> _______________________________________________
> barebox mailing list
> barebox@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/barebox
--
Best regards,
Antony Pavlov
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
next prev parent reply other threads:[~2013-03-04 19:32 UTC|newest]
Thread overview: 41+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-03-01 9:22 [PATCH 0/7] Rework current Tegra support Lucas Stach
2013-03-01 9:22 ` [PATCH 1/7] tegra: move address map to tegra20-silicon.h Lucas Stach
2013-03-01 13:17 ` Jean-Christophe PLAGNIOL-VILLARD
2013-03-02 23:25 ` Lucas Stach
2013-03-04 19:26 ` Antony Pavlov [this message]
2013-03-01 9:22 ` [PATCH 2/7] tegra: remove debug_ll Lucas Stach
2013-03-01 13:17 ` Jean-Christophe PLAGNIOL-VILLARD
2013-03-01 17:19 ` Sascha Hauer
2013-03-01 9:22 ` [PATCH 3/7] tegra: switch to ARMv7 cpu type Lucas Stach
2013-03-01 9:22 ` [PATCH 4/7] tegra: add blank tegra20 platform init Lucas Stach
2013-03-01 13:16 ` Jean-Christophe PLAGNIOL-VILLARD
2013-03-01 9:22 ` [PATCH 5/7] tegra: add driver for the clock and reset module Lucas Stach
2013-03-01 17:26 ` Sascha Hauer
2013-03-02 23:16 ` Lucas Stach
2013-03-01 9:22 ` [PATCH 6/7] tegra: add proper timer driver Lucas Stach
2013-03-01 13:14 ` Jean-Christophe PLAGNIOL-VILLARD
2013-03-01 17:23 ` Sascha Hauer
2013-03-02 23:13 ` Lucas Stach
2013-03-03 7:07 ` Antony Pavlov
2013-03-04 17:09 ` Lucas Stach
2013-03-04 19:14 ` Antony Pavlov
2013-03-01 9:22 ` [PATCH 7/7] tegra: add power management controller driver Lucas Stach
2013-03-01 13:15 ` Jean-Christophe PLAGNIOL-VILLARD
2013-03-01 17:28 ` Sascha Hauer
2013-03-01 18:00 ` Antony Pavlov
2013-03-02 23:21 ` Lucas Stach
2013-03-07 10:33 ` [PATCH v2 0/5] Rework current Tegra support Lucas Stach
2013-03-07 10:33 ` [PATCH v2 1/5] tegra: pull in iomap.h from the Linux kernel Lucas Stach
2013-03-07 10:33 ` [PATCH v2 2/5] tegra: switch to ARMv7 cpu type Lucas Stach
2013-03-08 6:03 ` Antony Pavlov
2013-03-08 13:17 ` Lucas Stach
2013-03-08 16:56 ` Antony Pavlov
2013-03-08 14:32 ` Sascha Hauer
2013-03-08 17:15 ` Antony Pavlov
2013-03-07 10:33 ` [PATCH v2 3/5] tegra: add driver for the clock and reset module Lucas Stach
2013-03-10 7:41 ` Antony Pavlov
2013-03-10 9:53 ` Sascha Hauer
2013-03-07 10:33 ` [PATCH v2 4/5] tegra: add proper timer driver Lucas Stach
2013-03-10 7:12 ` Antony Pavlov
2013-03-07 10:33 ` [PATCH v2 5/5] tegra: add power management controller driver Lucas Stach
2013-03-10 8:19 ` Antony Pavlov
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=CAA4bVAG9rkedNQmhT2EWMSSNG4Zc1s-6q+OYBe-AmpcuOoHfnA@mail.gmail.com \
--to=antonynpavlov@gmail.com \
--cc=barebox@lists.infradead.org \
--cc=dev@lynxeye.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