[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <cover.1303408522.git.bengardiner@nanometrics.ca>
Date: Thu, 21 Apr 2011 14:19:00 -0400
From: Ben Gardiner <bengardiner@...ometrics.ca>
To: Liam Girdwood <lrg@...mlogic.co.uk>, alsa-devel@...a-project.org
Cc: Mark Brown <broonie@...nsource.wolfsonmicro.com>,
Jaroslav Kysela <perex@...ex.cz>, Takashi Iwai <tiwai@...e.de>,
Sekhar Nori <nsekhar@...com>,
davinci-linux-open-source@...ux.davincidsp.com,
James Nuss <jamesnuss@...ometrics.ca>,
linux-kernel@...r.kernel.org
Subject: [PATCH 0/4] davinci-mcasp: fix tdm_slots and CBM/CFS
This patch series is comprised of three bugfixes and one cleanup that
were performed during prototyping of McASP operation in codec clock-
master frame-slave mode.
First we noticed that the check of the number of tdm slots requested
by platform data was always returning true -- unrelated to CMB/CFS
mode.
Then a cleanup: the PDIR values set are currently based on magic
numbers and there are available bitfield definitions. Not strictly
needed but it makes the changes introduced in the last patch simpler
to read.
It was found that the hardware parameters assigned when
codec clock-master frame-slave is requested were incorrect. This
change is required for correct CBM/CFS operation.
Finally, the direction of the pins is corrected to reflect the
implications of codec clock-master frame-slave -- i.e. mcasp clock-
input frame-output. This change is also required for correct operation.
The combination was tested with a logic analyzer and the hrtimer pwm
device from Bill Gatliff's PWM framework [1] on a da850evm with
hardware modifications to access the McASP lines.
[1] http://article.gmane.org/gmane.linux.kernel.embedded/3486/
Ben Gardiner (4):
davinci-mcasp: correct tdm_slots limit
davinci-mcasp: use bitfield definitions for PDIR
davinci-mcasp: fix _CBM_CFS hw_params
davinci-mcasp: fix _CBM_CFS pin directions
sound/soc/davinci/davinci-mcasp.c | 19 ++++++++++++-------
1 files changed, 12 insertions(+), 7 deletions(-)
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists