[#] Поздравьте с первым патчем :)
soko1(lenina,128) — All
2014-07-25 18:33:41


Как я ранее писал, нашёл патч для исправления проблемы постоянной работы кулера при простое на ThinkPad'ах и с ним отказывалось собираться ядро.
Сначала думал возложить эту проблему на знающих людей, а потом попробовал сам и...получилось :)
Сегодня закоммичу сюда : https://github.com/soko1/openbsd-acpithinkpad

[#] Re: Поздравьте с первым патчем :)
51t(lenina,1) — soko1
2014-07-25 18:34:41


С первым патчем! :) Respect!

[#] Re: Поздравьте с первым патчем :)
vaskas(lenina,132) — soko1
2014-07-25 18:55:48


Молодец! Похоже, что именно в этот момент я отправлял тебе pull request: https://github.com/soko1/openbsd-acpithinkpad/pull/1.
Я на x230t пока особой разницы не заметил, кроме одного момента: во время старта системы и выхода из suspend'a кулер резко разгоняется, чтобы потом быстро успокоиться.

[#] Re: Поздравьте с первым патчем :)
soko1(lenina,128) — vaskas
2014-07-25 19:07:43


У меня если честно тоже нифига не поменялось (ща с твоим соберу, но по видимому бессмысленно) :) На эмоциях написал, потому что собралось. А что не работает только как перезагрузился увидел.

Ну и да, спасибо за патч на патч в любом случае :)

[#] Re: Поздравьте с первым патчем :)
zhuk@(lenina,131) — soko1
2014-07-25 20:41:27


> У меня если честно тоже нифига не поменялось (ща с твоим соберу, но по видимому бессмысленно) :) На эмоциях написал, потому что собралось. А что не работает только как перезагрузился увидел.
>
> Ну и да, спасибо за патч на патч в любом случае :)

У меня тоже есть патчик к этому файлу, по другому поводу только, и делающий комп, наоборот, более шумным. :) Надо будет как-то скоординироваться после снятия релиз-лока...

[#] Re: Поздравьте с первым патчем :)
soko1(lenina,128) — vaskas
2014-07-26 00:47:21


С твоим патчем (https://github.com/vaskas/openbsd-acpithinkpad) вот с какими ошибками вылетает (amd4, 5.6-beta):

../../../../dev/acpi/acpithinkpad.c:151: error: redefinition of 'acpithinkpad_models_to_help'
../../../../dev/acpi/acpithinkpad.c:130: error: previous definition of 'acpithinkpad_models_to_help' was here
../../../../dev/acpi/acpithinkpad.c:155: error: redefinition of 'thinkpad_brightness_help'
../../../../dev/acpi/acpithinkpad.c:134: error: previous definition of 'thinkpad_brightness_help' was here
../../../../dev/acpi/acpithinkpad.c:159: error: redefinition of 'min_fan_temps'
../../../../dev/acpi/acpithinkpad.c:138: error: previous definition of 'min_fan_temps' was here
../../../../dev/acpi/acpithinkpad.c:167: error: redefinition of 'thinkpad_staged_fan'
../../../../dev/acpi/acpithinkpad.c:146: error: previous definition of 'thinkpad_staged_fan' was here
../../../../dev/acpi/acpithinkpad.c: In function 'thinkpad_hotkey':
../../../../dev/acpi/acpithinkpad.c:447: error: 'THINKPAD_BUTTON_FN_F6' undeclared (first use in this function)
../../../../dev/acpi/acpithinkpad.c:447: error: (Each undeclared identifier is reported only once
../../../../dev/acpi/acpithinkpad.c:447: error: for each function it appears in.)
../../../../dev/acpi/acpithinkpad.c: At top level:
../../../../dev/acpi/acpithinkpad.c:630: error: redefinition of 'thinkpad_set_fan_level'
../../../../dev/acpi/acpithinkpad.c:623: error: previous definition of 'thinkpad_set_fan_level' was here
*** Error 1 in /usr/src/sys/arch/amd64/compile/GENERIC.MP (Makefile:931 'acpithinkpad.o')

[#] Re: Поздравьте с первым патчем :)
vaskas(lenina,132) — soko1
2014-07-26 02:25:04


Только что перепроверил openbsd-acpithinkpad_fan_control.diff - у меня все собирается, странно. На всякий случай обновил и acpithinkpad.c.
В любом случае, видимо, твою проблему патч не решает.