<feed xmlns='http://www.w3.org/2005/Atom'>
<title>linux/drivers/staging/bcm, branch v3.4.30</title>
<subtitle>Linux kernel source tree</subtitle>
<id>https://git.amat.us/linux/atom/drivers/staging/bcm?h=v3.4.30</id>
<link rel='self' href='https://git.amat.us/linux/atom/drivers/staging/bcm?h=v3.4.30'/>
<link rel='alternate' type='text/html' href='https://git.amat.us/linux/'/>
<updated>2012-03-03T00:36:55Z</updated>
<entry>
<title>Staging: bcm: fix possible memory leak of 'pstAddIndication' in CmHost.c and removes a whitespace</title>
<updated>2012-03-03T00:36:55Z</updated>
<author>
<name>Kevin McKinney</name>
<email>klmckinney1@gmail.com</email>
</author>
<published>2012-03-02T05:17:04Z</published>
<link rel='alternate' type='text/html' href='https://git.amat.us/linux/commit/?id=cc55bb03ea17fcbeffb5ae3ec68ea1f8673bc0c8'/>
<id>urn:sha1:cc55bb03ea17fcbeffb5ae3ec68ea1f8673bc0c8</id>
<content type='text'>
Memory is being allocated by kmalloc and stored in
variable pstAddIndication.  However, this memory is
not being freed in all cases. Therefore, this patch
frees it on several exit paths. This patch also
removes a whitespace.

Signed-off-by: Kevin McKinney &lt;klmckinney1@gmail.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
</entry>
<entry>
<title>Staging: bcm: fix possible NULL dereference of psfLocalSet in CmHost.c</title>
<updated>2012-02-24T19:53:10Z</updated>
<author>
<name>Kevin McKinney</name>
<email>klmckinney1@gmail.com</email>
</author>
<published>2012-02-24T04:41:05Z</published>
<link rel='alternate' type='text/html' href='https://git.amat.us/linux/commit/?id=adc4a3a4b0b78a2ab910685d138e0e598fc0edc7'/>
<id>urn:sha1:adc4a3a4b0b78a2ab910685d138e0e598fc0edc7</id>
<content type='text'>
Variable psfLocalSet may not follow the correct
path in the code, and therefore may not be set
properly. As such, causing a null dereference.

Signed-off-by: Kevin McKinney &lt;klmckinney1@gmail.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
</entry>
<entry>
<title>Staging: bcm: Move directives for the preprocessor statement to enum value in led_control.h</title>
<updated>2012-02-14T04:16:14Z</updated>
<author>
<name>Kevin McKinney</name>
<email>klmckinney1@gmail.com</email>
</author>
<published>2012-02-13T03:24:53Z</published>
<link rel='alternate' type='text/html' href='https://git.amat.us/linux/commit/?id=00da8eddf9eb8d432533c3a83da3a6470a45dfc4'/>
<id>urn:sha1:00da8eddf9eb8d432533c3a83da3a6470a45dfc4</id>
<content type='text'>
DRIVER_HALT is a driver state that was originally
defined as a #define statement. This patch moves
it to the LedEvents type as an enumerated
value for the purpose of removing a compile time warning:

drivers/staging/bcm/led_control.c: In function ‘LEDControlThread’:
drivers/staging/bcm/led_control.c:817:3: warning: case value ‘255’ not in enumerated type ‘LedEventInfo_t’ [-Wswitch]

Signed-off-by: Kevin McKinney &lt;klmckinney1@gmail.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
</entry>
<entry>
<title>Staging: bcm: fix CodingStyle warnings/errors reported by checkpatch.pl in led_control.h</title>
<updated>2012-02-10T17:58:52Z</updated>
<author>
<name>Gerard Ryan</name>
<email>gerard@ryan.lt</email>
</author>
<published>2012-02-10T01:54:50Z</published>
<link rel='alternate' type='text/html' href='https://git.amat.us/linux/commit/?id=242501ce5d5296236701612940ed397a2b3fd4bf'/>
<id>urn:sha1:242501ce5d5296236701612940ed397a2b3fd4bf</id>
<content type='text'>
This is a patch to the led_control.h file that fixes numerous warnings
and errors reported by the checkpatch.pl tool. There still remain a few
more, but as this is my first attempt at a commit, I'm not going to be
too adventurous!

Signed-off-by: Gerard Ryan &lt;gerard@ryan.lt&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
</entry>
<entry>
<title>Staging: bcm: Use memdup_user rather than duplicating its implementation</title>
<updated>2012-02-09T01:22:16Z</updated>
<author>
<name>Thomas Meyer</name>
<email>thomas@m3y3r.de</email>
</author>
<published>2012-01-22T17:29:51Z</published>
<link rel='alternate' type='text/html' href='https://git.amat.us/linux/commit/?id=2d9ebe77b7665a431a9816eff3eb588e05176dfa'/>
<id>urn:sha1:2d9ebe77b7665a431a9816eff3eb588e05176dfa</id>
<content type='text'>
This is a little bit restricted to reduce false positives

The semantic patch that makes this change is available
in scripts/coccinelle/api/memdup_user.cocci.

More information about semantic patching is available at
http://coccinelle.lip6.fr/

Signed-off-by: Thomas Meyer &lt;thomas@m3y3r.de&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
</entry>
<entry>
<title>Staging: bcm: Alter name, datatype, and default value of iterator variables.</title>
<updated>2012-02-09T01:19:06Z</updated>
<author>
<name>Kevin McKinney</name>
<email>klmckinney1@gmail.com</email>
</author>
<published>2012-01-05T01:29:05Z</published>
<link rel='alternate' type='text/html' href='https://git.amat.us/linux/commit/?id=5cf4d6b936ca21f67aa763dd0d3f8fdd9873b22c'/>
<id>urn:sha1:5cf4d6b936ca21f67aa763dd0d3f8fdd9873b22c</id>
<content type='text'>
This patch renames variables used in iteration
statements with i, changes the datatype to int,
and removes any default value.

Signed-off-by: Kevin McKinney &lt;klmckinney1@gmail.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
</entry>
<entry>
<title>Staging: bcm: Replace variables and function outputs defined as INT with int.</title>
<updated>2012-02-09T01:19:05Z</updated>
<author>
<name>Kevin McKinney</name>
<email>klmckinney1@gmail.com</email>
</author>
<published>2012-01-05T01:29:04Z</published>
<link rel='alternate' type='text/html' href='https://git.amat.us/linux/commit/?id=ce4bbc2ae4626f6fdf1af8767d59852978e5f942'/>
<id>urn:sha1:ce4bbc2ae4626f6fdf1af8767d59852978e5f942</id>
<content type='text'>
This patch replaces all variables and function output
that have a datatype definition of "INT" with "int".

Signed-off-by: Kevin McKinney &lt;klmckinney1@gmail.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
</entry>
<entry>
<title>Staging: bcm: Remove assignment from if statement and reverse if logic for readability.</title>
<updated>2012-02-09T01:19:05Z</updated>
<author>
<name>Kevin McKinney</name>
<email>klmckinney1@gmail.com</email>
</author>
<published>2012-01-05T01:29:03Z</published>
<link rel='alternate' type='text/html' href='https://git.amat.us/linux/commit/?id=cffae184c3146abda6ef1153ac4f844d3ed6e177'/>
<id>urn:sha1:cffae184c3146abda6ef1153ac4f844d3ed6e177</id>
<content type='text'>
This patch removes an assignment from an if statement,
and it reverses the logic in several if statements to
make them more readable and understandable.

Signed-off-by: Kevin McKinney &lt;klmckinney1@gmail.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
</entry>
<entry>
<title>Staging: bcm: Remove typedef from CmHost.c and use enum.</title>
<updated>2012-02-09T01:19:05Z</updated>
<author>
<name>Kevin McKinney</name>
<email>klmckinney1@gmail.com</email>
</author>
<published>2012-01-05T01:29:02Z</published>
<link rel='alternate' type='text/html' href='https://git.amat.us/linux/commit/?id=5db125fb5ca2fed67a668c532af576a67a3d3bdc'/>
<id>urn:sha1:5db125fb5ca2fed67a668c532af576a67a3d3bdc</id>
<content type='text'>
This patch removes a typedef from a variable definition
in CmHost.c, and uses a enum instead. This Warning
was reported by checkpatch.pl.

Signed-off-by: Kevin McKinney &lt;klmckinney1@gmail.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
</entry>
<entry>
<title>Staging: bcm: Replace dated variable __FUNCTION__.</title>
<updated>2012-02-09T01:19:04Z</updated>
<author>
<name>Kevin McKinney</name>
<email>klmckinney1@gmail.com</email>
</author>
<published>2012-01-05T01:29:01Z</published>
<link rel='alternate' type='text/html' href='https://git.amat.us/linux/commit/?id=e4868623bfe1c05e1e17d86729907a3850d12882'/>
<id>urn:sha1:e4868623bfe1c05e1e17d86729907a3850d12882</id>
<content type='text'>
This patch replaces the obsolete variable, __FUNCTION__,
that holds the name of the current function with
variable, __func__.  Reported by checkpatch.pl.

Signed-off-by: Kevin McKinney &lt;klmckinney1@gmail.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
</entry>
</feed>
