On 08/08/2010 02:14 PM, Uwe Hermann wrote:
Hi,
On Fri, Mar 19, 2010 at 03:27:08PM -0700, Sean Nelson wrote:
This patch replaces the earlier one sent, and adds A8JM (and possibly all A8J series) support, and additionally adds DMI detection of A8J, after the r896 fix to dmi detection.
Signed-off-by: James Lancasterdeathstalker@gmail.com
The DMI identification triggers on either the 2nd or 5th dmi strings, which are both "A8J" followed by different numbers of spaces. Combined with the PCI IDs, it might work on any of the A8J series seeing that the main difference in all but the A8JS is the video card/screen (possibly that too, while there they did add sata in place of IDE). I believe (based on having a video card that was supposed to only be on the A8JS, that it has a different device number (1232), that the A8JS will not be picked up.)
I've tested this on my own computer, it picks it up and appears to flash just fine. I think I probably should have added Acked by as it works, but I'll be conservative on the issue.
- {0x8086, 0x27A0, 0x1043, 0x1287, 0x8086, 0x27DF, 0x1043, 0x1287, "^A8J", NULL, NULL, "ASUS", "A8JM", 0, NT, intel_ich_gpio34_raise},
- {0x8086, 0x27A0, 0, 0, 0x8086, 0x27b8, 0, 0, NULL, "kontron", "986lcd-m", "Kontron", "986LCD-M", 0, OK, board_kontron_986lcd_m},
- {0x8086, 0x27A0, 0x8086, 0x27a0, 0x8086, 0x27b8, 0x8086, 0x27b8, NULL, "kontron", "986lcd-m", "Kontron", "986LCD-M", 0, OK, board_kontron_986lcd_m},
Not sure what happened here. The patch description only talks about the A8JM, but the patch also changes Kontron 986LCD-M matching. Was this intentional? I CC'ed Stefan Reinauer as he might still have access to the Kontron board?
Note that there are at least two different versions of that board with different IDs, I had access to one at some point which was different from Stefan's (not sure which IDs differed how, I'll try to find out if I saved some lspci of the board).
Uwe.
Not intentional.. fix attached. Signed-off-by: Sean Nelson audiohacked@gmail.com