* [U-Boot] [PATCH] tegra: fdt: add back missing host1x node
@ 2013-01-25 18:46 Allen Martin
2013-01-26 0:51 ` Allen Martin
0 siblings, 1 reply; 6+ messages in thread
From: Allen Martin @ 2013-01-25 18:46 UTC (permalink / raw)
To: u-boot
Add back host1x node to seaboard dts file. This got dropped during
the tegra fdt sort.
Signed-off-by: Allen Martin <amartin@nvidia.com>
---
board/nvidia/dts/tegra20-seaboard.dts | 11 +++++++++++
1 file changed, 11 insertions(+)
diff --git a/board/nvidia/dts/tegra20-seaboard.dts b/board/nvidia/dts/tegra20-seaboard.dts
index 9cb9b5b..527a296 100644
--- a/board/nvidia/dts/tegra20-seaboard.dts
+++ b/board/nvidia/dts/tegra20-seaboard.dts
@@ -27,6 +27,17 @@
reg = < 0x00000000 0x40000000 >;
};
+ host1x {
+ status = "okay";
+ dc at 54200000 {
+ status = "okay";
+ rgb {
+ status = "okay";
+ nvidia,panel = <&lcd_panel>;
+ };
+ };
+ };
+
/* This is not used in U-Boot, but is expected to be in kernel .dts */
i2c at 7000d000 {
clock-frequency = <100000>;
--
1.7.10.4
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] tegra: fdt: add back missing host1x node
2013-01-25 18:46 [U-Boot] [PATCH] tegra: fdt: add back missing host1x node Allen Martin
@ 2013-01-26 0:51 ` Allen Martin
2013-01-26 10:57 ` Albert ARIBAUD
0 siblings, 1 reply; 6+ messages in thread
From: Allen Martin @ 2013-01-26 0:51 UTC (permalink / raw)
To: u-boot
On Fri, Jan 25, 2013 at 10:46:47AM -0800, Allen Martin wrote:
> Add back host1x node to seaboard dts file. This got dropped during
> the tegra fdt sort.
>
> Signed-off-by: Allen Martin <amartin@nvidia.com>
> ---
Hi Albert, would it be possible for you to apply this directly to your
u-boot-arm repository? It fixes a regression introduced by my
previous patch:
b7723f3 tegra: fdt: sort dts files
So I wanted to make sure it gets applied before that previous patch
makes it to u-boot/master, and Tom's not ready to do another tegra
pull request yet.
-Allen
> board/nvidia/dts/tegra20-seaboard.dts | 11 +++++++++++
> 1 file changed, 11 insertions(+)
>
> diff --git a/board/nvidia/dts/tegra20-seaboard.dts b/board/nvidia/dts/tegra20-seaboard.dts
> index 9cb9b5b..527a296 100644
> --- a/board/nvidia/dts/tegra20-seaboard.dts
> +++ b/board/nvidia/dts/tegra20-seaboard.dts
> @@ -27,6 +27,17 @@
> reg = < 0x00000000 0x40000000 >;
> };
>
> + host1x {
> + status = "okay";
> + dc at 54200000 {
> + status = "okay";
> + rgb {
> + status = "okay";
> + nvidia,panel = <&lcd_panel>;
> + };
> + };
> + };
> +
> /* This is not used in U-Boot, but is expected to be in kernel .dts */
> i2c at 7000d000 {
> clock-frequency = <100000>;
> --
> 1.7.10.4
>
--
nvpublic
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] tegra: fdt: add back missing host1x node
2013-01-26 0:51 ` Allen Martin
@ 2013-01-26 10:57 ` Albert ARIBAUD
2013-01-28 16:35 ` Tom Warren
0 siblings, 1 reply; 6+ messages in thread
From: Albert ARIBAUD @ 2013-01-26 10:57 UTC (permalink / raw)
To: u-boot
Hi Allen,
On Fri, 25 Jan 2013 16:51:09 -0800, Allen Martin <amartin@nvidia.com>
wrote:
> Hi Albert, would it be possible for you to apply this directly to your
> u-boot-arm repository? It fixes a regression introduced by my
> previous patch:
>
> b7723f3 tegra: fdt: sort dts files
>
> So I wanted to make sure it gets applied before that previous patch
> makes it to u-boot/master, and Tom's not ready to do another tegra
> pull request yet.
>
> -Allen
Actually, I did not take Tom's PR as it caused some warnings. Maybe
your fix can get in the TI tree before a fixed PR is sent out?
Amicalement,
--
Albert.
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] tegra: fdt: add back missing host1x node
2013-01-26 10:57 ` Albert ARIBAUD
@ 2013-01-28 16:35 ` Tom Warren
2013-01-28 18:14 ` Albert ARIBAUD
0 siblings, 1 reply; 6+ messages in thread
From: Tom Warren @ 2013-01-28 16:35 UTC (permalink / raw)
To: u-boot
Albert,
> -----Original Message-----
> From: Albert ARIBAUD [mailto:albert.u.boot at aribaud.net]
> Sent: Saturday, January 26, 2013 3:57 AM
> To: Allen Martin
> Cc: Tom Warren; swarren at wwwdotorg.org; u-boot at lists.denx.de
> Subject: Re: [PATCH] tegra: fdt: add back missing host1x node
>
> Hi Allen,
>
> On Fri, 25 Jan 2013 16:51:09 -0800, Allen Martin <amartin@nvidia.com>
> wrote:
>
> > Hi Albert, would it be possible for you to apply this directly to your
> > u-boot-arm repository? It fixes a regression introduced by my
> > previous patch:
> >
> > b7723f3 tegra: fdt: sort dts files
> >
> > So I wanted to make sure it gets applied before that previous patch
> > makes it to u-boot/master, and Tom's not ready to do another tegra
> > pull request yet.
> >
> > -Allen
>
> Actually, I did not take Tom's PR as it caused some warnings. Maybe your fix
> can get in the TI tree before a fixed PR is sent out?
Have you got the right Tom? This is for the Tegra tree (u-boot-tegra), not TI, Tom Warren, not Tom Rini.
My PR *has* been taken into the ARM/master branch, AFAICT, and I never release PRs with warnings or errors that aren't already pre-existing in other ARM builds (VCMA9 and smdk2410, for instance).
Allen's fix is for a Tegra DT file, not for any TI board/SoC/etc.
Thanks,
Tom
>
> Amicalement,
> --
> Albert.
--
Nvpublic
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] tegra: fdt: add back missing host1x node
2013-01-28 16:35 ` Tom Warren
@ 2013-01-28 18:14 ` Albert ARIBAUD
2013-01-28 22:51 ` Allen Martin
0 siblings, 1 reply; 6+ messages in thread
From: Albert ARIBAUD @ 2013-01-28 18:14 UTC (permalink / raw)
To: u-boot
Hi Tom,
On Mon, 28 Jan 2013 08:35:25 -0800, Tom Warren <TWarren@nvidia.com>
wrote:
> Albert,
>
> > -----Original Message-----
> > From: Albert ARIBAUD [mailto:albert.u.boot at aribaud.net]
> > Sent: Saturday, January 26, 2013 3:57 AM
> > To: Allen Martin
> > Cc: Tom Warren; swarren at wwwdotorg.org; u-boot at lists.denx.de
> > Subject: Re: [PATCH] tegra: fdt: add back missing host1x node
> >
> > Hi Allen,
> >
> > On Fri, 25 Jan 2013 16:51:09 -0800, Allen Martin <amartin@nvidia.com>
> > wrote:
> >
> > > Hi Albert, would it be possible for you to apply this directly to your
> > > u-boot-arm repository? It fixes a regression introduced by my
> > > previous patch:
> > >
> > > b7723f3 tegra: fdt: sort dts files
> > >
> > > So I wanted to make sure it gets applied before that previous patch
> > > makes it to u-boot/master, and Tom's not ready to do another tegra
> > > pull request yet.
> > >
> > > -Allen
> >
> > Actually, I did not take Tom's PR as it caused some warnings. Maybe your fix
> > can get in the TI tree before a fixed PR is sent out?
>
> Have you got the right Tom? This is for the Tegra tree (u-boot-tegra), not TI, Tom Warren, not Tom Rini.
>
> My PR *has* been taken into the ARM/master branch, AFAICT, and I never release PRs with warnings or errors that aren't already pre-existing in other ARM builds (VCMA9 and smdk2410, for instance).
>
> Allen's fix is for a Tegra DT file, not for any TI board/SoC/etc.
My bad -- I've indeed mixed up Toms here.
> Thanks,
>
> Tom
Amicalement,
--
Albert.
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] tegra: fdt: add back missing host1x node
2013-01-28 18:14 ` Albert ARIBAUD
@ 2013-01-28 22:51 ` Allen Martin
0 siblings, 0 replies; 6+ messages in thread
From: Allen Martin @ 2013-01-28 22:51 UTC (permalink / raw)
To: u-boot
> > Have you got the right Tom? This is for the Tegra tree (u-boot-tegra), not TI, Tom Warren, not Tom Rini.
> >
> > My PR *has* been taken into the ARM/master branch, AFAICT, and I never release PRs with warnings or errors that aren't already pre-existing in other ARM builds (VCMA9 and smdk2410, for instance).
> >
> > Allen's fix is for a Tegra DT file, not for any TI board/SoC/etc.
>
> My bad -- I've indeed mixed up Toms here.
>
No worries, so would it be ok to take this patch in through the arm
tree so the regression doesn't make it up into Wolfgang's tree?
-Allen
--
nvpublic
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2013-01-28 22:51 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-01-25 18:46 [U-Boot] [PATCH] tegra: fdt: add back missing host1x node Allen Martin
2013-01-26 0:51 ` Allen Martin
2013-01-26 10:57 ` Albert ARIBAUD
2013-01-28 16:35 ` Tom Warren
2013-01-28 18:14 ` Albert ARIBAUD
2013-01-28 22:51 ` Allen Martin
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox