Remove "Fix src component detection" part from this review, as it moved to a separate one (D21579)

Fix style issues.

Assuming this builds correctly in poudriere (didn't test myself). Thanks!

In D19920#432741, @mat wrote:

In D19920#432640, @mat wrote:

Could you please not rename the port so that pkg upgrade keeps on working.

Mmm, I just saw that the SLAVE_PKGNAMESUFFIX was only set in the "lib" case.

In D14103#431924, @seanc wrote:

Can someone summarize what's preventing this patch from being committed?

In D19920#428152, @crees wrote:

Yeah, I wondered what would happen about the php flavours :(
I'll look into the meta port idea, seems a good one! Or perhaps just a slave? They'll have the same effect.

I like the idea of this - unfortunately this approach kills the existing flavours:

Tested again, new version also works as expected, mptramp_pagetables is 0x7bf77000 on c720

Accepting to confirm that I tested this patch one more time against r338318 on the Acer Chromebook c720 (previous test was against an older version of the kernel). Obviously my feedback isn't sufficient and this still requires review by the original reviewers (aka experts).

@kib Updated by tree to r334733 and applied the patch, it's working as expected, thanks!

Fix typo and extra whitespace

In D14103#300013, @emaste wrote:

Testing with this change on X260 (LEN0068, MHKV 0x0200 AFAICT) I see warnings like:

# sysctl dev.acpi_ibm.0.kbd_backlight=0
ACPI Warning: \_SB.PCI0.LPC.EC.HKEY.MLCG: Insufficient arguments - Caller passed 0, method requires 1 (20180209/nsarguments-383)
ACPI Warning: \_SB.PCI0.LPC.EC.HKEY.MLCG: Insufficient arguments - Caller passed 0, method requires 1 (20180209/nsarguments-383)
dev.acpi_ibm.0.kbd_backlight: 1ACPI Warning: \_SB.PCI0.LPC.EC.HKEY.MLCG: Insufficient arguments - Caller passed 0, method requires 1 (20180209/nsarguments-383)
 -> ACPI Warning: \_SB.PCI0.LPC.EC.HKEY.MLCG: Insufficient arguments - Caller passed 0, method requires 1 (20180209/nsarguments-383)
ACPI Warning: \_SB.PCI0.LPC.EC.HKEY.MLCG: Insufficient arguments - Caller passed 0, method requires 1 (20180209/nsarguments-383)
0
#

Keyboard backlight control works despite the warnings. I haven't tested this machine on an unpatched kernel. echo 1 > /dev/led/micmute has no effect.

Add support for controlling micmute LED

Please note that man page changes are pending, but will be added to the review before committing any changes.

Add support for controlling the keyboard backlight found in newer lenovo models.

@rodrigc I think this can be closed.

@koobs Just glanced at it, trusting your judgement.

Thanks for taking over, I don't have any recent hardware or the time to maintain/test this myself at the moment.

Upstream fixed this in the meantime.

Never mind, apparently I can accept it myself and this is good enough to allow me to close it.

@wblock This has been committed quite some time ago, but as @hrs requested changes, I can't close it (this would only have happened if the svn revision mentioned the review). As it was committed, I would prefer not to abandon it. Any ideas how to solve this?