* [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
-- strict thread matches above, loose matches on Subject: below --
2013-08-08 6:21 Christian Gmeiner
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox