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

[Paulraj, Sandeep]

Hi Tom,

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=4162cf64973df51fc885825bc9ca4d055891c49f

I can see that the kernel does not have the updated name yet.

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.

Also in one of ur patches you had updates to serial driver. You will have to keep that
change in a separte patch.

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.
-----------------------------------------------------------------------------------