Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 230401 - HDD LED doesn't blink with SATA disk on Silicon Image 3112
Summary: HDD LED doesn't blink with SATA disk on Silicon Image 3112
Status: RESOLVED UPSTREAM
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Core system (show other bugs)
Hardware: x86 Linux
: High normal with 1 vote (vote)
Assignee: Gentoo Kernel Bug Wranglers and Kernel Maintainers
URL: http://bugzilla.kernel.org/show_bug.c...
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-07-01 21:47 UTC by badboy
Modified: 2008-08-28 18:05 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
sata_sil_led-2.6.25.patch (sata_sil_led-2.6.25.patch,3.79 KB, patch)
2008-07-02 16:54 UTC, Duane Griffin
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description badboy 2008-07-01 21:47:26 UTC
The HDD LED doesn't blink when using sil3112 chip although the disk itself works like it should. Afaik it's the kernel driver issue.

Reproducible: Always



Expected Results:  
The HDD LEDs should blink

I've found a patch for the /usr/src/linux/drivers/ata/sata_sil.c but it is only for kernels <=2.6.19. The patch is located at:
http://acyr.net/projects.php#sii3112
It would be really awesome to have blinking LEDs again.
Comment 1 Jeroen Roovers (RETIRED) gentoo-dev 2008-07-02 14:36:42 UTC
Looks fishy to me. :)
Comment 2 Duane Griffin 2008-07-02 16:37:57 UTC
An earlier version of the patch was posted to LKML by the author and received positive feedback. No-one was sure whether it was safe for add-on cards though, which I guess is why it didn't get integrated.

See here:
http://lkml.org/lkml/2005/7/6/12
Comment 3 Duane Griffin 2008-07-02 16:54:28 UTC
Created attachment 159341 [details, diff]
sata_sil_led-2.6.25.patch

If you're feeling adventurous, here is a forward-port of the patch to 2.6.25. It is pretty much a straight port, with only a few minor cleanups to fix checkpatch issues and such-like, so hopefully it should be fairly safe. Having said that, I can't test it here, so if you want to try it you do so entirely at your own risk.
Comment 4 badboy 2008-07-02 17:28:46 UTC
(In reply to comment #3)
> Created an attachment (id=159341) [edit]
> sata_sil_led-2.6.25.patch
> 
> If you're feeling adventurous, here is a forward-port of the patch to 2.6.25.
> It is pretty much a straight port, with only a few minor cleanups to fix
> checkpatch issues and such-like, so hopefully it should be fairly safe. Having
> said that, I can't test it here, so if you want to try it you do so entirely at
> your own risk.
> 

I'll give it a try if only you tell how do I apply it.
Should it be patch -p0 (or -p1 or maybe without it) < sata_sil_led-2.6.25.patch?
Comment 5 Duane Griffin 2008-07-02 17:58:12 UTC
You can use the following command, inside the kernel directory:
patch -p1 < sata_sil_led-2.6.25.patch
Comment 6 badboy 2008-07-02 18:50:17 UTC
(In reply to comment #3)
> Created an attachment (id=159341) [edit]
> sata_sil_led-2.6.25.patch
> 
> If you're feeling adventurous, here is a forward-port of the patch to 2.6.25.
> It is pretty much a straight port, with only a few minor cleanups to fix
> checkpatch issues and such-like, so hopefully it should be fairly safe. Having
> said that, I can't test it here, so if you want to try it you do so entirely at
> your own risk.
> 

Nope, it doesn't work :(
Comment 7 Duane Griffin 2008-07-02 19:30:05 UTC
Do you see "Using GPIO for LED control" in dmesg? Did the patch work for you on earlier kernels or is this the first time you've tried it?
Comment 8 badboy 2008-07-02 19:53:25 UTC
(In reply to comment #7)
> Do you see "Using GPIO for LED control" in dmesg? Did the patch work for you on
> earlier kernels or is this the first time you've tried it?
> 

This is the first I'am trying do solve this issue and no, there's no "Using GPIO for LED control" in dmesg.
Comment 9 Duane Griffin 2008-07-02 21:18:56 UTC
Ah, I see. Your system probably has the chip wired up in a different way to the author's then. I'm afraid that patch isn't going to work for you, at least not without modification.
Comment 10 badboy 2008-07-02 21:31:06 UTC
(In reply to comment #9)
> Ah, I see. Your system probably has the chip wired up in a different way to the
> author's then. I'm afraid that patch isn't going to work for you, at least not
> without modification.
> 

I have a Gigabyte GA-7N400 Pro2 (Rev 1) motherboard (http://www.gigabyte.com.tw/Products/Motherboard/Products_Overview.aspx?ProductID=1678) with sil 3112 integrated.
Anyway, I've noticed that the HDD LED blinks for a moment and stops when OpenRC is booting the system. Is that helpfull?
Comment 11 Duane Griffin 2008-07-03 01:01:04 UTC
It doesn't help me much, I'm afraid :) You could try contacting the original author.
Comment 12 Mike Pagano gentoo-dev 2008-07-21 17:44:24 UTC
Seems like this has never worked.

http://bugzilla.kernel.org/show_bug.cgi?id=8281
Comment 13 Mike Pagano gentoo-dev 2008-07-21 17:45:01 UTC
...on later kernels.
Comment 14 Mike Pagano gentoo-dev 2008-08-28 18:05:34 UTC
We'll watch the upstream bug for developments.