From patchwork Wed May 12 14:44:58 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Greg KH X-Patchwork-Id: 435563 Delivered-To: patch@linaro.org Received: by 2002:a02:c901:0:0:0:0:0 with SMTP id t1csp4941424jao; Wed, 12 May 2021 08:37:38 -0700 (PDT) X-Google-Smtp-Source: ABdhPJx7CyQ+vn9S0jPQ6cZvJLpBRJf0020PR0Nx/jPhQ8VIkhbmOl7foV3t4w5mVUIgRf+mfJas X-Received: by 2002:a05:6512:713:: with SMTP id b19mr24887936lfs.125.1620833858561; Wed, 12 May 2021 08:37:38 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1620833858; cv=none; d=google.com; s=arc-20160816; b=aEHnxAMdBNcwlrLarCrs4zCyP8hGshkRiEEni3oyLPAjUP+moqC2FP9B3jzQ1ZnEc8 GUewm3ste65J1IsPEs6xU+L2L4KmJGQv76Z/NDkp6xyD8Syqyxu1xjcymuys42/6lGxp Qk/qjwUmcg1eLnQj5szhX2QB+YQJpdieIRA7/eD+Y5r/H7sFsdn8lBWZ1vIeqzhreKOH ubzeR0KLqtEbDrSJBcvwODXdxX9ZJdB4m0+lRCK3TY2zb60q4ePYbeTB+clAP8A+sJCG ViEAXaukYfL+KqFCbjrHk6KwlBZzMSDmGFHZ2XC29+uRa9o+1J4ogd20JWzwmnvGYt/B YBNA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :user-agent:references:in-reply-to:message-id:date:subject:cc:to :from:dkim-signature; bh=MrWgKe+8SmBV8A5f8BzlBcKiNHZQIvbywVVCx1GEMCQ=; b=EC8JmA1L6+0voXgNAR2xrYjimdct0uCpsZ/hrQG412h8OrpSthS+rl+/qfVbCUxS5A U24eS44+/7XEVH3Bef1yQj5bfh7aAYbPA1fEnpShjO8JGco9juhLGzq+7zixjgHLCSch dQq+SpwiXfrg2U2p2MysBCZRlHeQ8OfTsZTlCZlZy2X0f/pSCZtUWgvFhRgqc9edAS46 Tt07BWKN1/NGIsbrfcy2JyW5XqQ986bBHXn364CEzYdsZKUubpAvZqGBpY5H2rIjZEu4 wiW0pPLMxbkC9/7raGUxku07AakLdwJTqyenFTMuI6PM+MH2nEhF6bhBPXOpJzRIKVtv IfcQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linuxfoundation.org header.s=korg header.b="piVblyx/"; spf=pass (google.com: domain of stable-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=stable-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linuxfoundation.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id m20si100438lji.284.2021.05.12.08.37.38; Wed, 12 May 2021 08:37:38 -0700 (PDT) Received-SPF: pass (google.com: domain of stable-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@linuxfoundation.org header.s=korg header.b="piVblyx/"; spf=pass (google.com: domain of stable-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=stable-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linuxfoundation.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232855AbhELPay (ORCPT + 12 others); Wed, 12 May 2021 11:30:54 -0400 Received: from mail.kernel.org ([198.145.29.99]:38716 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S235099AbhELP0v (ORCPT ); Wed, 12 May 2021 11:26:51 -0400 Received: by mail.kernel.org (Postfix) with ESMTPSA id 3348F619FE; Wed, 12 May 2021 15:11:40 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1620832300; bh=GXyJDdn2+9x1Pqz4ff0G9UGyDWMUp6D9K7Jnk6FdKGg=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=piVblyx/vrC0PTB/1v3yzf37vi8xAgzKaTzo5MXKwz8gqiO0h3//4D+sZhuOD9i54 8Th18ksPQdU3vsF22NeuOzHSsLn19C0O/fiTZezbyvpfz+JIqS7dnmFEdclHC6qex5 kOfqsVYRfKoFvI0wplzbvMCgjhejJQTBp3ylEsR4= From: Greg Kroah-Hartman To: linux-kernel@vger.kernel.org Cc: Greg Kroah-Hartman , stable@vger.kernel.org, Srinivas Kandagatla , Pierre-Louis Bossart , Vinod Koul , Sasha Levin Subject: [PATCH 5.10 188/530] soundwire: bus: Fix device found flag correctly Date: Wed, 12 May 2021 16:44:58 +0200 Message-Id: <20210512144826.015503180@linuxfoundation.org> X-Mailer: git-send-email 2.31.1 In-Reply-To: <20210512144819.664462530@linuxfoundation.org> References: <20210512144819.664462530@linuxfoundation.org> User-Agent: quilt/0.66 MIME-Version: 1.0 Precedence: bulk List-ID: X-Mailing-List: stable@vger.kernel.org From: Srinivas Kandagatla [ Upstream commit f03690f4f6992225d05dbd1171212e5be5a370dd ] found flag is used to indicate SoundWire devices that are both enumerated on the bus and available in the device list. However this flag is not reset correctly after one iteration, This could miss some of the devices that are enumerated on the bus but not in device list. So reset this correctly to fix this issue! Fixes: d52d7a1be02c ("soundwire: Add Slave status handling helpers") Signed-off-by: Srinivas Kandagatla Reviewed-by: Pierre-Louis Bossart Link: https://lore.kernel.org/r/20210309104816.20350-1-srinivas.kandagatla@linaro.org Signed-off-by: Vinod Koul Signed-off-by: Sasha Levin --- drivers/soundwire/bus.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) -- 2.30.2 diff --git a/drivers/soundwire/bus.c b/drivers/soundwire/bus.c index 1fe786855095..3317a02bcc17 100644 --- a/drivers/soundwire/bus.c +++ b/drivers/soundwire/bus.c @@ -703,7 +703,7 @@ static int sdw_program_device_num(struct sdw_bus *bus) struct sdw_slave *slave, *_s; struct sdw_slave_id id; struct sdw_msg msg; - bool found = false; + bool found; int count = 0, ret; u64 addr; @@ -735,6 +735,7 @@ static int sdw_program_device_num(struct sdw_bus *bus) sdw_extract_slave_id(bus, addr, &id); + found = false; /* Now compare with entries */ list_for_each_entry_safe(slave, _s, &bus->slaves, node) { if (sdw_compare_devid(slave, id) == 0) {