Linux on ARM based TI OMAP SoCs
 help / color / mirror / Atom feed
* ti,codec property in sound/soc/omap/omap-twl4030.c not used
@ 2015-03-11 22:12 Belisko Marek
  2015-03-12  9:19 ` Peter Ujfalusi
  0 siblings, 1 reply; 3+ messages in thread
From: Belisko Marek @ 2015-03-11 22:12 UTC (permalink / raw)
  To: peter.ujfalusi, jarkko.nikula
  Cc: linux-omap@vger.kernel.org, alsa-devel, Dr. H. Nikolaus Schaller

Hi,

I'm looking at sources for omap-twl4030.c driver which is used by many
boards. In DT bindings documentation is property ti,codec described as
mandatory but it's never parsed in driver code. I did small experiment
in gta04 board and remove ti,codec from devicetree and sound was
working. I think it can be removed from documentation and also from
various device trees. Is my assumption correct? Thanks.

BR,

marek

-- 
as simple and primitive as possible
-------------------------------------------------
Marek Belisko - OPEN-NANDRA
Freelance Developer

Ruska Nova Ves 219 | Presov, 08005 Slovak Republic
Tel: +421 915 052 184
skype: marekwhite
twitter: #opennandra
web: http://open-nandra.com

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: ti,codec property in sound/soc/omap/omap-twl4030.c not used
  2015-03-11 22:12 ti,codec property in sound/soc/omap/omap-twl4030.c not used Belisko Marek
@ 2015-03-12  9:19 ` Peter Ujfalusi
  2015-03-12  9:27   ` Belisko Marek
  0 siblings, 1 reply; 3+ messages in thread
From: Peter Ujfalusi @ 2015-03-12  9:19 UTC (permalink / raw)
  To: Belisko Marek, jarkko.nikula
  Cc: linux-omap@vger.kernel.org, alsa-devel, Dr. H. Nikolaus Schaller

On 03/12/2015 12:12 AM, Belisko Marek wrote:
> Hi,
> 
> I'm looking at sources for omap-twl4030.c driver which is used by many
> boards. In DT bindings documentation is property ti,codec described as
> mandatory but it's never parsed in driver code. I did small experiment
> in gta04 board and remove ti,codec from devicetree and sound was
> working. I think it can be removed from documentation and also from
> various device trees. Is my assumption correct? Thanks.

Correct. The ti,twl4030-audio is for the MFD driver and the device for the
ASoC codec is created in there w/o actual DT compatible string. The audio part
of the twl4030 is responsible for the audio and the vibra functionality.

Are you going to send the patches?

Thanks,
Péter
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: ti,codec property in sound/soc/omap/omap-twl4030.c not used
  2015-03-12  9:19 ` Peter Ujfalusi
@ 2015-03-12  9:27   ` Belisko Marek
  0 siblings, 0 replies; 3+ messages in thread
From: Belisko Marek @ 2015-03-12  9:27 UTC (permalink / raw)
  To: Peter Ujfalusi
  Cc: jarkko.nikula, linux-omap@vger.kernel.org, alsa-devel,
	Dr. H. Nikolaus Schaller

Hi,

On Thu, Mar 12, 2015 at 10:19 AM, Peter Ujfalusi <peter.ujfalusi@ti.com> wrote:
> On 03/12/2015 12:12 AM, Belisko Marek wrote:
>> Hi,
>>
>> I'm looking at sources for omap-twl4030.c driver which is used by many
>> boards. In DT bindings documentation is property ti,codec described as
>> mandatory but it's never parsed in driver code. I did small experiment
>> in gta04 board and remove ti,codec from devicetree and sound was
>> working. I think it can be removed from documentation and also from
>> various device trees. Is my assumption correct? Thanks.
>
> Correct. The ti,twl4030-audio is for the MFD driver and the device for the
> ASoC codec is created in there w/o actual DT compatible string. The audio part
> of the twl4030 is responsible for the audio and the vibra functionality.
Thanks for reply.
>
> Are you going to send the patches?
Yes will do.
>
> Thanks,
> Péter

BR,

marek



-- 
as simple and primitive as possible
-------------------------------------------------
Marek Belisko - OPEN-NANDRA
Freelance Developer

Ruska Nova Ves 219 | Presov, 08005 Slovak Republic
Tel: +421 915 052 184
skype: marekwhite
twitter: #opennandra
web: http://open-nandra.com
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2015-03-12  9:27 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-03-11 22:12 ti,codec property in sound/soc/omap/omap-twl4030.c not used Belisko Marek
2015-03-12  9:19 ` Peter Ujfalusi
2015-03-12  9:27   ` Belisko Marek

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox