Re: [PATCH RFC] video: fbdev: sis: condition with no effect

2015-02-06 Thread Thomas Winischhofer
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Nicholas Mc Guire wrote: > On Fri, 06 Feb 2015, Thomas Winischhofer wrote: > >> -BEGIN PGP SIGNED MESSAGE- >> Hash: SHA1 >> >> Tormod Volden wrote: >>> On Thu, Feb 5, 2015 at 9:45 PM, Scot Doyle wrote: On Wed, 4 Feb 2015, Nicholas Mc Guir

Re: [PATCH RFC] video: fbdev: sis: condition with no effect

2015-02-06 Thread Nicholas Mc Guire
On Fri, 06 Feb 2015, Thomas Winischhofer wrote: > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA1 > > Tormod Volden wrote: > > On Thu, Feb 5, 2015 at 9:45 PM, Scot Doyle wrote: > >> On Wed, 4 Feb 2015, Nicholas Mc Guire wrote: > >>> The if and the else branch code are identical - so the condition

Re: [PATCH RFC] video: fbdev: sis: condition with no effect

2015-02-06 Thread Nicholas Mc Guire
On Thu, 05 Feb 2015, Tormod Volden wrote: > On Thu, Feb 5, 2015 at 9:45 PM, Scot Doyle wrote: > > On Wed, 4 Feb 2015, Nicholas Mc Guire wrote: > >> The if and the else branch code are identical - so the condition has no > >> effect on the effective code - this patch removes the condition and the >

Re: [PATCH RFC] video: fbdev: sis: condition with no effect

2015-02-05 Thread Thomas Winischhofer
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Tormod Volden wrote: > On Thu, Feb 5, 2015 at 9:45 PM, Scot Doyle wrote: >> On Wed, 4 Feb 2015, Nicholas Mc Guire wrote: >>> The if and the else branch code are identical - so the condition has no >>> effect on the effective code - this patch removes t

Re: [PATCH RFC] video: fbdev: sis: condition with no effect

2015-02-05 Thread Scot Doyle
On Thu, 5 Feb 2015, Tormod Volden wrote: > On Thu, Feb 5, 2015 at 9:45 PM, Scot Doyle wrote: > > On Wed, 4 Feb 2015, Nicholas Mc Guire wrote: > >> The if and the else branch code are identical - so the condition has no > >> effect on the effective code - this patch removes the condition and the > >

Re: [PATCH RFC] video: fbdev: sis: condition with no effect

2015-02-05 Thread Tormod Volden
On Thu, Feb 5, 2015 at 9:45 PM, Scot Doyle wrote: > On Wed, 4 Feb 2015, Nicholas Mc Guire wrote: >> The if and the else branch code are identical - so the condition has no >> effect on the effective code - this patch removes the condition and the >> duplicated code. >> >> Signed-off-by: Nicholas Mc

Re: [PATCH RFC] video: fbdev: sis: condition with no effect

2015-02-05 Thread Scot Doyle
On Wed, 4 Feb 2015, Nicholas Mc Guire wrote: > The if and the else branch code are identical - so the condition has no > effect on the effective code - this patch removes the condition and the > duplicated code. > > Signed-off-by: Nicholas Mc Guire > --- > > This code has been in here since comm