Two new acpi modules, acpi_wmi and acpi_hp
Michael
freebsdusb at bindone.de
Sun Jun 21 17:03:06 UTC 2009
Rui Paulo wrote:
> On 21 Jun 2009, at 17:52, Michael wrote:
>
>> Rui Paulo wrote:
>>> On 21 Jun 2009, at 15:08, Rui Paulo wrote:
>>>
>>>> On 21 Jun 2009, at 12:20, Michael wrote:
>>>>
>>>>> Hello,
>>>>>
>>>>> I wrote two new acpi modules last year and finally found the time
>>>>> to fix
>>>>> them, add some missing features and write man pages.
>>>>
>>>> Michael, this is great work! I'll review and see if we can ship it
>>>> with 8.0!
>>>
>>> Michael, this looks good, but there are two style nits that we need
>>> to fix.
>>> * we should try not going over the 80 column mark
>>> * add a empty new line before every return
>>>
>>> If you could send me a new patch for 8.0 with these fixes, I'll commit
>>> it (unless someone else has objections).
>>>
>>> Regards,
>>> --
>>> Rui Paulo
>>>
>>
>> Hi Rui,
>>
>> I figured that adding newlines in conditional returns makes no sense
>> (see my last post), so I did reformat everything to be (close to) 80
>> columns a line add added newlines in front of returns at the end of a
>> function.
>
> Yeah, forget about the newlines before return.
>
>> Please find attached the complete patch again (against clean
>> 7.2-RELEASE) and the files that changed (acpi_hp.c and acpi_wmi.c).
>
> Can you send it in MIME format or store it at some webserver so I could
> fetch it?
>
>> Btw, what is the normal adoption path for 7-STABLE (I would assume first
>> it gets into HEAD and if there's interest there will be a backport to 7?
>> It should at least work in there, even so for HP laptops my guess is
>> that most users will use 8 anyway because of wireless support).
>
>
> It first goes into 8.0 and then 7-STABLE.
>
> Thanks,
> --
> Rui Paulo
>
Hmm, that should have been MIME Format (are you using MacMail by any
chance - seen issues there before..)
I attached it again as an archive, does that work for you?
cheers
Michael
-------------- next part --------------
A non-text attachment was scrubbed...
Name: files_for_rui.tgz
Type: application/octet-stream
Size: 52022 bytes
Desc: not available
Url : http://lists.freebsd.org/pipermail/freebsd-acpi/attachments/20090621/21886a78/files_for_rui.obj
More information about the freebsd-acpi
mailing list