All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mario Limonciello <mario_limonciello@dell.com>
To: Jason Ekstrand <jason@jlekstrand.net>
Cc: "Benjamin Tissoires" <benjamin.tissoires@gmail.com>,
	"Pali Rohár" <pali.rohar@gmail.com>,
	"Dmitry Torokhov" <dmitry.torokhov@gmail.com>,
	LKML <linux-kernel@vger.kernel.org>,
	"linux-input@vger.kernel.org" <linux-input@vger.kernel.org>,
	Rob <robr.bensson@gmail.com>
Subject: Re: [PATCH] Add a quirk for the Dell XPS 13 (2015) when in PS/2 mode.
Date: Mon, 16 Mar 2015 13:50:01 -0500	[thread overview]
Message-ID: <550725D9.20805@dell.com> (raw)
In-Reply-To: <CAOFGe97na_mw-_AHJn6zyxVb-kDAxntjEgvXRcy6eg7uPfhQ8A@mail.gmail.com>


On 03/16/2015 12:10 PM, Jason Ekstrand wrote:
> On Mon, Mar 16, 2015 at 7:29 AM, Mario Limonciello
> <mario_limonciello@dell.com> wrote:
> It's nothing about the wireless.  I swapped it out for an intel card
> on day 3 or so.
>
> Yes, I am almost 100% sure that this affects suspend/resume.  Prior to
> the _REV hack, my laptop *never* suspended correctly.  Now it
> regularly suspends just fine.  I also double-checked that rebasing on
> Linus' master didn't affect it.  With the _REV hack it doesn't
> suspend.  Without it, it's just fine.
>
> I don't know much about suspend/resume but is it possible that the
> problem is the kernel not being able to properly shut down the sound
> card?
Thanks for that confirmation.  That does sound like a very likely candidate that it was because the audio device was in the wrong state.

> That matches what I've seen.  With the _REV hack, I see nothing about
> HDA in dmesg and I get different audio messages than with an _OSI of
> Windows 2009.
>
> I look forward to seeing the update.  Thanks for working on this!
> --Jason Ekstrand
Sure.  In in the interim, here is the patch you can apply to your local DSDT and put in the initramfs to see what it will look like w/ A02.  This will leave the touchpad in I2C and audio in HDA.

--- DSDT.A01    2015-02-18 18:12:44.891286890 -0600
+++ DSDT.A02    2015-03-03 23:17:24.075746660 -0600
@@ -21567,6 +21567,7 @@
                  {
                      ACOS = 0x80
                      ACSE = Zero
+                    OSTP = One
                  }
              }
          }


  reply	other threads:[~2015-03-16 18:50 UTC|newest]

Thread overview: 44+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-02-19  1:43 [PATCH] Add a quirk for the Dell XPS 13 (2015) when in PS/2 mode Mario Limonciello
2015-02-19 17:16 ` Dmitry Torokhov
2015-02-19 18:16   ` Mario Limonciello
2015-02-20 18:47     ` Dmitry Torokhov
2015-02-20 19:24       ` Pali Rohár
2015-02-20 19:56         ` Mario Limonciello
2015-02-20 19:56           ` Mario Limonciello
2015-02-20 20:41           ` Pali Rohár
2015-02-20 21:21             ` Mario Limonciello
2015-02-20 21:21               ` Mario Limonciello
2015-02-20 21:31               ` Benjamin Tissoires
2015-02-20 21:40                 ` Mario Limonciello
2015-02-20 21:46                   ` Pali Rohár
2015-02-20 21:54                     ` Mario Limonciello
2015-02-20 21:54                       ` Mario Limonciello
2015-02-22 16:55               ` Pali Rohár
2015-02-23 23:31                 ` Mario Limonciello
2015-02-24  0:01                   ` Pali Rohár
2015-02-25 18:16                     ` [SUSPECT SPAM] " Mario Limonciello
2015-02-24  0:51                 ` Dmitry Torokhov
2015-02-25 18:26                   ` Mario Limonciello
2015-02-25 18:48             ` Mario Limonciello
2015-02-25 20:45               ` Pali Rohár
2015-03-14 19:17                 ` Benjamin Tissoires
2015-03-16 14:29                   ` Mario Limonciello
2015-03-16 14:40                     ` Benjamin Tissoires
2015-03-16 17:10                     ` Jason Ekstrand
2015-03-16 18:50                       ` Mario Limonciello [this message]
2015-03-16 20:42                         ` Jason Ekstrand
2015-03-16 20:50                           ` Mario Limonciello
2015-03-16 20:57                             ` Jason Ekstrand
2015-03-16 21:07                               ` Benjamin Tissoires
2015-03-17  0:45                                 ` Mario Limonciello
2015-04-10 22:39                 ` Pali Rohár
2015-04-10 23:07                   ` Mario Limonciello
2015-04-10 23:07                     ` Mario Limonciello
2015-04-10 23:14                     ` Pali Rohár
2015-04-10 23:32                       ` Mario Limonciello
2015-04-10 23:32                         ` Mario Limonciello
2015-04-11  2:19                         ` Ben Gamari
2015-04-11  2:19                           ` Ben Gamari
2015-04-13 18:55                           ` Mario Limonciello
2015-03-21 17:21           ` Ben Gamari
2015-03-21 17:21             ` Ben Gamari

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=550725D9.20805@dell.com \
    --to=mario_limonciello@dell.com \
    --cc=benjamin.tissoires@gmail.com \
    --cc=dmitry.torokhov@gmail.com \
    --cc=jason@jlekstrand.net \
    --cc=linux-input@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=pali.rohar@gmail.com \
    --cc=robr.bensson@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.