diff mbox

[U-Boot,v2] OMAP: Remove omapfb.debug=y from Beagle and Overo env settings

Message ID 1313090926-18989-1-git-send-email-agnel.joel@gmail.com
State Awaiting Upstream
Delegated to: Sandeep Paulraj
Headers show

Commit Message

Joel Fernandes Aug. 11, 2011, 7:28 p.m. UTC 
From: Steve Sakoman <steve@sakoman.com>

The kernel DSS2 code is mature now, and keeping this setting hurts performance

Signed-off-by: Steve Sakoman <steve@sakoman.com>
Signed-off-by: Joel A Fernandes <agnel.joel@gmail.com>
---
Changes since v1:
Added myself to SOB

 include/configs/omap3_beagle.h |    2 --
 include/configs/omap3_overo.h  |    2 --
 2 files changed, 0 insertions(+), 4 deletions(-)

Comments

Albert ARIBAUD Sept. 5, 2011, 7:19 p.m. UTC | #1 
Le 11/08/2011 21:28, Joel A Fernandes a écrit :
> From: Steve Sakoman<steve@sakoman.com>
>
> The kernel DSS2 code is mature now, and keeping this setting hurts performance
>
> Signed-off-by: Steve Sakoman<steve@sakoman.com>
> Signed-off-by: Joel A Fernandes<agnel.joel@gmail.com>
> ---
> Changes since v1:
> Added myself to SOB

I see no indication of this anywhere, but this change seems to be 
already in git, from a patch submitted by Steve Sakoman on december 2010:

<http://git.denx.de/?p=u-boot.git;a=commit;h=2490e591b4b5450d2b011deac7b07345a1f1daf4>

... although patchwork does not know it yet:

<http://patchwork.ozlabs.org/project/uboot/list/?q=Remove+omapfb.debug>

Amicalement,
Sandeep Paulraj Sept. 5, 2011, 8:04 p.m. UTC | #2 
> 
> Le 11/08/2011 21:28, Joel A Fernandes a écrit :
> > From: Steve Sakoman<steve@sakoman.com>
> >
> > The kernel DSS2 code is mature now, and keeping this setting hurts
> performance
> >
> > Signed-off-by: Steve Sakoman<steve@sakoman.com>
> > Signed-off-by: Joel A Fernandes<agnel.joel@gmail.com>
> > ---
> > Changes since v1:
> > Added myself to SOB
> 
> I see no indication of this anywhere, but this change seems to be
> already in git, from a patch submitted by Steve Sakoman on december 2010:
> 
> <http://git.denx.de/?p=u-
> boot.git;a=commit;h=2490e591b4b5450d2b011deac7b07345a1f1daf4>

I just submitted it as part of the TI pull request the other day

> 
> ... although patchwork does not know it yet:
> 
> <http://patchwork.ozlabs.org/project/uboot/list/?q=Remove+omapfb.debug>

I'll update.

--Sandeep
diff mbox

Patch

diff --git a/include/configs/omap3_beagle.h b/include/configs/omap3_beagle.h
index 9fd80ed..42575f1 100644
--- a/include/configs/omap3_beagle.h
+++ b/include/configs/omap3_beagle.h
@@ -214,7 +214,6 @@ 
 		"mpurate=${mpurate} " \
 		"vram=${vram} " \
 		"omapfb.mode=dvi:${dvimode} " \
-		"omapfb.debug=y " \
 		"omapdss.def_disp=${defaultdisplay} " \
 		"root=${mmcroot} " \
 		"rootfstype=${mmcrootfstype}\0" \
@@ -222,7 +221,6 @@ 
 		"mpurate=${mpurate} " \
 		"vram=${vram} " \
 		"omapfb.mode=dvi:${dvimode} " \
-		"omapfb.debug=y " \
 		"omapdss.def_disp=${defaultdisplay} " \
 		"root=${nandroot} " \
 		"rootfstype=${nandrootfstype}\0" \
diff --git a/include/configs/omap3_overo.h b/include/configs/omap3_overo.h
index 242b317..0f64054 100644
--- a/include/configs/omap3_overo.h
+++ b/include/configs/omap3_overo.h
@@ -170,7 +170,6 @@ 
 		"mpurate=${mpurate} " \
 		"vram=${vram} " \
 		"omapfb.mode=dvi:${dvimode} " \
-		"omapfb.debug=y " \
 		"omapdss.def_disp=${defaultdisplay} " \
 		"root=${mmcroot} " \
 		"rootfstype=${mmcrootfstype}\0" \
@@ -178,7 +177,6 @@ 
 		"mpurate=${mpurate} " \
 		"vram=${vram} " \
 		"omapfb.mode=dvi:${dvimode} " \
-		"omapfb.debug=y " \
 		"omapdss.def_disp=${defaultdisplay} " \
 		"root=${nandroot} " \
 		"rootfstype=${nandrootfstype}\0" \