From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from alsa0.perex.cz (alsa0.perex.cz [77.48.224.243]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 5A72EC05027 for ; Wed, 8 Feb 2023 08:35:42 +0000 (UTC) Received: from alsa1.perex.cz (alsa1.perex.cz [207.180.221.201]) (using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by alsa0.perex.cz (Postfix) with ESMTPS id 669C182E; Wed, 8 Feb 2023 09:34:49 +0100 (CET) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa0.perex.cz 669C182E DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=alsa-project.org; s=default; t=1675845339; bh=2oAzgFVIHohjOqZ/o96N6xOmvtqNj9lMcans/gTTlKk=; h=Date:From:To:Subject:References:In-Reply-To:CC:List-Id: List-Archive:List-Help:List-Owner:List-Post:List-Subscribe: List-Unsubscribe:From; b=gtLfL2kQin2Doo/R3U/v/uRxw9cnU5Z2LAEqaJHHpvUbTp+gluWCTzXE1inxzPSpz l3OTTgfUtgFXCqsYT/u1LrBI9lf1TvC+/WU7+vSN5WiaN3iG6Rep/n/s8qWCSHlrVf uO8m2onTBQRs3ZBDIdnXcm9hgbaerdU+fybiJioI= Received: from mailman-core.alsa-project.org (mailman-core.alsa-project.org [10.254.200.10]) by alsa1.perex.cz (Postfix) with ESMTP id 22479F8012B; Wed, 8 Feb 2023 09:34:20 +0100 (CET) Received: by alsa1.perex.cz (Postfix, from userid 50401) id 4430BF8012B; Wed, 8 Feb 2023 09:34:15 +0100 (CET) Received: from out4-smtp.messagingengine.com (out4-smtp.messagingengine.com [66.111.4.28]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by alsa1.perex.cz (Postfix) with ESMTPS id 1039EF80095 for ; Wed, 8 Feb 2023 09:33:56 +0100 (CET) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa1.perex.cz 1039EF80095 Authentication-Results: alsa1.perex.cz; dkim=pass (2048-bit key, unprotected) header.d=sakamocchi.jp header.i=@sakamocchi.jp header.a=rsa-sha256 header.s=fm1 header.b=yMaUdS+a; dkim=pass (2048-bit key, unprotected) header.d=messagingengine.com header.i=@messagingengine.com header.a=rsa-sha256 header.s=fm1 header.b=DPWWMLRf Received: from compute1.internal (compute1.nyi.internal [10.202.2.41]) by mailout.nyi.internal (Postfix) with ESMTP id D48CD5C0072; Wed, 8 Feb 2023 03:33:54 -0500 (EST) Received: from mailfrontend1 ([10.202.2.162]) by compute1.internal (MEProxy); Wed, 08 Feb 2023 03:33:54 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sakamocchi.jp; h=cc:cc:content-transfer-encoding:content-type:date:date:from :from:in-reply-to:in-reply-to:message-id:mime-version:references :reply-to:sender:subject:subject:to:to; s=fm1; t=1675845234; x= 1675931634; bh=jukRv2y1xt9Uct7GPytfpJjy5G82Ak1THGb0pVwupSk=; b=y MaUdS+a0CTiq9wEFVIKTlIcX6qaKtsUGQqpQDpuKvgJV0zbEfcqoTCeXMuRP23WV NBmPljneI3tPYtDlgOuSgBg5rVHcnbF5mosJFpAKlYG1cfp7SiewCBKXHMELgUui 0HlnlVkgujKNWpHp2EZsQEfNgOsB/WulnsyNVLYLhVJG3qKTDxEBbzOJJU8lVk6r Of7/RKXbxiW0WDHPmy5Yd1oS+N65oXVipuq6Vs7scxLwIQ31wBeT/wrwU2S6c74K rzpbAfA2KwtQCNJMg7zrOYQS41rJu0c1FjnuP5X3dF8RjcOj0x/DzCrsWuvZAeAH W5AslNJPEnWfL85F2HQpQ== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:cc:content-transfer-encoding :content-type:date:date:feedback-id:feedback-id:from:from :in-reply-to:in-reply-to:message-id:mime-version:references :reply-to:sender:subject:subject:to:to:x-me-proxy:x-me-proxy :x-me-sender:x-me-sender:x-sasl-enc; s=fm1; t=1675845234; x= 1675931634; bh=jukRv2y1xt9Uct7GPytfpJjy5G82Ak1THGb0pVwupSk=; b=D PWWMLRfuim1VO+n+LmGYnpfrCUG2ISKP8EStW4KqUz95X60TYknJm4QgZbC5jetY KXS8Kuw7mxYcamXzTvcVsgGYE5SZKVYX5DhZF+kglLFZVhRO7VFlE0cXdo9rRMHz DaqCMy1rR/nMS3Y3pStv10RUTthXTPFzx6tLus7put4GA7u7o6x3RfrvSbJSwhio 6SCKQtsjS/f4m0+dLXZMCcPFXBu4WLX1zEf+ZFren+uEgp6TTIY/7AfPY4PVmokS 4/JQgeijKEH9VAPNVpY0G27yKPvDlELbjn/lyKLeEiG4bNoc4QU+rccg6y6MuEwx AnNaRDuMAZWVSJ6pmz6ow== X-ME-Sender: X-ME-Received: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedvhedrudegledguddukecutefuodetggdotefrod ftvfcurfhrohhfihhlvgemucfhrghsthforghilhdpqfgfvfdpuffrtefokffrpgfnqfgh necuuegrihhlohhuthemuceftddtnecusecvtfgvtghiphhivghnthhsucdlqddutddtmd enucfjughrpeffhffvvefukfhfgggtugfgjgesthekredttddtudenucfhrhhomhepvfgr khgrshhhihcuufgrkhgrmhhothhouceoohdqthgrkhgrshhhihesshgrkhgrmhhotggthh hirdhjpheqnecuggftrfgrthhtvghrnhepgeejteekhfejteeiffelleffhefgieffkefh leeujeffkeeiuedvieegfeffieelnecuffhomhgrihhnpehkvghrnhgvlhdrohhrghenuc evlhhushhtvghrufhiiigvpedtnecurfgrrhgrmhepmhgrihhlfhhrohhmpehoqdhtrghk rghshhhisehsrghkrghmohgttghhihdrjhhp X-ME-Proxy: Feedback-ID: ie8e14432:Fastmail Received: by mail.messagingengine.com (Postfix) with ESMTPA; Wed, 8 Feb 2023 03:33:51 -0500 (EST) Date: Wed, 8 Feb 2023 17:33:48 +0900 From: Takashi Sakamoto To: Uwe =?iso-8859-1?Q?Kleine-K=F6nig?= Subject: Re: [PATCH 0/3] ALSA: core: Make some functions return void Message-ID: Mail-Followup-To: Uwe =?iso-8859-1?Q?Kleine-K=F6nig?= , Jaroslav Kysela , Takashi Iwai , Thierry Reding , Jonathan Hunter , Geoff Levand , Michael Ellerman , alsa-devel@alsa-project.org, kernel@pengutronix.de, Nicholas Piggin , Christophe Leroy , linux-tegra@vger.kernel.org, linuxppc-dev@lists.ozlabs.org References: <20230207191907.467756-1-u.kleine-koenig@pengutronix.de> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20230207191907.467756-1-u.kleine-koenig@pengutronix.de> Message-ID-Hash: VRINWLNXSD5PDZ5QNR5KU3OUITTFPGHN X-Message-ID-Hash: VRINWLNXSD5PDZ5QNR5KU3OUITTFPGHN X-MailFrom: o-takashi@sakamocchi.jp X-Mailman-Rule-Misses: dmarc-mitigation; no-senders; approved; emergency; loop; banned-address; member-moderation; header-match-alsa-devel.alsa-project.org-0; header-match-alsa-devel.alsa-project.org-1; nonmember-moderation; administrivia; implicit-dest; max-recipients; max-size; news-moderation; no-subject; digests; suspicious-header CC: Takashi Iwai , Thierry Reding , Jonathan Hunter , Geoff Levand , Michael Ellerman , alsa-devel@alsa-project.org, kernel@pengutronix.de, Nicholas Piggin , Christophe Leroy , linux-tegra@vger.kernel.org, linuxppc-dev@lists.ozlabs.org X-Mailman-Version: 3.3.8 Precedence: list List-Id: "Alsa-devel mailing list for ALSA developers - http://www.alsa-project.org" Archived-At: List-Archive: List-Help: List-Owner: List-Post: List-Subscribe: List-Unsubscribe: Hi, On Tue, Feb 07, 2023 at 08:19:04PM +0100, Uwe Kleine-König wrote: > Hello, > > while checking in which cases hda_tegra_remove() can return a non-zero value, I > found that actually cannot happen. This series makes the involved functions > return void to make this obvious. > > This is a preparation for making platform_driver::remove return void, too. > > Best regards > Uwe > > Uwe Kleine-König (3): > ALSA: core: Make snd_card_disconnect() return void > ALSA: core: Make snd_card_free_when_closed() return void > ALSA: core: Make snd_card_free() return void > > include/sound/core.h | 6 +++--- > sound/core/init.c | 40 ++++++++++++++------------------------- > sound/pci/hda/hda_tegra.c | 6 ++---- > sound/ppc/snd_ps3.c | 4 +--- > 4 files changed, 20 insertions(+), 36 deletions(-) All of the changes in the patches look good to me, as the return value seems not to be so effective for a long time (15 years or more) and expected so for future. Reviewed-by: Takashi Sakamoto One of my concern is that we are mostly in the last week for v6.3 development. When taking influence of the changes into account, it would be possible to postpone to v6.4 development. But it's up to the maintainer. > base-commit: 05ecb680708a1dbe6554d6fc17e5d9a8a7cb5e6a A small nitpicking. I think you use Linus' tree or so: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=05ecb680708a Because the hash is not reachable from the branch for next merge window on the tree of sound subsystem upstream: https://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git/commit/?h=for-next&id=05ecb680708a I guess it is safer to use his tree as your work-base, even if the three patches can be applied to it as is (fortunately). Regards Takashi Sakamoto