From: Peter Ujfalusi <peter.ujfalusi@ti.com>
To: Sebastian Reichel <sre@kernel.org>
Cc: "Mark Rutland" <mark.rutland@arm.com>,
devicetree@vger.kernel.org, "Nishanth Menon" <nm@ti.com>,
"Mauro Carvalho Chehab" <mchehab@kernel.org>,
"Matthijs van Duin" <matthijsvanduin@gmail.com>,
"Tony Lindgren" <tony@atomide.com>,
"Tomi Valkeinen" <tomi.valkeinen@ti.com>,
"Liam Girdwood" <lgirdwood@gmail.com>,
"Rob Herring" <robh+dt@kernel.org>,
"Tero Kristo" <t-kristo@ti.com>,
"Mark Brown" <broonie@kernel.org>,
"Sakari Ailus" <sakari.ailus@iki.fi>,
"Laurent Pinchart" <laurent.pinchart@ideasonboard.com>,
"Benoît Cousson" <bcousson@baylibre.com>,
linux-omap@vger.kernel.org, "Paul Walmsley" <paul@pwsan.com>,
linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 5/7] ARM: dts: Add nodes for missing omap4 interconnect target modules
Date: Thu, 12 Oct 2017 09:07:55 +0300 [thread overview]
Message-ID: <da8d1d7f-f16d-8d71-cf49-acff79614610@ti.com> (raw)
In-Reply-To: <20171011165850.jnaievrgcszcawh3@earth>
Hi Sebastian,
Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki
On 2017-10-11 19:58, Sebastian Reichel wrote:
>> I'll look at the schema of PandaBoard to see if I can route out the McASP
>> lines to somewhere then I can add support for this revision to davinci-mcasp
>> driver.
>
> Not sure if this is helpful information:
>
> Motorola Droid 4 is OMAP4430 based and has McASP lines routed
> to the audio codec (either McASP or McBSP can be used). Codec
> is a custom one, which is not yet supported mainline, but
> patches are WIP (*):
>
> https://lkml.org/lkml/2017/7/12/368
Really cool!
I don't see much benefit of using McASP on OMAP4, only drawbacks... It
lacks AFIFO so it is prone to underrun and if it underruns you need to
reset McASP. Been there with the Samsung Galaxy Nexus.
I wonder how Motorola is using the McASP as the OMAP4 version
(officially) does not support I2S. It might work, I have not tried it.
I do have one Galaxy Nexus, but there the McASP is used for the S/PIDIF
on the dock (the three pins on the side). Yes, I do not have the dock
and my Galaxy Nexus is still in active use -> not rooted, and never run
mainline Linux.
> (*) My patches use McBSP instead of McASP, since McASP looked
> mostly unsupported. Motorola's kernel uses McASP.
If I have something I might ask you for testing, if it is OK.
>
> -- Sebastian
>
- Péter
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
WARNING: multiple messages have this Message-ID (diff)
From: peter.ujfalusi@ti.com (Peter Ujfalusi)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 5/7] ARM: dts: Add nodes for missing omap4 interconnect target modules
Date: Thu, 12 Oct 2017 09:07:55 +0300 [thread overview]
Message-ID: <da8d1d7f-f16d-8d71-cf49-acff79614610@ti.com> (raw)
In-Reply-To: <20171011165850.jnaievrgcszcawh3@earth>
?Hi Sebastian,
Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki
On 2017-10-11 19:58, Sebastian Reichel wrote:
>> I'll look at the schema of PandaBoard to see if I can route out the McASP
>> lines to somewhere then I can add support for this revision to davinci-mcasp
>> driver.
>
> Not sure if this is helpful information:
>
> Motorola Droid 4 is OMAP4430 based and has McASP lines routed
> to the audio codec (either McASP or McBSP can be used). Codec
> is a custom one, which is not yet supported mainline, but
> patches are WIP (*):
>
> https://lkml.org/lkml/2017/7/12/368
Really cool!
I don't see much benefit of using McASP on OMAP4, only drawbacks... It
lacks AFIFO so it is prone to underrun and if it underruns you need to
reset McASP. Been there with the Samsung Galaxy Nexus.
I wonder how Motorola is using the McASP as the OMAP4 version
(officially) does not support I2S. It might work, I have not tried it.
I do have one Galaxy Nexus, but there the McASP is used for the S/PIDIF
on the dock (the three pins on the side). Yes, I do not have the dock
and my Galaxy Nexus is still in active use -> not rooted, and never run
mainline Linux.
> (*) My patches use McBSP instead of McASP, since McASP looked
> mostly unsupported. Motorola's kernel uses McASP.
If I have something I might ask you for testing, if it is OK.
>
> -- Sebastian
>
- P?ter
next prev parent reply other threads:[~2017-10-12 6:07 UTC|newest]
Thread overview: 46+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-09-29 22:34 [PATCHv4 0/7] Fix remaining issues to drop more omap platform data Tony Lindgren
2017-09-29 22:34 ` Tony Lindgren
[not found] ` <20170929223411.9691-1-tony-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org>
2017-09-29 22:34 ` [PATCH 1/7] dt-bindings: bus: Minimal TI sysc interconnect target module binding Tony Lindgren
2017-09-29 22:34 ` Tony Lindgren
2017-10-01 13:11 ` Sebastian Reichel
2017-10-01 13:11 ` Sebastian Reichel
2017-10-01 17:14 ` Tony Lindgren
2017-10-01 17:14 ` Tony Lindgren
[not found] ` <20171001171406.GL4394-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org>
2017-10-01 20:48 ` Sebastian Reichel
2017-10-01 20:48 ` Sebastian Reichel
2017-10-01 21:03 ` Tony Lindgren
2017-10-01 21:03 ` Tony Lindgren
[not found] ` <20170929223411.9691-2-tony-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org>
2017-10-10 15:46 ` Rob Herring
2017-10-10 15:46 ` Rob Herring
2017-10-10 16:45 ` Tony Lindgren
2017-10-10 16:45 ` Tony Lindgren
2017-09-29 22:34 ` [PATCH 2/7] ARM: OMAP2+: Parse module IO range from dts for legacy "ti,hwmods" support Tony Lindgren
2017-09-29 22:34 ` [PATCH 2/7] ARM: OMAP2+: Parse module IO range from dts for legacy "ti, hwmods" support Tony Lindgren
2017-09-29 22:34 ` [PATCH 3/7] ARM: OMAP2+: Populate legacy resources for dma and smartreflex Tony Lindgren
2017-09-29 22:34 ` Tony Lindgren
2017-09-29 22:34 ` [PATCH 5/7] ARM: dts: Add nodes for missing omap4 interconnect target modules Tony Lindgren
2017-09-29 22:34 ` Tony Lindgren
[not found] ` <20170929223411.9691-6-tony-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org>
2017-10-11 15:54 ` Peter Ujfalusi
2017-10-11 15:54 ` Peter Ujfalusi
[not found] ` <f5c57f0b-1423-c362-0fda-4069b77a2e38-l0cyMroinI0@public.gmane.org>
2017-10-11 16:58 ` Sebastian Reichel
2017-10-11 16:58 ` Sebastian Reichel
2017-10-12 6:07 ` Peter Ujfalusi [this message]
2017-10-12 6:07 ` Peter Ujfalusi
[not found] ` <da8d1d7f-f16d-8d71-cf49-acff79614610-l0cyMroinI0@public.gmane.org>
2017-10-12 8:40 ` Matthijs van Duin
2017-10-12 8:40 ` Matthijs van Duin
2017-10-12 9:10 ` Peter Ujfalusi
2017-10-12 9:10 ` Peter Ujfalusi
2017-10-13 16:46 ` Tony Lindgren
2017-10-13 16:46 ` Tony Lindgren
2017-09-29 22:34 ` [PATCH 6/7] ARM: dts: Configure SmartReflex only to idle the interconnect target module Tony Lindgren
2017-09-29 22:34 ` Tony Lindgren
2017-09-29 22:34 ` [PATCH 7/7] ARM: dts: Use ti-sysc module driver for omap4 musb Tony Lindgren
2017-09-29 22:34 ` Tony Lindgren
[not found] ` <20170929223411.9691-8-tony-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org>
2017-10-10 17:01 ` Tony Lindgren
2017-10-10 17:01 ` Tony Lindgren
2017-09-29 22:34 ` [PATCH 4/7] bus: ti-sysc: Add minimal TI sysc interconnect target driver Tony Lindgren
2017-09-29 22:34 ` Tony Lindgren
2017-09-29 22:34 ` Tony Lindgren
2017-10-13 17:51 ` Tony Lindgren
2017-10-13 17:51 ` Tony Lindgren
2017-10-13 17:51 ` Tony Lindgren
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=da8d1d7f-f16d-8d71-cf49-acff79614610@ti.com \
--to=peter.ujfalusi@ti.com \
--cc=bcousson@baylibre.com \
--cc=broonie@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=laurent.pinchart@ideasonboard.com \
--cc=lgirdwood@gmail.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-omap@vger.kernel.org \
--cc=mark.rutland@arm.com \
--cc=matthijsvanduin@gmail.com \
--cc=mchehab@kernel.org \
--cc=nm@ti.com \
--cc=paul@pwsan.com \
--cc=robh+dt@kernel.org \
--cc=sakari.ailus@iki.fi \
--cc=sre@kernel.org \
--cc=t-kristo@ti.com \
--cc=tomi.valkeinen@ti.com \
--cc=tony@atomide.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.