Commit 5920a29d authored by Pierre-Louis Bossart's avatar Pierre-Louis Bossart Committed by Vinod Koul
Browse files

soundwire: stream: remove useless initialization 



Cppcheck complains about possible null pointer dereferences, but it's
more like there are unnecessary initializations before walking
through a list.

Signed-off-by: default avatarPierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Reviewed-by: default avatarGuennadi Liakhovetski <guennadi.liakhovetski@linux.intel.com>
Reviewed-by: default avatarRander Wang <rander.wang@intel.com>
Reviewed-by: default avatarKai Vehmanen <kai.vehmanen@linux.intel.com>
Signed-off-by: default avatarBard Liao <yung-chuan.liao@linux.intel.com>
Link: https://lore.kernel.org/r/20210302091122.13952-11-yung-chuan.liao@linux.intel.com


Signed-off-by: default avatarVinod Koul <vkoul@kernel.org>
parent a5943e4f

drivers/soundwire/stream.c

+2 −2
Original line number Diff line number Diff line
@@ -261,7 +261,7 @@ static int sdw_program_master_port_params(struct sdw_bus *bus, 
 */

static int sdw_program_port_params(struct sdw_master_runtime *m_rt)

{

	struct sdw_slave_runtime *s_rt = NULL;

	struct sdw_slave_runtime *s_rt;

	struct sdw_bus *bus = m_rt->bus;

	struct sdw_port_runtime *p_rt;

	int ret = 0;
@@ -1470,7 +1470,7 @@ static void sdw_acquire_bus_lock(struct sdw_stream_runtime *stream) 
 */

static void sdw_release_bus_lock(struct sdw_stream_runtime *stream)

{

	struct sdw_master_runtime *m_rt = NULL;

	struct sdw_master_runtime *m_rt;

	struct sdw_bus *bus = NULL;



	/* Iterate for all Master(s) in Master list */