On Wed, Dec 23, 2015 at 02:01:14PM -0500, Alex Deucher wrote:
> + stream = adata->play_stream;
> + rtd = stream ? stream->runtime : NULL;
> + if (rtd != NULL) {
> + /* Resume playback stream from a suspended state */
> + sdata = rtd->private_data;
> + config_acp_dma(adata->acp_mmio, sdata);
> + }
Please don't use the ternery operator like this, it's not bad to write
if statements and they're much more legible. I was having to think far
too much about the various variables and if we were safely handling all
of them before I realised the next block overwrites them all anyway.
This is really
stream = adata->play_stream;
if (stream && stream->rtd)
config_acp_dma(adata->acp_mmio, stream->rtd->private_data);
or a couple of nested if statements. Otherwise this patch looks OK.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: not available
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20160105/8f2e15d4/attachment-0001.sig>
