[PATCH 5/5] videomode: rename fields

2013-03-18 Thread Tomi Valkeinen
On 2013-03-18 10:00, Daniel Vetter wrote: > On Tue, Mar 12, 2013 at 03:40:14PM +0200, Tomi Valkeinen wrote: >> Hi, >> >> On 2013-03-12 15:37, Laurent Pinchart wrote: >>> Hi Tomi, >>> >>> Thanks for the patch. >>> >>> On Tuesday 12 March 2013 12:19:38 Tomi Valkeinen wrote: Structs videomode and

Re: [PATCH 5/5] videomode: rename fields

2013-03-18 Thread Tomi Valkeinen
On 2013-03-18 10:00, Daniel Vetter wrote: > On Tue, Mar 12, 2013 at 03:40:14PM +0200, Tomi Valkeinen wrote: >> Hi, >> >> On 2013-03-12 15:37, Laurent Pinchart wrote: >>> Hi Tomi, >>> >>> Thanks for the patch. >>> >>> On Tuesday 12 March 2013 12:19:38 Tomi Valkeinen wrote: Structs videomode and

[PATCH 5/5] videomode: rename fields

2013-03-18 Thread Daniel Vetter
On Tue, Mar 12, 2013 at 03:40:14PM +0200, Tomi Valkeinen wrote: > Hi, > > On 2013-03-12 15:37, Laurent Pinchart wrote: > > Hi Tomi, > > > > Thanks for the patch. > > > > On Tuesday 12 March 2013 12:19:38 Tomi Valkeinen wrote: > >> Structs videomode and display_timing have rather long field names

Re: [PATCH 5/5] videomode: rename fields

2013-03-18 Thread Daniel Vetter
On Tue, Mar 12, 2013 at 03:40:14PM +0200, Tomi Valkeinen wrote: > Hi, > > On 2013-03-12 15:37, Laurent Pinchart wrote: > > Hi Tomi, > > > > Thanks for the patch. > > > > On Tuesday 12 March 2013 12:19:38 Tomi Valkeinen wrote: > >> Structs videomode and display_timing have rather long field names

[PATCH 5/5] videomode: rename fields

2013-03-12 Thread Tomi Valkeinen
Hi, On 2013-03-12 15:37, Laurent Pinchart wrote: > Hi Tomi, > > Thanks for the patch. > > On Tuesday 12 March 2013 12:19:38 Tomi Valkeinen wrote: >> Structs videomode and display_timing have rather long field names for >> the timing values. Nothing wrong with that as such, but this patch >> chan

[PATCH 5/5] videomode: rename fields

2013-03-12 Thread Steffen Trumtrar
On Tue, Mar 12, 2013 at 12:19:38PM +0200, Tomi Valkeinen wrote: > Structs videomode and display_timing have rather long field names for > the timing values. Nothing wrong with that as such, but this patch > changes them to abbreviations for the following reasons: > > * The timing values often need

[PATCH 5/5] videomode: rename fields

2013-03-12 Thread Laurent Pinchart
Hi Tomi, Thanks for the patch. On Tuesday 12 March 2013 12:19:38 Tomi Valkeinen wrote: > Structs videomode and display_timing have rather long field names for > the timing values. Nothing wrong with that as such, but this patch > changes them to abbreviations for the following reasons: > > * The

[PATCH 5/5] videomode: rename fields

2013-03-12 Thread Tomi Valkeinen
Structs videomode and display_timing have rather long field names for the timing values. Nothing wrong with that as such, but this patch changes them to abbreviations for the following reasons: * The timing values often need to be used in calculations, and long field names makes their direct use

Re: [PATCH 5/5] videomode: rename fields

2013-03-12 Thread Tomi Valkeinen
Hi, On 2013-03-12 15:37, Laurent Pinchart wrote: > Hi Tomi, > > Thanks for the patch. > > On Tuesday 12 March 2013 12:19:38 Tomi Valkeinen wrote: >> Structs videomode and display_timing have rather long field names for >> the timing values. Nothing wrong with that as such, but this patch >> chan

Re: [PATCH 5/5] videomode: rename fields

2013-03-12 Thread Steffen Trumtrar
On Tue, Mar 12, 2013 at 12:19:38PM +0200, Tomi Valkeinen wrote: > Structs videomode and display_timing have rather long field names for > the timing values. Nothing wrong with that as such, but this patch > changes them to abbreviations for the following reasons: > > * The timing values often need

Re: [PATCH 5/5] videomode: rename fields

2013-03-12 Thread Laurent Pinchart
Hi Tomi, Thanks for the patch. On Tuesday 12 March 2013 12:19:38 Tomi Valkeinen wrote: > Structs videomode and display_timing have rather long field names for > the timing values. Nothing wrong with that as such, but this patch > changes them to abbreviations for the following reasons: > > * The

[PATCH 5/5] videomode: rename fields

2013-03-12 Thread Tomi Valkeinen
Structs videomode and display_timing have rather long field names for the timing values. Nothing wrong with that as such, but this patch changes them to abbreviations for the following reasons: * The timing values often need to be used in calculations, and long field names makes their direct use