[PATCH 5/5] ASoC: Intel: sof_sdw: clarify operator precedence
Pierre-Louis Bossart
pierre-louis.bossart at linux.intel.com
Thu Aug 13 21:43:50 CEST 2020
On 8/13/20 1:45 PM, Mark Brown wrote:
> On Thu, Aug 13, 2020 at 12:58:39PM -0500, Pierre-Louis Bossart wrote:
>
>> - hdmi_num = sof_sdw_quirk & SOF_SDW_TGL_HDMI ?
>> + hdmi_num = (sof_sdw_quirk & SOF_SDW_TGL_HDMI) ?
>> SOF_TGL_HDMI_COUNT : SOF_PRE_TGL_HDMI_COUNT;
>
> Or better yet, just don't abuse the ternery operator like this and write
> normal conditional statements.
I count 795 uses of the ternary operator in sound/soc and 68776 in my
local kernel branch.
Can you clarify in what way this is an abuse? I don't mind changing
this, I wasn't aware this is frowned upon.
More information about the Alsa-devel
mailing list