From: marcel.ziswiler@toradex.com (Marcel Ziswiler)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] arm: tegra: update default configuration
Date: Fri, 28 Aug 2015 20:30:35 +0000 [thread overview]
Message-ID: <1440793833.2795.57.camel@toradex.com> (raw)
In-Reply-To: <20150828165132.GQ12027@sirena.org.uk>
On Fri, 2015-08-28 at 16:51 +0000, Mark Brown wrote:
> Please fix your mailer to word wrap within paragraphs so that your
> mails
> are legible.
Sure, sorry. At times I forget that Android isn't sane in that respect
and can't be used to send mailing list compliant emails.
> > On 27 Aug 2015 16:55, Mark Brown <broonie@kernel.org> wrote:
>
> > > Does the system really have the wm9712 wired up as a battery
> > > monitor?
>
> > Well, remember we are talking about a system on module. Those
> > WM9712
> > pins all go to the 200 pin SODIMM edge connector and I know that
> > certain customers indeed have carrier boards that use it that way.
> > If
> > there is any opposition and considering this being rather optional
> > I
> > may also drop this one but audio/touch is definitely a standard
> > feature available on all or carrier boards.
>
> Does that actually work with mainline? The driver requires platform
> data to do anything useful (there's no DT bindings for it) and I
> can't
> see anything providing that on Tegra systems.
To be honest I have not really tried it resp. I did actually notice it
bailing out but haven't had time to look closer into it yet. We are
using it with NVIDIA's still platform data based downstream L4T R16.5
and I do know that it works there so I just kind of assumed there will
be a way to make it work in mainline as well. However rather than
adding such platform data I guess migrating it to DT would make more
sense. But until such a migration has been completed I guess we rather
just drop enabling it for now and I will send out a v2 in that respect.
next prev parent reply other threads:[~2015-08-28 20:30 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <986a8cee-f090-4888-91f2-7b564c4b3d1d@email.android.com>
2015-08-28 16:51 ` [PATCH] arm: tegra: update default configuration Mark Brown
2015-08-28 20:30 ` Marcel Ziswiler [this message]
2015-08-27 10:10 Marcel Ziswiler
2015-08-27 14:54 ` Mark Brown
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=1440793833.2795.57.camel@toradex.com \
--to=marcel.ziswiler@toradex.com \
--cc=linux-arm-kernel@lists.infradead.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;
as well as URLs for NNTP newsgroup(s).