On Mon, Nov 13, 2017 at 07:22:30PM +0100, SF Markus Elfring wrote: > - > - if ( (cpcm->pcm_channel = cs46xx_dsp_create_pcm_channel (chip, sample_rate, cpcm, > - cpcm->hw_buf.addr, > - cpcm->pcm_channel_id)) == NULL) { > + cpcm > + ->pcm_channel = cs46xx_dsp_create_pcm_channel(chip, ^^^^^^^^^^^^^ Please don't break it up like this > + sample_rate, > + cpcm, > + cpcm->hw_buf.addr, > + cpcm > + ->pcm_channel_id); ^^^^^^^^^^^^^^^^ or this. It's better to go over 80 characters than to do that. regards, dan carpenter -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html