[alsa-devel] [PATCH] FSL_SSI: fix fsl_ssi interrupt reporting so that interrupts are only reported when they have been enabled.

Caleb Crome caleb at crome.org
Wed Oct 28 21:47:58 CET 2015


On Wed, Oct 28, 2015 at 1:25 PM, Fabio Estevam <festevam at gmail.com> wrote:
> On Wed, Oct 28, 2015 at 6:18 PM, Caleb Crome <caleb at crome.org> wrote:
>> On Wed, Oct 28, 2015 at 1:08 PM, Fabio Estevam <festevam at gmail.com> wrote:
>>> Hi Caleb,
>>>
>>> On Wed, Oct 28, 2015 at 5:53 PM, Caleb Crome <caleb at crome.org> wrote:
>>>
>>>> +// call fsl_ssi_dbg_update_sier_enbled every time sier is
>>>> +// updated.  it tracks which interrupts have ever been enabled.
>>>> +// so that the print function can keep track.
>>>
>>> The standard style for multi-line comments is:
>>>
>>> /*
>>>  * Bla bla bla
>>>  * bla bla
>>>  * foo foo
>>>  */
>> Ah, thanks.  Will fix it and resubmit.
>
> Some more tips:
>
> - The Subject should be something like: ASoC: fsl_ssi: Fix fsl_ssi
> interrupt reporting
>
> - Then run  ./scripts/checkpatch.pl 0001-yourpatch.patch to see
> potential issues. The // comment issue would be reported by checkpatch
>
> - Also run  ./scripts/get_maintainer.pl 0001-yourpatch.patch so that
> you get the correct people to Cc the patch too. The email address you
> put for Timur is no longer valid and Nicolin must be on Cc.
>
> Regards,
>
> Fabio Estevam

Great.  Thank you, that will make things go more smoothly for me.

Thanks,
 -Caleb


More information about the Alsa-devel mailing list