<feed xmlns='http://www.w3.org/2005/Atom'>
<title>linux.git/drivers/staging/bcm/InterfaceIdleMode.c, branch v3.17</title>
<subtitle>Linux kernel source tree</subtitle>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux.git/'/>
<entry>
<title>drivers/staging: Remove useless return variables</title>
<updated>2014-05-26T17:32:15+00:00</updated>
<author>
<name>Peter Senna Tschudin</name>
<email>peter.senna@gmail.com</email>
</author>
<published>2014-05-26T14:08:50+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux.git/commit/?id=4764ca981b040048766e4f39a45a4b9c5cecff9c'/>
<id>4764ca981b040048766e4f39a45a4b9c5cecff9c</id>
<content type='text'>
This patch remove variables that are initialized with a constant,
are never updated, and are only used as parameter of return.
Return the constant instead of using a variable.

Verified by compilation only.

The coccinelle script that find and fixes this issue is:
// &lt;smpl&gt;
@@
type T;
constant C;
identifier ret;
@@
- T ret = C;
... when != ret
    when strict
return
- ret
+ C
;
// &lt;/smpl&gt;

Signed-off-by: Peter Senna Tschudin &lt;peter.senna@gmail.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This patch remove variables that are initialized with a constant,
are never updated, and are only used as parameter of return.
Return the constant instead of using a variable.

Verified by compilation only.

The coccinelle script that find and fixes this issue is:
// &lt;smpl&gt;
@@
type T;
constant C;
identifier ret;
@@
- T ret = C;
... when != ret
    when strict
return
- ret
+ C
;
// &lt;/smpl&gt;

Signed-off-by: Peter Senna Tschudin &lt;peter.senna@gmail.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>staging: bcm: fixed warning about no and prohibited space in InterfaceIdleMode.c.</title>
<updated>2013-12-20T19:47:42+00:00</updated>
<author>
<name>Gokulnath Avanashilingam</name>
<email>Gokulnath.Avanashilingam@in.bosch.com</email>
</author>
<published>2013-12-20T15:24:32+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux.git/commit/?id=8b80448d478f8cbaeb9deac10731f06db94c69d6'/>
<id>8b80448d478f8cbaeb9deac10731f06db94c69d6</id>
<content type='text'>
This patch fixes all the warning related to spacing issues
found by checkpatch.pl script in InterfaceIdleMode.c

Signed-off-by: Gokulnath Avanashilingam &lt;Gokulnath.Avanashilingam@in.bosch.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This patch fixes all the warning related to spacing issues
found by checkpatch.pl script in InterfaceIdleMode.c

Signed-off-by: Gokulnath Avanashilingam &lt;Gokulnath.Avanashilingam@in.bosch.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>staging: bcm: using time_after and time_before in InterfaceIdleMode.c</title>
<updated>2013-12-20T19:47:42+00:00</updated>
<author>
<name>Gokulnath Avanashilingam</name>
<email>Gokulnath.Avanashilingam@in.bosch.com</email>
</author>
<published>2013-12-20T15:24:31+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux.git/commit/?id=e42ff08dd9626e1cbb824837734e3c3c633d5ad1'/>
<id>e42ff08dd9626e1cbb824837734e3c3c633d5ad1</id>
<content type='text'>
Used the time_after and time_before insted of comparing
the jiffies directly.This will fix the warnings and errors
found by the checkpatch.pl script.

Signed-off-by: Gokulnath Avanashilingam &lt;Gokulnath.Avanashilingam@in.bosch.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Used the time_after and time_before insted of comparing
the jiffies directly.This will fix the warnings and errors
found by the checkpatch.pl script.

Signed-off-by: Gokulnath Avanashilingam &lt;Gokulnath.Avanashilingam@in.bosch.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>staging: bcm: line over 80 characters in InterfaceIdleMode.c</title>
<updated>2013-12-20T19:47:42+00:00</updated>
<author>
<name>Gokulnath Avanashilingam</name>
<email>Gokulnath.Avanashilingam@in.bosch.com</email>
</author>
<published>2013-12-20T15:24:30+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux.git/commit/?id=786bdddc36ba707f9f60de499199e3835e8911f3'/>
<id>786bdddc36ba707f9f60de499199e3835e8911f3</id>
<content type='text'>
Fixed all the line over 80 characters warning found by
checkpatch.pl script.

Signed-off-by: Gokulnath Avanashilingam &lt;Gokulnath.Avanashilingam@in.bosch.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Fixed all the line over 80 characters warning found by
checkpatch.pl script.

Signed-off-by: Gokulnath Avanashilingam &lt;Gokulnath.Avanashilingam@in.bosch.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>staging: bcm: Remove Developer Debug prints in InterfaceIdleMode.c</title>
<updated>2013-12-20T19:47:42+00:00</updated>
<author>
<name>Gokulnath Avanashilingam</name>
<email>Gokulnath.Avanashilingam@in.bosch.com</email>
</author>
<published>2013-12-20T15:24:29+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux.git/commit/?id=d52dd37e19b6dada6cfeace4784346497357fa24'/>
<id>d52dd37e19b6dada6cfeace4784346497357fa24</id>
<content type='text'>
Removed the developer debug prints BCM_DEBUG_PRINT()
as per the TODO list, also removed braces for the
if-statement to match coding style

Signed-off-by: Gokulnath Avanashilingam &lt;Gokulnath.Avanashilingam@in.bosch.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Removed the developer debug prints BCM_DEBUG_PRINT()
as per the TODO list, also removed braces for the
if-statement to match coding style

Signed-off-by: Gokulnath Avanashilingam &lt;Gokulnath.Avanashilingam@in.bosch.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>staging: bcm: Replace FALSE with false</title>
<updated>2013-10-28T21:13:46+00:00</updated>
<author>
<name>Lisa Nguyen</name>
<email>lisa@xenapiadmin.com</email>
</author>
<published>2013-10-28T08:36:19+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux.git/commit/?id=f70c8a91cd0e743d0531a158d939bbdb6c0874dc'/>
<id>f70c8a91cd0e743d0531a158d939bbdb6c0874dc</id>
<content type='text'>
Replace user-defined type FALSE with C defined false keyword.

Signed-off-by: Lisa Nguyen &lt;lisa@xenapiadmin.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Replace user-defined type FALSE with C defined false keyword.

Signed-off-by: Lisa Nguyen &lt;lisa@xenapiadmin.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>Staging: bcm: Fixed warning 'space required before the open parenthesis '(''.</title>
<updated>2013-05-16T22:48:56+00:00</updated>
<author>
<name>Tülin İzer</name>
<email>tulinizer@gmail.com</email>
</author>
<published>2013-05-15T00:43:51+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux.git/commit/?id=db95f150b98691bc2a6e8673d7802bb6ee1b5447'/>
<id>db95f150b98691bc2a6e8673d7802bb6ee1b5447</id>
<content type='text'>
This patch fixes warning 'space required before the open parenthesis '(''
found by checkpacth.pl in driver bcm.

Signed-off-by: Tülin İzer &lt;tulinizer@gmail.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This patch fixes warning 'space required before the open parenthesis '(''
found by checkpacth.pl in driver bcm.

Signed-off-by: Tülin İzer &lt;tulinizer@gmail.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>Staging: bcm: Fixed error 'else should follow close brace '}''.</title>
<updated>2013-05-16T22:48:56+00:00</updated>
<author>
<name>Tülin İzer</name>
<email>tulinizer@gmail.com</email>
</author>
<published>2013-05-15T00:43:18+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux.git/commit/?id=461c8d2423fc9c00e8834f379ee80f79c1d6fb9f'/>
<id>461c8d2423fc9c00e8834f379ee80f79c1d6fb9f</id>
<content type='text'>
This patch fixes error 'else should follow close brace '}''
found by checkpatch in driver bcm.

Signed-off-by: Tülin İzer &lt;tulinizer@gmail.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This patch fixes error 'else should follow close brace '}''
found by checkpatch in driver bcm.

Signed-off-by: Tülin İzer &lt;tulinizer@gmail.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>Staging: bcm: Fixed warning 'braces {} are not necessary for single statement blocks'.</title>
<updated>2013-05-16T22:48:56+00:00</updated>
<author>
<name>Tülin İzer</name>
<email>tulinizer@gmail.com</email>
</author>
<published>2013-05-15T00:42:46+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux.git/commit/?id=3b5ecbab99c9ed10b30c18f984ef144167b2c3fc'/>
<id>3b5ecbab99c9ed10b30c18f984ef144167b2c3fc</id>
<content type='text'>
This patch fixes warning: 'braces {} are not necessary for single
statement blocks' found by checkpatch.pl in driver bcm.

Signed-off-by: Tülin İzer &lt;tulinizer@gmail.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This patch fixes warning: 'braces {} are not necessary for single
statement blocks' found by checkpatch.pl in driver bcm.

Signed-off-by: Tülin İzer &lt;tulinizer@gmail.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>Staging: bcm: Fixed error 'that open brace { should be on the previous line'.</title>
<updated>2013-05-16T22:48:55+00:00</updated>
<author>
<name>Tülin İzer</name>
<email>tulinizer@gmail.com</email>
</author>
<published>2013-05-15T00:42:12+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux.git/commit/?id=ec8451372b8612f7595a3c9f70961aaa1c85a0c2'/>
<id>ec8451372b8612f7595a3c9f70961aaa1c85a0c2</id>
<content type='text'>
This patch fixes error 'that open brace { should be on the previous line'
found by checkpatch in driver bcm.

Signed-off-by: Tülin İzer &lt;tulinizer@gmail.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This patch fixes error 'that open brace { should be on the previous line'
found by checkpatch in driver bcm.

Signed-off-by: Tülin İzer &lt;tulinizer@gmail.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</pre>
</div>
</content>
</entry>
</feed>
