linux-i2c.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Dominik Brodowski <linux@dominikbrodowski.net>
To: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Cc: tim@krieglstein.org, jarkko.nikula@linux.intel.com,
	preid@electromag.com.au, wsa@the-dreams.de,
	linux-i2c@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: i2c-designware: sound and s2ram broken on Broadwell-U system since commit ca382f5b38f367b6 (add i2c gpio recovery option)
Date: Sun, 18 Feb 2018 10:41:35 +0100	[thread overview]
Message-ID: <20180218094135.GA2459@light.dominikbrodowski.net> (raw)
In-Reply-To: <1518901021.22495.383.camel@linux.intel.com>

On Sat, Feb 17, 2018 at 10:57:01PM +0200, Andy Shevchenko wrote:
> On Sat, 2018-02-17 at 17:58 +0100, Dominik Brodowski wrote:
> > On Sat, Feb 17, 2018 at 05:09:40PM +0200, Andy Shevchenko wrote:
> 
> > This patch fixes the issue, thanks.
> 
> I will send it soon out.
> 
> >  Do you still need the DEBUG_GPIO output?
> 
> It would be nice to have, though if it makes difficulties to you, then
> don't bother.

Note that I have to enable CONFIG_GPIOLIB before I can enable DEBUG_GPIO;
CONFIG_PINCTRL was off as well. Maybe some "SELECT" is missing for
CONFIG_I2C_DESIGNWARE_BAYTRAIL ?

The following output is with your patch applied. Do you need it without
your fixup patch as well?

$ dmesg | egrep -i "(gpio|pinctrl|i2c|broadw)"
[    0.591578] calling  dw_i2c_init_driver+0x0/0x13 @ 1
[    0.625040] i2c_designware INT3432:00: GPIO lookup for consumer scl
[    0.625073] i2c_designware INT3432:00: using ACPI for GPIO lookup
[    0.625102] acpi INT3432:00: GPIO: looking up scl-gpios
[    0.625129] acpi INT3432:00: GPIO: looking up scl-gpio
[    0.625154] i2c_designware INT3432:00: using lookup tables for GPIO lookup
[    0.625202] i2c_designware INT3432:00: lookup for GPIO scl failed
[    0.678297] i2c_designware INT3433:00: GPIO lookup for consumer scl
[    0.678326] i2c_designware INT3433:00: using ACPI for GPIO lookup
[    0.678353] acpi INT3433:00: GPIO: looking up scl-gpios
[    0.678376] acpi INT3433:00: GPIO: looking up scl-gpio
[    0.678398] i2c_designware INT3433:00: using lookup tables for GPIO lookup
[    0.678427] i2c_designware INT3433:00: lookup for GPIO scl failed
[    0.701403] initcall dw_i2c_init_driver+0x0/0x13 returned 0 after 107421 usecs
[    1.711543] calling  gpio_clk_driver_init+0x0/0x13 @ 1
[    1.712419] initcall gpio_clk_driver_init+0x0/0x13 returned 0 after 44 usecs
[    1.933569] calling  synaptics_i2c_driver_init+0x0/0x13 @ 1
[    1.933602] initcall synaptics_i2c_driver_init+0x0/0x13 returned 0 after 30 usecs
[    1.933996] calling  rmi_i2c_driver_init+0x0/0x13 @ 1
[    1.934060] initcall rmi_i2c_driver_init+0x0/0x13 returned 0 after 59 usecs
[    1.935852] calling  i2c_i801_init+0x0/0xc9 @ 1
[    1.957452] initcall i2c_i801_init+0x0/0xc9 returned 0 after 21089 usecs
[    1.977716] calling  i2c_hid_driver_init+0x0/0x13 @ 1
[    2.025330] input: DLL0665:01 06CB:76AD Touchpad as /devices/pci0000:00/INT3433:00/i2c-1/i2c-DLL0665:01/0018:06CB:76AD.0001/input/input9
[    2.025853] hid-multitouch 0018:06CB:76AD.0001: input: I2C HID v1.00 Mouse [DLL0665:01 06CB:76AD] on i2c-DLL0665:01
[    2.026376] initcall i2c_hid_driver_init+0x0/0x13 returned 0 after 47514 usecs
[    2.065027] calling  max98090_i2c_driver_init+0x0/0x13 @ 1
[    2.065075] initcall max98090_i2c_driver_init+0x0/0x13 returned 0 after 44 usecs
[    2.065077] calling  rt286_i2c_driver_init+0x0/0x13 @ 1
[    2.118805] initcall rt286_i2c_driver_init+0x0/0x13 returned 0 after 52461 usecs
[    2.118809] calling  rt5640_i2c_driver_init+0x0/0x13 @ 1
[    2.118853] initcall rt5640_i2c_driver_init+0x0/0x13 returned 0 after 38 usecs
[    2.118856] calling  rt5651_i2c_driver_init+0x0/0x13 @ 1
[    2.118894] initcall rt5651_i2c_driver_init+0x0/0x13 returned 0 after 34 usecs
[    2.165208] calling  broadwell_audio_init+0x0/0x13 @ 1
[    2.167941] broadwell-audio broadwell-audio: snd-soc-dummy-dai <-> System Pin mapping ok
[    2.168171] broadwell-audio broadwell-audio: snd-soc-dummy-dai <-> Offload0 Pin mapping ok
[    2.168377] broadwell-audio broadwell-audio: snd-soc-dummy-dai <-> Offload1 Pin mapping ok
[    2.168575] broadwell-audio broadwell-audio: snd-soc-dummy-dai <-> Loopback Pin mapping ok
[    2.170833] broadwell-audio broadwell-audio: rt286-aif1 <-> snd-soc-dummy-dai mapping ok
[    2.179624] input: broadwell-rt286 Headset as /devices/pci0000:00/INT3438:00/broadwell-audio/sound/card1/input15
[    2.180363] initcall broadwell_audio_init+0x0/0x13 returned 0 after 14793 usecs

Thanks,
	Dominik

      reply	other threads:[~2018-02-18  9:48 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-02-17 13:25 i2c-designware: sound and s2ram broken on Broadwell-U system since commit ca382f5b38f367b6 (add i2c gpio recovery option) Dominik Brodowski
2018-02-17 15:02 ` Andy Shevchenko
2018-02-17 15:09   ` Andy Shevchenko
2018-02-17 16:58     ` Dominik Brodowski
2018-02-17 20:57       ` Andy Shevchenko
2018-02-18  9:41         ` Dominik Brodowski [this message]

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=20180218094135.GA2459@light.dominikbrodowski.net \
    --to=linux@dominikbrodowski.net \
    --cc=andriy.shevchenko@linux.intel.com \
    --cc=jarkko.nikula@linux.intel.com \
    --cc=linux-i2c@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=preid@electromag.com.au \
    --cc=tim@krieglstein.org \
    --cc=wsa@the-dreams.de \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).