RE: [PATCH 1/1] ARM: tegra: paz00: Fix board pinmux table.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Marc Dietrich wrote at Saturday, October 15, 2011 10:34 AM:
> Am Samstag 15 Oktober 2011, 17:18:33 schrieb Leon Romanovsky:
> > This fix updates the CDEV1 pinmux for the paz00 board to be as in the
> > Harmony board. Paz00 board is originally based on Harmony design.
> 
> the fact that this patch makes sound work on paz00 does not necessary mean
> that this is the "right thing" to do.

> First, the initial state should be
> TRISTATE, because sound layer should switch between NORMAL/TRISTATE
> automaticly to save some power.

None of the other boards do that, and the drivers don't support it. I
believe the clock would need to be started as soon as the audio driver
module were loaded, and kept running as long as it was loaded, so there's
not much chance of optimizing power here at the moment.

> Second, Android has TEGRA_PUPD_PULL_DOWN
> with sound working and I didn't found out yet why this also works.

I don't know why there's a pull-down there in the Android kernel; there's
no need as far as I know, since it's a clock signal. I'd go so far as to
believe that a pull-down would distort the clock signal slightly.

But yes, it probably makes sense to include this patch in a series that
enables audio on PAZ00 whenever that appears.

-- 
nvpublic

��.n��������+%������w��{.n�����{��נ���^n�r������&��z�ޗ�zf���h���~����������_��+v���)ߣ�

[Index of Archives]     [ARM Kernel]     [Linux ARM]     [Linux ARM MSM]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux