<div dir="ltr">Hello<br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Dec 9, 2014 at 5:29 PM, Garreau, Alexandre <span dir="ltr"><<a href="mailto:galex-713@galex-713.eu" target="_blank">galex-713@galex-713.eu</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="">
</span>Oh wait, sorry if I only get the thread here and I lost the beginning,<br>
but, does that mean you found how to make screen position (tablet or<br>
laptop mode) finally detectable through ACPI? ’cos I still didn’t<br>
understand why it didn’t.<br></blockquote><div><br></div><div>Yes, I did. Never give up, never surrender.<br><br>It's hackish but it works. It didn't before, because there is a missing entry in the DSDT : MKHG, which thinkpad-acpi test if it exists before reporting the screen position event.<br><br></div><div>Many such entries are missing (ex: GBDC to control bluetooth from within thinkpad-acpi) but now there are ways around most of them - cf the rfkill message.<br><br></div><div>However, that doesn't mean the DSDT is clean, as Windows still can't be booted. It's a good test because it's picky about DSDTs. It will be a good milestone to reach. FYI, the x201 can boot windows from coreboot according to the wiki.<br></div><div> <br></div><div>If you want screen position ACPI events, I suggest you flash the latest libreboot (it includes φcoder original patch), and edit the thinkpad-acpi file from the kernel as suggested to comment this one line. It is a module in debian, so you only need to recompile it, not the whole kernel.<br><br></div><div>There are various kernel compilation guide for debian that can help you recompile the modules after editing the source file - it's very straightforward. If you are member of a LUG, they can help you too.<br><br></div><div>If you are confortable editing grub config file to deploy a new bzImage, I can send you my kernel binaries and modules (I use 3.14.25), but I believe that would be more complex. I would not recommand that.<br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
If it’s the case, how much time it could take to have needed changes<br>
arriving to Debian testing? is there any way not implying to recompile<br>
the whole kernel to have them before?<br></blockquote><div><br></div><div>Debian uses standard OS components, and I'm sure such an ugly patch would never be accepted. Most people don't use coreboot and don't want spare ACPI events.<br><br>Like most of things I do, it's just a hack, an experiment, to test an hypothesis about the cause of the problem, and see if it can be made to work. <br></div><div><br>The proper way to support that  would be to add the MKHG field to coreboot DSDT, it would be instantaneous when you reboot after having flashed this new version.<br><br>I believe the following should be enough:<br><br>                        Method (MHKG, 0, NotSerialized)<br>                        {<br>                            Store (Zero, Local0)<br>                            ShiftLeft (TMOD, 0x03, Local0)<br>                            Return (Local0)<br>                        }<br><br></div><div><br></div><div>Unfortunately, I won't be able to do more test until the end of next week. Maybe someone else will want to try that and, if it works, submit a patch.<br><br>Charles<br></div></div></div></div>