On Thu, Jul 25, 2024 at 01:23:45PM GMT, Krzysztof Kozlowski wrote: > Two arrays (with 'struct sdw_dpn_prop' and 'struct sdw_port_config') > store configuration of Soundwire ports, thus each of their element is > indexed according to the port number (enum wsa884x_port_ids, e.g. > WSA884X_PORT_DAC). Except the indexing, they also store port number > offset by one in member 'num'. > > Entire code depends on that correlation between array index and port > number, thus make it explicit by using designators. The code is > functionally the same, but more obvious for reading. > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx> > --- > sound/soc/codecs/wsa884x.c | 34 ++++++++++++++++++++++------------ > 1 file changed, 22 insertions(+), 12 deletions(-) > Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx> -- With best wishes Dmitry