On Tue, Nov 04, 2008 at 05:22:17PM +0900, ISHIKAWA Mutsumi wrote: > >>>>> In <1225705280-12775-5-git-send-email-malattia@xxxxxxxx> > >>>>> Mattia Dongili <malattia@xxxxxxxx> wrote: > >> From: ISHIKAWA Mutsumi <ishikawa@xxxxxxxxxxx> > >> > >> Signed-off-by: Mattia Dongili <malattia@xxxxxxxx> > >> --- > >> drivers/misc/sony-laptop.c | 7 +++++++ > >> 1 files changed, 7 insertions(+), 0 deletions(-) > > >> diff --git a/drivers/misc/sony-laptop.c b/drivers/misc/sony-laptop.c > >> index c9e5651..19f3306 100644 > >> --- a/drivers/misc/sony-laptop.c > >> +++ b/drivers/misc/sony-laptop.c > >> @@ -1589,6 +1589,13 @@ static void sony_pic_detect_device_type(struct sony_pic_dev *dev) > >> goto out; > >> } > > >> + pcidev = pci_get_device(PCI_VENDOR_ID_INTEL, > >> + PCI_DEVICE_ID_INTEL_ICH9_1, NULL); > >> + if (pcidev) { > >> + dev->control = &spic_types[4]; > > In patch [PATCH 1/7] [sony-laptop] merge Type4 into Type3, > type 3 and type 4 spic models are merged, and type 5 model > I added (of cause It is not problem, it is completly same as > type 4) are dropped. > > So I think spic_types[4] is invalid. This line should be as > bellow (Use Type 3 on ICH9_1 machines): > > dev->control = &spic_types[2]; gosh! /me looks for his emergency brown paperbag Len, I'll resend the patchset in the next days, I'll also see if I can rework the rfkill one to use the proper subsystem. thanks -- mattia :wq! -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html