Re: [PATCH 1/2 v6] of: add helper to parse display timings
From: Thierry Reding <hidden>
Date: 2012-10-20 19:59:51
Also in:
dri-devel, linux-devicetree, linux-media
On Thu, Oct 04, 2012 at 07:59:19PM +0200, Steffen Trumtrar wrote: [...]
quoted hunk ↗ jump to hunk
diff --git a/include/linux/of_display_timings.h b/include/linux/of_display_timings.h
[...]
+struct display_timings {
+ unsigned int num_timings;
+ unsigned int default_timing;
+
+ struct signal_timing **timings;
+};
+
+struct timing_entry {
+ u32 min;
+ u32 typ;
+ u32 max;
+};
+
+struct signal_timing {I'm slightly confused by the naming here. signal_timing seems overly generic in this context. Is there any reason why this isn't called display_timing or even display_mode?
Attachments
- (unnamed) [application/pgp-signature] 836 bytes