From: "Mika Penttilä" <mika.penttila@nextfour.com>
To: Fabio Estevam <festevam@gmail.com>
Cc: "aisheng.dong@nxp.com" <aisheng.dong@nxp.com>,
"shawnguo@kernel.org" <shawnguo@kernel.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: Regression: i.MX6: pinctrl: fsl: add scu based pinctrl support
Date: Mon, 14 Jan 2019 17:17:36 +0000 [thread overview]
Message-ID: <8b6b2967-9ca7-b7dc-49c6-1cd5cd77851f@nextfour.com> (raw)
In-Reply-To: <CAOMZO5CB_b3QbPQRPEW-0ybU1j1Vg1qah0xZhxynf-5d13-ogg@mail.gmail.com>
Hi!
On 14.1.2019 18.58, Fabio Estevam wrote:
> Hi Mika,
>
> On Mon, Jan 14, 2019 at 8:21 AM Mika Penttilä
> <mika.penttila@nextfour.com> wrote:
>> Hello,
>>
>>
>> The patch titled "pinctrl: fsl: add scu based pinctrl support" causes
>> regression on i.MX6. Tested on a custom board based on
>> i.MX6Q and sgtl5000 codec, there is no sound, tested with simple wav
>> playing.
>>
>> Reverting the patch makes audio work again.
> Which kernel version do you use?
>
> Most likely this has already been fixed by the following commit:
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/drivers/pinctrl/freescale/pinctrl-imx.c?h=v5.0-rc2&id=571610678bf344006ab4c47c6fd0a842e9ac6a1b
>
> Please let us know if you still have issues with such commit applied.
I am merging mainline and this regression is present in 5.0-rc2 so it should have the mentioned patch applied.
Obviously I had to revert them both ("pinctrl: imx: fix NO_PAD_CTL setting for MMIO pads"
and "pinctrl: fsl: add scu based pinctrl support"), and after that it works. Is there something more I could test?
--Mika
next prev parent reply other threads:[~2019-01-14 17:17 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-01-14 10:21 Mika Penttilä
2019-01-14 16:58 ` Fabio Estevam
2019-01-14 17:17 ` Mika Penttilä [this message]
2019-01-14 17:35 ` Fabio Estevam
2019-01-18 8:29 ` Aisheng Dong
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=8b6b2967-9ca7-b7dc-49c6-1cd5cd77851f@nextfour.com \
--to=mika.penttila@nextfour.com \
--cc=aisheng.dong@nxp.com \
--cc=festevam@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=shawnguo@kernel.org \
/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
all inboxes | Powered by JetHome®