Hi Gustavo, On Mon, Apr 08, 2019 at 10:20:25PM -0500, Gustavo A. R. Silva wrote: > >>> diff --git a/sound/soc/fsl/fsl_esai.c b/sound/soc/fsl/fsl_esai.c index > >>> c7410bbfd2af..bad0dfed6b68 100644 > >>> --- a/sound/soc/fsl/fsl_esai.c > >>> +++ b/sound/soc/fsl/fsl_esai.c > >>> @@ -251,7 +251,7 @@ static int fsl_esai_set_dai_sysclk(struct > >> snd_soc_dai *dai, int clk_id, > >>> break; > >>> case ESAI_HCKT_EXTAL: > >>> ecr |= ESAI_ECR_ETI; > >> > >> Also, you should use a simple assignment operator "=" instead of "|=" in > >> both cases. > > > > The result is same for "=" and "|=", because there is "ecr = 0" in beginning of > > This function. > > > > Following that same logic, then why not use "+=" instead? > > The point is: is "|=" or any other assignment operator other than "=" necessary? > The answer in this case is: no, it is not. So, go for the simple one and avoid > any unnecessary confusion. I would like to keep "|=" here, just in case that someday it'd be easier to insert something to ecr before this chunk. So please get easy on this one. Thanks Nicolin _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx https://mailman.alsa-project.org/mailman/listinfo/alsa-devel
