28 Jun
2017
28 Jun
'17
6:02 p.m.
On Fri, Jun 23, 2017 at 12:35:01PM -0400, Alex Deucher wrote:
- if (adev->asic_type == CHIP_STONEY) {
i2s_pdata[1].quirks = DW_I2S_QUIRK_COMP_REG_OFFSET |
DW_I2S_QUIRK_COMP_PARAM1 |
DW_I2S_QUIRK_16BIT_IDX_OVERRIDE;
- } else {
i2s_pdata[1].quirks = DW_I2S_QUIRK_COMP_REG_OFFSET |
DW_I2S_QUIRK_COMP_PARAM1;
- }
Quirks like this are better written as switch statements since that makes it easier to handle further variants in future.