From nobody Fri Apr 17 00:19:47 2026 Received: from mail-244123.protonmail.ch (mail-244123.protonmail.ch [109.224.244.123]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 7031D223702 for ; Tue, 24 Feb 2026 18:55:12 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=109.224.244.123 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1771959313; cv=none; b=X6WB646+HBCv32Qc/lqgS6nIqp+3EfTIHs8de3TSd1SypVgKCL3bsKC1r1+nFFm4LUjeaPAehbcPSvO4mtP73DeIwFEEnYZbKCO2VJZiaSRWEvcuV3KowgTK+vsy87ZLNgK4qUvVOBFhtg6ywHNA1lzaRsDYICRbG84DdwMQamk= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1771959313; c=relaxed/simple; bh=/I0g4pAJtB4CFwLX59jPUjFNm+tlip377Oon3+mS7XQ=; h=Date:To:From:Cc:Subject:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=RXdFKXHsxToMWbx7Dn9Uc94vLRmoM2EWbx9s7OiiAFIL+LNSVlF1rjymnR/qdtGKeKooeRFY6LLyRsNnSk/+SI+9jTKDLmOB0zK9phWpbA0/pmmJRa/DZFRX54BP5LVCehBqfceYYJU3LEDfrV2TFlpOqtramHMK2hz+2rCIHWs= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=pm.me; spf=pass smtp.mailfrom=pm.me; dkim=pass (2048-bit key) header.d=pm.me header.i=@pm.me header.b=dkujm8ee; arc=none smtp.client-ip=109.224.244.123 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=pm.me Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=pm.me Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=pm.me header.i=@pm.me header.b="dkujm8ee" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=pm.me; s=protonmail3; t=1771959304; x=1772218504; bh=kxiMSX85FR0vHxuxYmk5tZWATlOkxSIi5aOd5/WnLM0=; h=Date:To:From:Cc:Subject:Message-ID:In-Reply-To:References: Feedback-ID:From:To:Cc:Date:Subject:Reply-To:Feedback-ID: Message-ID:BIMI-Selector; b=dkujm8ee7Ata9gtdli/pM7KahdJMlCGjYPVi06KCxD6nx3OrmIuEGmmrBFdp2HhHk ZL1WQzRSqRr3zdHii9Tr6+bz19/qGZM3N9kMBAJX6Yd5eSKYItkyvk6AMHC1qHw3zN XqTKXGoz3XUFSMVMbK6bTEJcGUHCILDl+PBzRAEjQryC4mRAOjepC9n8Bd0t65ROQs 87neun0NLkqF0Ir1wrzamYmXKYfVt31362MIyp11X0MbUXQYIc6b4suRE3nkCevziy Grmo3RkAhtp6yGepo11f8kEDY/Yw+g7w8Zz3d4h7QvOIELhj6lHLn2d/O0FVWEY27v AnDMkq4fTpeFA== Date: Tue, 24 Feb 2026 18:54:59 +0000 To: Greg Kroah-Hartman , Vaibhav Agarwal From: Hardik Phalet Cc: Dan Carpenter , Mark Greer , Johan Hovold , Alex Elder , greybus-dev@lists.linaro.org, linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org, Hardik Phalet Subject: [PATCH v3 1/2] staging: greybus: audio: remove unused gb_audio_manager_get_module() Message-ID: <20260224185421.824210-2-hardik.phalet@pm.me> In-Reply-To: <20260224185421.824210-1-hardik.phalet@pm.me> References: <20260224185421.824210-1-hardik.phalet@pm.me> Feedback-ID: 166659585:user:proton X-Pm-Message-ID: 8bc6976a10a119dab266dad39635e92155bcfb25 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" gb_audio_manager_get_module() has no in-tree callers. The previously reported NULL dereference is therefore unreachable. Remove the unused function to avoid carrying dead code. Signed-off-by: Hardik Phalet --- drivers/staging/greybus/audio_manager.c | 12 ------------ drivers/staging/greybus/audio_manager.h | 7 ------- 2 files changed, 19 deletions(-) diff --git a/drivers/staging/greybus/audio_manager.c b/drivers/staging/grey= bus/audio_manager.c index 27ca5f796c5f..118ada9b909b 100644 --- a/drivers/staging/greybus/audio_manager.c +++ b/drivers/staging/greybus/audio_manager.c @@ -105,18 +105,6 @@ void gb_audio_manager_remove_all(void) } EXPORT_SYMBOL_GPL(gb_audio_manager_remove_all); =20 -struct gb_audio_manager_module *gb_audio_manager_get_module(int id) -{ - struct gb_audio_manager_module *module; - - down_read(&modules_rwsem); - module =3D gb_audio_manager_get_locked(id); - kobject_get(&module->kobj); - up_read(&modules_rwsem); - return module; -} -EXPORT_SYMBOL_GPL(gb_audio_manager_get_module); - void gb_audio_manager_put_module(struct gb_audio_manager_module *module) { kobject_put(&module->kobj); diff --git a/drivers/staging/greybus/audio_manager.h b/drivers/staging/grey= bus/audio_manager.h index be605485a8ce..c3ef62ee22c8 100644 --- a/drivers/staging/greybus/audio_manager.h +++ b/drivers/staging/greybus/audio_manager.h @@ -54,13 +54,6 @@ int gb_audio_manager_remove(int id); */ void gb_audio_manager_remove_all(void); =20 -/* - * Retrieves a gb_audio_manager_module_descriptor for the specified id. - * Returns the gb_audio_manager_module_descriptor structure, - * or NULL if there is no module with the specified ID. - */ -struct gb_audio_manager_module *gb_audio_manager_get_module(int id); - /* * Decreases the refcount of the module, obtained by the get function. * Modules are removed via gb_audio_manager_remove --=20 2.53.0 From nobody Fri Apr 17 00:19:47 2026 Received: from mail-106121.protonmail.ch (mail-106121.protonmail.ch [79.135.106.121]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 4074F27E054 for ; Tue, 24 Feb 2026 18:55:13 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=79.135.106.121 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1771959314; cv=none; b=szQT1VlwTNDtK5ZRmtHR5I/k46kyH9k39nX4uQEhW3Z03LYb1E5WhcQlvenf7qNS1PP7Uk0NOMrSCkSGu/0v7Qs+meqUFlWuhf+HZ+USGGbE8pn3oy3Dw0UIxCx51DjeAyee5wRdRsb80RZ7WirVZ+QV1dCnDb4q1gc8nGIxh8s= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1771959314; c=relaxed/simple; bh=Kl8cL99lfSpUtDgg6mYNf0ILtuKuuEtwDU8t89wUAlk=; h=Date:To:From:Cc:Subject:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=r6xCYSxmjyuqWOZ/s9NVOpagNGjX88Sz6/2mr/Pqj63hn4BG+htUxNN+meIR1igU3Np5Fr/8ZVM9FQlySO/obYHR2G75PnmoR/5V8l/60rjADT0mvqsFTeVBfYjM9E4K2z+fBCOGuajLF2PRWcVpC61f+XKA+PsYn+1C+C0eYok= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=pm.me; spf=pass smtp.mailfrom=pm.me; dkim=pass (2048-bit key) header.d=pm.me header.i=@pm.me header.b=dWuZ2AfK; arc=none smtp.client-ip=79.135.106.121 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=pm.me Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=pm.me Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=pm.me header.i=@pm.me header.b="dWuZ2AfK" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=pm.me; s=protonmail3; t=1771959310; x=1772218510; bh=Kl8cL99lfSpUtDgg6mYNf0ILtuKuuEtwDU8t89wUAlk=; h=Date:To:From:Cc:Subject:Message-ID:In-Reply-To:References: Feedback-ID:From:To:Cc:Date:Subject:Reply-To:Feedback-ID: Message-ID:BIMI-Selector; b=dWuZ2AfKQmaXikae5oESe0j40qUqHOPM2CvUb01cHe+o5ofU9HmJqGxaansJDVTJE Oia6dJUttjuuMuvHXyNvwVTTfN/xqsXAldwnLTrBTw4pGXYKp8iy1jEN++VWsFtsI6 JZ7mSrk3Tqrn8m23xbOUI8YfdlxmHavT44Wq48PTUMMr2gT2twimmQ9jBNDiGhN0Mg VTPYtymUHCHOVZg8X904zWQS5oOH1djI9N4fdXtjIkwNrA7RyjTjDTMN3RRUbN3IB3 dutewSYsAcA9J6Q8XyBKeMpU0dwOUlVkxQTkrmRdGzPUnydhpaintxKbwrCn8as7dn wh6alJJW2WFuA== Date: Tue, 24 Feb 2026 18:55:07 +0000 To: Greg Kroah-Hartman , Vaibhav Agarwal From: Hardik Phalet Cc: Dan Carpenter , Mark Greer , Johan Hovold , Alex Elder , greybus-dev@lists.linaro.org, linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org, Hardik Phalet Subject: [PATCH v3 2/2] staging: greybus: audio: drop stale TODO comment Message-ID: <20260224185421.824210-3-hardik.phalet@pm.me> In-Reply-To: <20260224185421.824210-1-hardik.phalet@pm.me> References: <20260224185421.824210-1-hardik.phalet@pm.me> Feedback-ID: 166659585:user:proton X-Pm-Message-ID: 3bceb23eaaa3ac5046980882608cdb28ad275398 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Modules are removed from modules_list in gb_audio_manager_remove() and gb_audio_manager_remove_all() before kobject_put(). The TODO suggesting list deletion in the kobject release callback is stale and misleading. Signed-off-by: Hardik Phalet --- drivers/staging/greybus/audio_manager_module.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/greybus/audio_manager_module.c b/drivers/stagi= ng/greybus/audio_manager_module.c index e87b82ca6d8a..97d67989b181 100644 --- a/drivers/staging/greybus/audio_manager_module.c +++ b/drivers/staging/greybus/audio_manager_module.c @@ -69,7 +69,6 @@ static void gb_audio_module_release(struct kobject *kobj) struct gb_audio_manager_module *module =3D to_gb_audio_module(kobj); =20 pr_info("Destroying audio module #%d\n", module->id); - /* TODO -> delete from list */ kfree(module); } =20 --=20 2.53.0