Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 1/1] ti-gfx.mk typo corrected
@ 2013-09-02 15:18 Martin Creutziger
  2013-09-02 15:33 ` Thomas Petazzoni
  2013-09-02 15:42 ` Peter Korsgaard
  0 siblings, 2 replies; 5+ messages in thread
From: Martin Creutziger @ 2013-09-02 15:18 UTC (permalink / raw)
  To: buildroot

There is a simple typo in package/ti-gfx/ti-gfx.mk, which the following 
patch corrects:

Signed-off-by: Martin Creutziger <buildroot@martin.creutziger.net>
---
Index: package/ti-gfx/ti-gfx.mk
===================================================================
--- package/ti-gfx/ti-gfx.mk	(revision 14043)
+++ package/ti-gfx/ti-gfx.mk	(working copy)
@@ -29,7 +29,7 @@
  TI_GFX_PLATFORM = omap3630
  endif
  ifeq ($(BR2_PACKAGE_TI_GFX_ES6),y)
-TI_GFX_OMPAES = 6.x
+TI_GFX_OMAPES = 6.x
  TI_GFX_PLATFORM = ti81xx
  endif
  ifeq ($(BR2_PACKAGE_TI_GFX_ES8),y)
--

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [Buildroot] [PATCH 1/1] ti-gfx.mk typo corrected
  2013-09-02 15:18 [Buildroot] [PATCH 1/1] ti-gfx.mk typo corrected Martin Creutziger
@ 2013-09-02 15:33 ` Thomas Petazzoni
  2013-09-02 15:42 ` Peter Korsgaard
  1 sibling, 0 replies; 5+ messages in thread
From: Thomas Petazzoni @ 2013-09-02 15:33 UTC (permalink / raw)
  To: buildroot

Dear Martin Creutziger,

On Mon, 02 Sep 2013 17:18:48 +0200, Martin Creutziger wrote:
> There is a simple typo in package/ti-gfx/ti-gfx.mk, which the following 
> patch corrects:
> 
> Signed-off-by: Martin Creutziger <buildroot@martin.creutziger.net>

Indeed, thanks!

Acked-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>

Martin, does this mean you are successfully using the ti-gfx package?
If so, can you tell us a bit more (which hardware platform, what
software stack on top of OpenGL, etc.) ? Our Google Summer of Code
student Spenser Gilliland is still working on improving OpenGL support
in Buildroot, so we're definitely interested in getting feedback.

Thanks a lot!

Thomas
-- 
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [Buildroot] [PATCH 1/1] ti-gfx.mk typo corrected
  2013-09-02 15:18 [Buildroot] [PATCH 1/1] ti-gfx.mk typo corrected Martin Creutziger
  2013-09-02 15:33 ` Thomas Petazzoni
@ 2013-09-02 15:42 ` Peter Korsgaard
  2013-09-03  9:21   ` Martin Creutziger
  1 sibling, 1 reply; 5+ messages in thread
From: Peter Korsgaard @ 2013-09-02 15:42 UTC (permalink / raw)
  To: buildroot

>>>>> "Martin" == Martin Creutziger <buildroot@martin.creutziger.net> writes:

 Martin> There is a simple typo in package/ti-gfx/ti-gfx.mk, which the
 Martin> following patch corrects:

 Martin> Signed-off-by: Martin Creutziger <buildroot@martin.creutziger.net>
 Martin> ---
 Martin> Index: package/ti-gfx/ti-gfx.mk
 Martin> ===================================================================
 Martin> --- package/ti-gfx/ti-gfx.mk	(revision 14043)
 Martin> +++ package/ti-gfx/ti-gfx.mk	(working copy)

Committed, thanks.

In the future, please consider creating the patch using git so it can be
directly committed with git am (or atleast add one more directory level,
as git am expects patch -p1 format).

-- 
Bye, Peter Korsgaard

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [Buildroot] [PATCH 1/1] ti-gfx.mk typo corrected
  2013-09-02 15:42 ` Peter Korsgaard
@ 2013-09-03  9:21   ` Martin Creutziger
  2013-09-03 12:01     ` Peter Korsgaard
  0 siblings, 1 reply; 5+ messages in thread
From: Martin Creutziger @ 2013-09-03  9:21 UTC (permalink / raw)
  To: buildroot

Peter Korsgaard wrote:
>>>>>> "Martin" == Martin Creutziger <buildroot@martin.creutziger.net> writes:
>
>   Martin> There is a simple typo in package/ti-gfx/ti-gfx.mk, which the
>   Martin> following patch corrects:
>
>   Martin> Signed-off-by: Martin Creutziger <buildroot@martin.creutziger.net>
>   Martin> ---
>   Martin> Index: package/ti-gfx/ti-gfx.mk
>   Martin> ===================================================================
>   Martin> --- package/ti-gfx/ti-gfx.mk	(revision 14043)
>   Martin> +++ package/ti-gfx/ti-gfx.mk	(working copy)
>
> Committed, thanks.
>
> In the future, please consider creating the patch using git so it can be
> directly committed with git am (or atleast add one more directory level,
> as git am expects patch -p1 format).
>

Hello Peter,

as this was my first patch here on the mailing list, I already expected 
that I won't get it 100% right ;)

I created it directly out of our SVN repository, but thanks for the 
pointers about the expected format - I'll try to get closer to it next 
time. :)

BR,
Martin

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [Buildroot] [PATCH 1/1] ti-gfx.mk typo corrected
  2013-09-03  9:21   ` Martin Creutziger
@ 2013-09-03 12:01     ` Peter Korsgaard
  0 siblings, 0 replies; 5+ messages in thread
From: Peter Korsgaard @ 2013-09-03 12:01 UTC (permalink / raw)
  To: buildroot

>>>>> "Martin" == Martin Creutziger <buildroot@martin.creutziger.net> writes:

Hi Martin,

 Martin> as this was my first patch here on the mailing list, I already
 Martin> expected that I won't get it 100% right ;)

 Martin> I created it directly out of our SVN repository, but thanks for the
 Martin> pointers about the expected format - I'll try to get closer to it next
 Martin> time. :)

Don't worry, and thanks for submitting the fix!

-- 
Bye, Peter Korsgaard

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2013-09-03 12:01 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-09-02 15:18 [Buildroot] [PATCH 1/1] ti-gfx.mk typo corrected Martin Creutziger
2013-09-02 15:33 ` Thomas Petazzoni
2013-09-02 15:42 ` Peter Korsgaard
2013-09-03  9:21   ` Martin Creutziger
2013-09-03 12:01     ` Peter Korsgaard

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox