[U-Boot] [PATCH 3/3] arm: Tegra2: Add support for NVIDIA Seaboard board Signed-off-by: Tom Warren <twarren at nvidia.com>

[Tom Warren]

Replying from my Gmail account until NVIDIA IT can remove the canned disclaimer.

Sorry for the top-quote - just wanted to see if Gmail would correctly
quote previous text.

If there are no other critiques or NAKs, etc., then I'll start working
on a V2 patch with
Sandeep's changes.

Thanks,

Tom

On Wed, Jan 12, 2011 at 10:05 AM, Tom Warren <TWarren at nvidia.com> wrote:
>> -----Original Message-----
>> From: Paulraj, Sandeep [mailto:s-paulraj at ti.com]
>> Sent: Tuesday, January 11, 2011 7:00 PM
>> To: Tom Warren; Tom Warren; u-boot at lists.denx.de
>> Subject: RE: [U-Boot] [PATCH 3/3] arm: Tegra2: Add support for NVIDIA
>> Seaboard board Signed-off-by: Tom Warren <twarren at nvidia.com>
>>
>> Hi Tom,
> [Tom Warren] Hi. Sorry for top-quoting before. My comments are now in-line.
> Note that I've put in a request to my IT department to remove the 'confidential information'
> b.s. disclaimer that is appended to all my outgoing Outlook emails. If anyone knows how to get
> Gmail to use the standard ">" reply-to quoting, I can start using my Gmail account instead.
>
>>
>> We keep the U-Boot in sync with what is at
>>
>> http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-
>> 2.6.git;a=history;f=arch/arm/tools/mach-
>> types;h=2fea897ebeb12959415120a7c1cdcaf5d8ac8fee;hb=4162cf64973df51fc885825b
>> c9ca4d055891c49f
>>
>> I can see that the kernel does not have the updated name yet.
> [Tom Warren] I'm surprised, since I was told the kernel's copy had been updated.
> I'll check with the kernel guys - perhaps it only went into Android's kernel, etc.
>
>>
>> Since it is only required while booting a kernel,
>> i would just keep the old name for the time being. When the kernel has this
>> update, we will
>> update U-Boot and then you will have to send in a patch to reflect this
>> change.
>> In any case u mention that you have follow up patches. By the time u get all
>> the patches
>> accepted, the mach-types would have been updated in the kernel and in U-
>> boot.
>>
> [Tom Warren] Good idea. I'll pull out the mach-types.h change in the next (V2) patch series.
>
>> Also in one of ur patches you had updates to serial driver. You will have to
>> keep that change in a separate patch.
> [Tom Warren] Why separate? It's an integral part of adding Tegra2 support, which is what the 1st patch does.
> But if that's required, then I'll change that, too, in V2.
>>
>> Regards,
>> Sandeep
>> ________________________________________
>> From: Tom Warren [TWarren at nvidia.com]
>> Sent: Tuesday, January 11, 2011 7:43 PM
>> To: Paulraj, Sandeep; Tom Warren; u-boot at lists.denx.de
>> Subject: RE: [U-Boot] [PATCH 3/3] arm: Tegra2: Add support for NVIDIA
>> Seaboard  board Signed-off-by: Tom Warren <twarren at nvidia.com>
>>
>> Thanks, Sandeep.
>>
>> As I understand it, one of our Tegra2 kernel jockeys (Allen Martin or
>> similar) has already had the kernel version of mach-types.h updated for
>> Seaboard - I was just attempting to use the correct #define
>> (MACH_TYPE_SEABOARD - we dropped all of the MACH_TYPE_TEGRA_ names AFAIK) so
>> my patch would build/boot. Of course this define doesn't matter until the
>> kernel is booting w/the upstreamed U-Boot, which is a ways off (it boots
>> fine here w/our private version of U-Boot based on V2010-09).
>>
>> Assuming I'm correct about the Linux version of mach-types already having
>> been updated, how do I get a sync request in?  And how is that phased in to
>> match with my patches so that it all builds/boots?
>>
>> Thanks,
>>
>> Tom
>>
>> > -----Original Message-----
>> > From: Paulraj, Sandeep [mailto:s-paulraj at ti.com]
>> > Sent: Tuesday, January 11, 2011 5:16 PM
>> > To: Tom Warren; u-boot at lists.denx.de
>> > Cc: Tom Warren
>> > Subject: RE: [U-Boot] [PATCH 3/3] arm: Tegra2: Add support for NVIDIA
>> > Seaboard board Signed-off-by: Tom Warren <twarren at nvidia.com>
>> >
>> > Hi Tom,
>> >
>> > Nice to see support for this board
>> >
>> > > #define MACH_TYPE_TI8148EVM            3004
>> > >#define MACH_TYPE_TEGRA_SEABOARD       3005
>> > >+#define MACH_TYPE_SEABOARD             3005
>> >
>> >
>> > I don't think this is the way it is supposed to be done.
>> > Patches should not contain an addition to mach-types.h.
>> > What is generally done is you ask for an ARM sync request and in the past
>> > Tom Rix and I have updated mach-types.h.
>> >
>> > I did this some weeks ago.
>> >
>> > Also i think we cannot have 2 EVMs with same MACH_TYPE_
>> > You should probably go to ARM Linux website and register as a separate
>> EVM.
>> > The Number you get from there should be used. I beleive you can also
>> change
>> > the name of the
>> > board there; assu,ing you intend to use MACH_TYPE_SEABOARD  instead of
>> > MACH_TYPE_TEGRA_SEABOARD.
>> >
>> > It usually takes a little bit of time for the mach-types to be updated in
>> > the
>> > linux kernel. Once that is done we sync up with the kernel.
>> >
>> > Regards,
>> > Sandeep
>> ----------------------------------------------------------------------------
>> -------
>> This email message is for the sole use of the intended recipient(s) and may
>> contain
>> confidential information.  Any unauthorized review, use, disclosure or
>> distribution
>> is prohibited.  If you are not the intended recipient, please contact the
>> sender by
>> reply email and destroy all copies of the original message.
>> ----------------------------------------------------------------------------
>> -------
>