<feed xmlns='http://www.w3.org/2005/Atom'>
<title>user/sven/linux.git/drivers/video/sis, branch tmp/leds/core</title>
<subtitle>Linux Kernel
</subtitle>
<id>https://git.stealer.net/cgit.cgi/user/sven/linux.git/atom?h=tmp%2Fleds%2Fcore</id>
<link rel='self' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/atom?h=tmp%2Fleds%2Fcore'/>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/'/>
<updated>2008-07-24T17:47:37Z</updated>
<entry>
<title>video/sis/: remove compat code</title>
<updated>2008-07-24T17:47:37Z</updated>
<author>
<name>Adrian Bunk</name>
<email>bunk@kernel.org</email>
</author>
<published>2008-07-24T04:31:12Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=14aefd1b49ff3bd13caa37fb06bd53488d5d1486'/>
<id>urn:sha1:14aefd1b49ff3bd13caa37fb06bd53488d5d1486</id>
<content type='text'>
This patch removes compat code for older kernel versions.

Signed-off-by: Adrian Bunk &lt;bunk@kernel.org&gt;
Cc: &lt;thomas@winischhofer.net&gt;
Signed-off-by: Andrew Morton &lt;akpm@linux-foundation.org&gt;
Signed-off-by: Linus Torvalds &lt;torvalds@linux-foundation.org&gt;
</content>
</entry>
<entry>
<title>fbdev: fix integer as NULL pointer warning</title>
<updated>2008-05-23T15:11:07Z</updated>
<author>
<name>Harvey Harrison</name>
<email>harvey.harrison@gmail.com</email>
</author>
<published>2008-05-22T22:45:08Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=5e2daeb3c982ea19ecad0c2e720a4052034be14b'/>
<id>urn:sha1:5e2daeb3c982ea19ecad0c2e720a4052034be14b</id>
<content type='text'>
drivers/video/aty/atyfb_base.c:3359:26: warning: Using plain integer as NULL pointer
drivers/video/aty/radeon_base.c:2280:32: warning: Using plain integer as NULL pointer
drivers/video/matrox/matroxfb_base.h:203:25: warning: Using plain integer as NULL pointer
drivers/video/matrox/matroxfb_base.h:203:25: warning: Using plain integer as NULL pointer
drivers/video/sis/sis_main.c:5790:44: warning: Using plain integer as NULL pointer

Signed-off-by: Harvey Harrison &lt;harvey.harrison@gmail.com&gt;
Signed-off-by: Linus Torvalds &lt;torvalds@linux-foundation.org&gt;
</content>
</entry>
<entry>
<title>video: replace remaining __FUNCTION__ occurrences</title>
<updated>2008-04-28T15:58:42Z</updated>
<author>
<name>Harvey Harrison</name>
<email>harvey.harrison@gmail.com</email>
</author>
<published>2008-04-28T09:15:47Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=5ae121705bed9ea7425daef4d7d29038f7312f3f'/>
<id>urn:sha1:5ae121705bed9ea7425daef4d7d29038f7312f3f</id>
<content type='text'>
__FUNCTION__ is gcc-specific, use __func__

Signed-off-by: Harvey Harrison &lt;harvey.harrison@gmail.com&gt;
Cc: "Antonino A. Daplas" &lt;adaplas@pol.net&gt;
Cc: Krzysztof Helt &lt;krzysztof.h1@wp.pl&gt;
Cc: Antonino Daplas &lt;adaplas@gmail.com&gt;
Cc: Antonino A. Daplas &lt;adaplas@gmail.com&gt;
Cc: Antonino Daplas &lt;adaplas@pol.net&gt;
Cc: Richard Purdie &lt;rpurdie@rpsys.net&gt;
Cc: Jean Delvare &lt;khali@linux-fr.org&gt;
Cc: Adrian Bunk &lt;bunk@stusta.de&gt;
Cc: Russell King &lt;rmk+kernel@arm.linux.org.uk&gt;
Cc: Benjamin Herrenschmidt &lt;benh@kernel.crashing.org&gt;
Cc: Ben Dooks &lt;ben-linux@fluff.org&gt;
Signed-off-by: Andrew Morton &lt;akpm@linux-foundation.org&gt;
Signed-off-by: Linus Torvalds &lt;torvalds@linux-foundation.org&gt;
</content>
</entry>
<entry>
<title>drivers/video: remove unnecessary pci_dev_put</title>
<updated>2008-02-06T18:41:15Z</updated>
<author>
<name>Julia Lawall</name>
<email>julia@diku.dk</email>
</author>
<published>2008-02-06T09:39:07Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=ea237a6ae953b19d03f29236f095389d7906a0b4'/>
<id>urn:sha1:ea237a6ae953b19d03f29236f095389d7906a0b4</id>
<content type='text'>
pci_get_class implicitly does a pci_dev_put on its second argument, so
pci_dev_put is only needed if there is a break out of the loop.

The semantic match detecting this problem is as follows:

// &lt;smpl&gt;
@@
expression dev;
expression E;
@@

* pci_dev_put(dev)
  ... when != dev = E
(
* pci_get_device(...,dev)
|
* pci_get_device_reverse(...,dev)
|
* pci_get_subsys(...,dev)
|
* pci_get_class(...,dev)
)
// &lt;/smpl&gt;

Signed-off-by: Julia Lawall &lt;julia@diku.dk&gt;
Cc: Thomas Winischhofer &lt;thomas@winischhofer.net&gt;
Cc: "Antonino A. Daplas" &lt;adaplas@pol.net&gt;
Signed-off-by: Andrew Morton &lt;akpm@linux-foundation.org&gt;
Signed-off-by: Linus Torvalds &lt;torvalds@linux-foundation.org&gt;
</content>
</entry>
<entry>
<title>fix ! versus &amp; precedence in various places</title>
<updated>2008-02-06T18:40:59Z</updated>
<author>
<name>Alexey Dobriyan</name>
<email>adobriyan@gmail.com</email>
</author>
<published>2008-02-06T09:36:06Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=eaa0ff15c30dc9799eb4d12660edb73aeb6d32c5'/>
<id>urn:sha1:eaa0ff15c30dc9799eb4d12660edb73aeb6d32c5</id>
<content type='text'>
Fix various instances of

	if (!expr &amp; mask)

which should probably have been

	if (!(expr &amp; mask))

Signed-off-by: Alexey Dobriyan &lt;adobriyan@gmail.com&gt;
Cc: Jens Axboe &lt;jens.axboe@oracle.com&gt;
Cc: Peter Osterlund &lt;petero2@telia.com&gt;
Cc: Karsten Keil &lt;kkeil@suse.de&gt;
Cc: Mauro Carvalho Chehab &lt;mchehab@infradead.org&gt;
Cc: "Antonino A. Daplas" &lt;adaplas@pol.net&gt;
Cc: Mark Fasheh &lt;mark.fasheh@oracle.com&gt;
Cc: "David S. Miller" &lt;davem@davemloft.net&gt;
Cc: Jeff Garzik &lt;jeff@garzik.org&gt;
Cc: Al Viro &lt;viro@zeniv.linux.org.uk&gt;
Signed-off-by: Andrew Morton &lt;akpm@linux-foundation.org&gt;
Signed-off-by: Linus Torvalds &lt;torvalds@linux-foundation.org&gt;
</content>
</entry>
<entry>
<title>USB: sisusb: *_ioctl32_conversion functions do not exist in recent kernels</title>
<updated>2008-02-01T22:34:48Z</updated>
<author>
<name>Fernando Luis Vázquez Cao</name>
<email>fernando@oss.ntt.co.jp</email>
</author>
<published>2007-11-05T07:21:11Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=ed8a5d49a0d4b5cf411a71476e6582f153df54e3'/>
<id>urn:sha1:ed8a5d49a0d4b5cf411a71476e6582f153df54e3</id>
<content type='text'>
Remove dead code while at it.

Signed-off-by: Fernando Luis Vazquez Cao &lt;fernando@oss.ntt.co.jp&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@suse.de&gt;
</content>
</entry>
<entry>
<title>video/sis/: fix negative array index</title>
<updated>2007-11-15T02:45:38Z</updated>
<author>
<name>Adrian Bunk</name>
<email>bunk@kernel.org</email>
</author>
<published>2007-11-15T00:59:02Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=a9e60e5c3c4721dd216047e4c58c4eb89789f519'/>
<id>urn:sha1:a9e60e5c3c4721dd216047e4c58c4eb89789f519</id>
<content type='text'>
This patch fixes the possible usage of a negative value as an array
index spotted by the Coverity checker.

sisfb_validate_mode() could return a negative error code and we must check for
that prior to using its return value as an array index.

Signed-off-by: Adrian Bunk &lt;bunk@kernel.org&gt;
Cc: Thomas Winischhofer &lt;thomas@winischhofer.net&gt;
Cc: "Antonino A. Daplas" &lt;adaplas@pol.net&gt;
Signed-off-by: Andrew Morton &lt;akpm@linux-foundation.org&gt;
Signed-off-by: Linus Torvalds &lt;torvalds@linux-foundation.org&gt;
</content>
</entry>
<entry>
<title>fbdev: change asm/uaccess.h to linux/uaccess.h</title>
<updated>2007-10-16T16:43:17Z</updated>
<author>
<name>Krzysztof Helt</name>
<email>krzysztof.h1@wp.pl</email>
</author>
<published>2007-10-16T08:29:04Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=84902b7af642c86a518c17629c0dbe705a4b6d14'/>
<id>urn:sha1:84902b7af642c86a518c17629c0dbe705a4b6d14</id>
<content type='text'>
This patch replaces &lt;asm/uaccess.h&gt; with &lt;linux/uaccess.h&gt; after the
checkpatch.pl hint.  The include of &lt;asm/uaccess.h&gt; is removed if the driver
does not use it.

Signed-off-by: Krzysztof Helt &lt;krzysztof.h1@wp.pl&gt;
Signed-off-by: Antonino Daplas &lt;adaplas@gmail.com&gt;
Signed-off-by: Andrew Morton &lt;akpm@linux-foundation.org&gt;
Signed-off-by: Linus Torvalds &lt;torvalds@linux-foundation.org&gt;
</content>
</entry>
<entry>
<title>sisfb: fix pseudo_palette array size and overrun</title>
<updated>2007-07-17T17:23:13Z</updated>
<author>
<name>Antonino A. Daplas</name>
<email>adaplas@gmail.com</email>
</author>
<published>2007-07-17T11:05:44Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=000d5335c6e7f5dbcd849b62b4be7ae005aa6974'/>
<id>urn:sha1:000d5335c6e7f5dbcd849b62b4be7ae005aa6974</id>
<content type='text'>
- the pseudo_palette is only 16 elements long.
- do not write to the pseudo_palette if regno (array index) is more than 15.

Signed-off-by: Antonino Daplas &lt;adaplas@gmail.com&gt;
Signed-off-by: Andrew Morton &lt;akpm@linux-foundation.org&gt;
Signed-off-by: Linus Torvalds &lt;torvalds@linux-foundation.org&gt;
</content>
</entry>
<entry>
<title>PCI: Change all drivers to use pci_device-&gt;revision</title>
<updated>2007-07-11T23:02:10Z</updated>
<author>
<name>Auke Kok</name>
<email>auke-jan.h.kok@intel.com</email>
</author>
<published>2007-06-08T22:46:36Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=44c10138fd4bbc4b6d6bff0873c24902f2a9da65'/>
<id>urn:sha1:44c10138fd4bbc4b6d6bff0873c24902f2a9da65</id>
<content type='text'>
Instead of all drivers reading pci config space to get the revision
ID, they can now use the pci_device-&gt;revision member.

This exposes some issues where drivers where reading a word or a dword
for the revision number, and adding useless error-handling around the
read. Some drivers even just read it for no purpose of all.

In devices where the revision ID is being copied over and used in what
appears to be the equivalent of hotpath, I have left the copy code
and the cached copy as not to influence the driver's performance.

Compile tested with make all{yes,mod}config on x86_64 and i386.

Signed-off-by: Auke Kok &lt;auke-jan.h.kok@intel.com&gt;
Acked-by: Dave Jones &lt;davej@redhat.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@suse.de&gt;

</content>
</entry>
</feed>
