From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bjorn Helgaas Subject: Re: [Fwd: Re: [PATCH] acerhdf: Limit modalias matching to supported boards] Date: Mon, 5 Oct 2009 09:19:49 -0600 Message-ID: <200910050919.51581.bjorn.helgaas@hp.com> References: <4AC9E0F3.5030605@canonical.com> <4AC9E122.8050609@canonical.com> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-15" Content-Transfer-Encoding: 7bit Return-path: Received: from g1t0029.austin.hp.com ([15.216.28.36]:24841 "EHLO g1t0029.austin.hp.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751262AbZJEPUa (ORCPT ); Mon, 5 Oct 2009 11:20:30 -0400 In-Reply-To: <4AC9E122.8050609@canonical.com> Content-Disposition: inline Sender: linux-acpi-owner@vger.kernel.org List-Id: linux-acpi@vger.kernel.org To: Stefan Bader Cc: lenb@kernel.org, "linux-acpi@vger.kernel.org" , Borislav Petkov , Peter Feuerer On Monday 05 October 2009 06:05:54 am Stefan Bader wrote: > >>>> wrote: > >>>>>>> I search my acerhdf-inbox for the people who sent me the BIOS > >>>>>>> versions, > >>>>>> maybe they still have the netbook and can me send the modalias > >>>>>> content. I'll > >>>>>> hopefully find some time upcoming weekend. > >>>>>> > >>>>>> But, can't we simply assume, following dmi lines? > >>>>>> > >>>>>> MODULE_ALIAS("dmi:*:*Acer*:*AOA*:"); > >>>>>> MODULE_ALIAS("dmi:*:*Gateway*:*AOA*:"); > >>>>>> MODULE_ALIAS("dmi:*:*Packard Bell*:*AOA*:"); > >>>>>> MODULE_ALIAS("dmi:*:*Packard Bell*:*DOA*:"); > >>>>>> > >>>>>> We have exactly those constellations within the BIOS settings > >>>>>> table. If I > >>>>>> interpret the modalias line correctly, "pnAOA110" means "Product Name > >>>>>> AOA110" and that's what we have already in the BIOS settings > >>>>>> table. So I > >>>>>> think we do already have all information we need to create a > >>>>>> complete patch > >>>>>> for the problem, or am I wrong? > >>>>> > >>>>> Right pn means that. I would strongly believe the above changes > >>>>> should be > >>>>> good. I sometimes tend to be over-cautious. Attaching a refreshed > >>>>> patch > >>>>> which includes them all. > >>>> > >>>> Well, acerhdf.c and dmi-id.c both do > >>>> dmi_get_system_info(DMI_PRODUCT_NAME) when querying product name which > >>>> means that the product names in the acerhdf table and the dmi-id ones > >>>> are actually one and the same thing. This smells like a maintenance problem. The driver only works on the specific machines and BIOS versions compiled into it. That means you have to update the driver every time a new machine comes out. A major goal of ACPI is to avoid stuff like that. Are you sure there's no way to make this driver work in a generic way, e.g., by claiming an ACPI device? I can't believe that Windows would tolerate a solution like this, although I guess maybe people are using to having to install Windows, then install random OEM-specific junk on top to get full functionality, and OEMs can probably do whatever ugly things they want there. Or is this another case of "the OS is supposed to use WMI to do this, and no, you can't have the WMI information"? Bjorn