[alsa-devel] [PATCH v3] ARM: kirkwood: extend the kirkwood i2s driver for DT usage

[Mark Brown]

On Tue, Jul 23, 2013 at 10:46:15AM +0200, Jean-Francois Moine wrote:

> +	np = pdev->dev.of_node;
> +	if (np) {
> +		struct of_phandle_args clkspec;
> +
> +		priv->burst = 128;			/* might be 32 or 128 */

The comment says this needs to be variable (depending on what?) but it's
hard coded.

> +		priv->clk = of_clk_get(np, 0);	/* internal clock */
> +		err = of_parse_phandle_with_args(np,
> +					"clocks", "#clock-cells", 1,
> +					&clkspec);

As others have pointed out if you need to change the clock get code
there's something wrong here, DT should be handled transparently by the
clock API.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20130723/b7f576c4/attachment.sig>