All of lore.kernel.org
 help / color / mirror / Atom feed
* [U-Boot] [PATCH] edid: rename struct member to fix two EDID_* macros
@ 2013-08-07  7:22 Christian Gmeiner
  2013-08-12  6:14 ` Christian Gmeiner
  2013-08-12 21:38 ` Anatolij Gustschin
  0 siblings, 2 replies; 4+ messages in thread
From: Christian Gmeiner @ 2013-08-07  7:22 UTC (permalink / raw)
  To: u-boot

Without this change EDID_DETAILED_TIMING_VSYNC_OFFSET
and EDID_DETAILED_TIMING_VSYNC_PULSE_WIDTH macros can
not be used (compile error).
The fix is quite trivial: rename struct member to the
expected name.

Signed-off-by: Christian Gmeiner <christian.gmeiner@gmail.com>
---
 include/edid.h |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/edid.h b/include/edid.h
index f153091..480a773 100644
--- a/include/edid.h
+++ b/include/edid.h
@@ -54,7 +54,7 @@ struct edid_detailed_timing {
 	 (_x).vertical_blanking)
 	unsigned char hsync_offset;
 	unsigned char hsync_pulse_width;
-	unsigned char sync_offset_pulse_width;
+	unsigned char vsync_offset_pulse_width;
 	unsigned char hsync_vsync_offset_pulse_width_hi;
 #define EDID_DETAILED_TIMING_HSYNC_OFFSET(_x) \
 	((GET_BITS((_x).hsync_vsync_offset_pulse_width_hi, 7, 6) << 8) + \
-- 
1.7.10.4

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

* [U-Boot] [PATCH] edid: rename struct member to fix two EDID_* macros
  2013-08-07  7:22 [U-Boot] [PATCH] edid: rename struct member to fix two EDID_* macros Christian Gmeiner
@ 2013-08-12  6:14 ` Christian Gmeiner
  2013-08-12 21:38 ` Anatolij Gustschin
  1 sibling, 0 replies; 4+ messages in thread
From: Christian Gmeiner @ 2013-08-12  6:14 UTC (permalink / raw)
  To: u-boot

2013/8/7 Christian Gmeiner <christian.gmeiner@gmail.com>:
> Without this change EDID_DETAILED_TIMING_VSYNC_OFFSET
> and EDID_DETAILED_TIMING_VSYNC_PULSE_WIDTH macros can
> not be used (compile error).
> The fix is quite trivial: rename struct member to the
> expected name.
>
> Signed-off-by: Christian Gmeiner <christian.gmeiner@gmail.com>
> ---
>  include/edid.h |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/include/edid.h b/include/edid.h
> index f153091..480a773 100644
> --- a/include/edid.h
> +++ b/include/edid.h
> @@ -54,7 +54,7 @@ struct edid_detailed_timing {
>          (_x).vertical_blanking)
>         unsigned char hsync_offset;
>         unsigned char hsync_pulse_width;
> -       unsigned char sync_offset_pulse_width;
> +       unsigned char vsync_offset_pulse_width;
>         unsigned char hsync_vsync_offset_pulse_width_hi;
>  #define EDID_DETAILED_TIMING_HSYNC_OFFSET(_x) \
>         ((GET_BITS((_x).hsync_vsync_offset_pulse_width_hi, 7, 6) << 8) + \
> --
> 1.7.10.4
>

ping
--
Christian Gmeiner, MSc

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

* [U-Boot] [PATCH] edid: rename struct member to fix two EDID_* macros
  2013-08-07  7:22 [U-Boot] [PATCH] edid: rename struct member to fix two EDID_* macros Christian Gmeiner
  2013-08-12  6:14 ` Christian Gmeiner
@ 2013-08-12 21:38 ` Anatolij Gustschin
  1 sibling, 0 replies; 4+ messages in thread
From: Anatolij Gustschin @ 2013-08-12 21:38 UTC (permalink / raw)
  To: u-boot

On Wed,  7 Aug 2013 09:22:33 +0200
Christian Gmeiner <christian.gmeiner@gmail.com> wrote:

> Without this change EDID_DETAILED_TIMING_VSYNC_OFFSET
> and EDID_DETAILED_TIMING_VSYNC_PULSE_WIDTH macros can
> not be used (compile error).
> The fix is quite trivial: rename struct member to the
> expected name.
> 
> Signed-off-by: Christian Gmeiner <christian.gmeiner@gmail.com>
> ---
>  include/edid.h |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Applied to u-boot-video/master. Thanks!

Anatolij

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

* [U-Boot] [PATCH] edid: rename struct member to fix two EDID_* macros
@ 2013-08-08  6:21 Christian Gmeiner
  0 siblings, 0 replies; 4+ messages in thread
From: Christian Gmeiner @ 2013-08-08  6:21 UTC (permalink / raw)
  To: u-boot

Without this change EDID_DETAILED_TIMING_VSYNC_OFFSET
and EDID_DETAILED_TIMING_VSYNC_PULSE_WIDTH macros can
not be used (compile error).
The fix is quite trivial: rename struct member to the
expected name.

Signed-off-by: Christian Gmeiner <christian.gmeiner@gmail.com>
---
 include/edid.h |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/edid.h b/include/edid.h
index f153091..480a773 100644
--- a/include/edid.h
+++ b/include/edid.h
@@ -54,7 +54,7 @@ struct edid_detailed_timing {
         (_x).vertical_blanking)
        unsigned char hsync_offset;
        unsigned char hsync_pulse_width;
-       unsigned char sync_offset_pulse_width;
+       unsigned char vsync_offset_pulse_width;
        unsigned char hsync_vsync_offset_pulse_width_hi;
 #define EDID_DETAILED_TIMING_HSYNC_OFFSET(_x) \
        ((GET_BITS((_x).hsync_vsync_offset_pulse_width_hi, 7, 6) << 8) + \

--
1.7.10.4

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

end of thread, other threads:[~2013-08-12 21:38 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-08-07  7:22 [U-Boot] [PATCH] edid: rename struct member to fix two EDID_* macros Christian Gmeiner
2013-08-12  6:14 ` Christian Gmeiner
2013-08-12 21:38 ` Anatolij Gustschin
2013-08-08  6:21 Christian Gmeiner

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.