soundwire: bus: test all write status

In the existing code we don't check all write transactions so may miss
the root cause of an issue.

Make sure all writes are tested and errors not propagated further.

Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Reviewed-by: Guennadi Liakhovetski <guennadi.liakhovetski@intel.com>
Reviewed-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
Reviewed-by: Bard Liao <bard.liao@intel.com>

Author: plbossart

drivers/soundwire/bus.c

@@ -1279,7 +1279,12 @@ static int sdw_initialize_slave(struct sdw_slave *slave)
 		}
 		if (status & SDW_SCP_INT1_BUS_CLASH) {
 			dev_warn(&slave->dev, "Bus clash detected before INT mask is enabled\n");
-			sdw_write_no_pm(slave, SDW_SCP_INT1, SDW_SCP_INT1_BUS_CLASH);
+			ret = sdw_write_no_pm(slave, SDW_SCP_INT1, SDW_SCP_INT1_BUS_CLASH);
+			if (ret < 0) {
+				dev_err(&slave->dev,
+					"SDW_SCP_INT1 write failed:%d\n", ret);
+				return ret;
+			}
 		}
 	}
 	if ((slave->bus->prop.quirks & SDW_MASTER_QUIRKS_CLEAR_INITIAL_PARITY) &&
@@ -1293,7 +1298,13 @@ static int sdw_initialize_slave(struct sdw_slave *slave)
 		}
 		if (status & SDW_SCP_INT1_PARITY) {
 			dev_warn(&slave->dev, "PARITY error detected before INT mask is enabled\n");
-			sdw_write_no_pm(slave, SDW_SCP_INT1, SDW_SCP_INT1_PARITY);
+			ret = sdw_write_no_pm(slave, SDW_SCP_INT1, SDW_SCP_INT1_PARITY);
+			if (ret < 0) {
+				dev_err(&slave->dev,
+					"SDW_SCP_INT1 write failed:%d\n", ret);
+				return ret;
+			}
+
 		}
 	}