Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1760759AbXHEUDp (ORCPT ); Sun, 5 Aug 2007 16:03:45 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1757216AbXHEUDg (ORCPT ); Sun, 5 Aug 2007 16:03:36 -0400 Received: from gepetto.dc.ltu.se ([130.240.42.40]:49473 "EHLO gepetto.dc.ltu.se" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757014AbXHEUDf (ORCPT ); Sun, 5 Aug 2007 16:03:35 -0400 Message-ID: <46B62CFE.2010404@student.ltu.se> Date: Sun, 05 Aug 2007 22:03:10 +0200 From: Richard Knutsson User-Agent: Thunderbird 1.5.0.12 (X11/20070719) MIME-Version: 1.0 To: trenn@suse.de CC: Ben Collins , "Brown, Len" , linux-acpi@vger.kernel.org, Stelian Pop , linux-kernel@vger.kernel.org Subject: Re: [PATCH] sonypi: Fix initialization warning References: <46B61F70.90005@student.ltu.se> <1186343443.3858.46.camel@fanta4.site> In-Reply-To: <1186343443.3858.46.camel@fanta4.site> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1133 Lines: 31 Thomas Renninger wrote: > On Sun, 2007-08-05 at 21:05 +0200, Richard Knutsson wrote: > >> Signed-off-by: Richard Knutsson >> --- >> Got this from the compiler (gcc (GCC) 4.1.2 20070626 (Red Hat 4.1.2-13)): >> drivers/char/sonypi.c:1153: warning: initialization from incompatible pointer type >> >> >> diff --git a/drivers/char/sonypi.c b/drivers/char/sonypi.c >> index 73037a4..2dcd519 100644 >> --- a/drivers/char/sonypi.c >> +++ b/drivers/char/sonypi.c >> @@ -1147,10 +1147,15 @@ static int sonypi_acpi_remove(struct acpi_device *device, int type) >> return 0; >> } >> >> +static const struct acpi_device_id sonypi_acpi_driver_ids[] = { >> + {ACPI_PROCESSOR_HID, 0}, >> > This is wrong. You need to take the HID the driver should match for, in > this case: {"SNY6001", 0 }, > Oh bugger, a cut 'n' paste error... Thanks for reviewing it. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/