From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from wfhigh2-smtp.messagingengine.com (wfhigh2-smtp.messagingengine.com [64.147.123.153]) (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 3AF5D26ACC for ; Mon, 27 May 2024 16:00:53 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=64.147.123.153 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1716825655; cv=none; b=bI8JJ/LrYCqEillvDiKOi5Z+gWMN2fqHVth6bhQH2VuCv+m6iYZ3XzZJ8wyAjNvpD4yin/fPddoZArirZWgV0kHwXWnjMP14wB3XbYbCHBFihHRE2lmBUwB2wUiqnS7hK4yrTrAHp6QFBDc8ieLYU3xKanV6LqlODa3oiZivj4U= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1716825655; c=relaxed/simple; bh=eu9ERRY2/2tzbNM7/sRA/5PblbcpQRZ7jEKvETm7HZ8=; h=MIME-Version:Message-Id:In-Reply-To:References:Date:From:To:Cc: Subject:Content-Type; b=G5UY9Iwh6e+/cx6XiC1FdwxekhDSMZIXH81fQkCx61iPcouR2qqnZn0EeuCwsL2PZrFotZUn21QosagWvB9aV12ahMkhdM5REu+/kba9flWTpdeEdW+W/TbwbgM+1oMusfULdBeo9jVLiJxuy4rHDDRYk82z0fTAaPvPOxPZ1m0= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=svenpeter.dev; spf=pass smtp.mailfrom=svenpeter.dev; dkim=pass (2048-bit key) header.d=svenpeter.dev header.i=@svenpeter.dev header.b=j9SWpHo0; dkim=pass (2048-bit key) header.d=messagingengine.com header.i=@messagingengine.com header.b=o2Hw3g8U; arc=none smtp.client-ip=64.147.123.153 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=svenpeter.dev Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=svenpeter.dev Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=svenpeter.dev header.i=@svenpeter.dev header.b="j9SWpHo0"; dkim=pass (2048-bit key) header.d=messagingengine.com header.i=@messagingengine.com header.b="o2Hw3g8U" Received: from compute1.internal (compute1.nyi.internal [10.202.2.41]) by mailfhigh.west.internal (Postfix) with ESMTP id D87B61800114; Mon, 27 May 2024 12:00:51 -0400 (EDT) Received: from imap47 ([10.202.2.97]) by compute1.internal (MEProxy); Mon, 27 May 2024 12:00:52 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=svenpeter.dev; h=cc:cc:content-transfer-encoding:content-type:content-type :date:date:from:from:in-reply-to:in-reply-to:message-id :mime-version:references:reply-to:subject:subject:to:to; s=fm1; t=1716825651; x=1716912051; bh=JR/OInhF2POmXtVueCyiCaZ7btN5anbT nKy85NzXzCk=; b=j9SWpHo0SpO6FNmBkyvG5sFOOUyDv1WMR2oQv2fxilgGy2Cz 4f2tYinh11rf1j9bOmjX3pdmOboFrKCih6tbvrq2nx8TREF8IjTYxCvLHg0i7fCL neay14nDKND8yud/+lr+gNVjXlce+LeLzmvuk1HjVTat/rb0WMfS4AUSiQp4JRB3 0ZGLYRMxvwnq7a02yuCZf4+2Xwl0b/8610IU5RboRdwSwNsLeXju7jiUKjyaQzTa 1ZEI9dGgQGd5breYjYSKzQ0ieeY+uhusK3sf0n4b5BqyZeR3L/OQfQHjudkBM0tw FtRm5HTCYcadMLMnxvacMsSHVXyK90f/8CwgOw== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:cc:content-transfer-encoding :content-type:content-type:date:date:feedback-id:feedback-id :from:from:in-reply-to:in-reply-to:message-id:mime-version :references:reply-to:subject:subject:to:to:x-me-proxy:x-me-proxy :x-me-sender:x-me-sender:x-sasl-enc; s=fm1; t=1716825651; x= 1716912051; bh=JR/OInhF2POmXtVueCyiCaZ7btN5anbTnKy85NzXzCk=; b=o 2Hw3g8UFyT1ekXk3sYpkq0UJ38uCCtoe2QUqKq+b/BjKv/S/YEdtZqAMSmnfMcqu V9/R3YUapZebOpEdQhwCC0kQTNVpJxksWt7oGyKBoiEPZx3bPR0baZR598HD0c0G u2ai5oCETQZNJp4Bko6w7nGNslRgJ0P5mJ06YRQHqunEP5Z3Y9O2UOMG7WNucwE8 26IJz/d4s/lHSNcB3p5UvLoo0SAa965+C2Dqt9Nm84Z5U5YJ7+HoI0C3Ku4EOxU/ GMfMYwwvacfEE6BV12j5dUGQ/xwf2ooyxdrOn/XGZW5GeveB8xmEf+phXyxOOBOu 12r7VP1GO3ImTvdDzChUg== X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedvledrvdejgedgleefucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurhepofgfggfkjghffffhvfevufgtgfesthhqredtreerjeenucfhrhhomhepfdfu vhgvnhcurfgvthgvrhdfuceoshhvvghnsehsvhgvnhhpvghtvghrrdguvghvqeenucggtf frrghtthgvrhhnpeevhedthffgffelhedujefgueduudeutdefleevvdetudelhfeihfdv ffelteeuudenucevlhhushhtvghrufhiiigvpedtnecurfgrrhgrmhepmhgrihhlfhhroh hmpehsvhgvnhesshhvvghnphgvthgvrhdruggvvh X-ME-Proxy: Feedback-ID: i51094778:Fastmail Received: by mailuser.nyi.internal (Postfix, from userid 501) id 68666A60078; Mon, 27 May 2024 12:00:49 -0400 (EDT) X-Mailer: MessagingEngine.com Webmail Interface User-Agent: Cyrus-JMAP/3.11.0-alpha0-491-g033e30d24-fm-20240520.001-g033e30d2 Precedence: bulk X-Mailing-List: asahi@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Message-Id: In-Reply-To: <20240527132552.14119-1-ilpo.jarvinen@linux.intel.com> References: <20240527132552.14119-1-ilpo.jarvinen@linux.intel.com> Date: Mon, 27 May 2024 18:00:48 +0200 From: "Sven Peter" To: =?UTF-8?Q?Ilpo_J=C3=A4rvinen?= , "Hector Martin" , "Alyssa Rosenzweig" , "Marcel Holtmann" , "Luiz Augusto von Dentz" , asahi@lists.linux.dev, linux-arm-kernel@lists.infradead.org, linux-bluetooth@vger.kernel.org, linux-kernel@vger.kernel.org Cc: stable@vger.kernel.org Subject: Re: [PATCH 1/1] Bluetooth: hci_bcm4377: Convert PCIBIOS_* return codes to errnos Content-Type: text/plain;charset=utf-8 Content-Transfer-Encoding: quoted-printable Hi, On Mon, May 27, 2024, at 15:25, Ilpo J=C3=A4rvinen wrote: > bcm4377_init_cfg() uses pci_{read,write}_config_dword() that return > PCIBIOS_* codes. The return codes are returned into the calling > bcm4377_probe() which directly returns the error which is of incorrect > type (a probe should return normal errnos). Good catch! > > Convert PCIBIOS_* returns code using pcibios_err_to_errno() into normal > errno before returning it from bcm4377_init_cfg. This conversion is the > easiest by adding a label next to return and doing the conversion there > once rather than adding pcibios_err_to_errno() into every single return > statement. Given that bcm4377_init_cfg is only called at one place from bcm4377_pro= be we could also just do something like ret =3D bcm4377_init_cfg(bcm4377); if (ret) return pcibios_err_to_errno(ret); there, but either way is fine with me. Best, Sven 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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (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 4F792C25B74 for ; Mon, 27 May 2024 16:01:19 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:Subject:Cc:To:From:Date:References: In-Reply-To:Message-Id:MIME-Version:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=0K8nmSR9x7Gm2T6Wygqne3pB8dpywbeQZRT7Z7AYzog=; b=j9e/YVWOs76kOj jxrjVRXpXOblRICOuUr8gKrQjKjj5SYeyxJzmuWiWRYxvhTX/DY/6zXGYZntkyoWCI5sLuL0yQu+M rLJcgVXNViIGRGhkcNz1fUtcXCNaTrmTOZpxZepP8+SIkRmW84WwG7A1IaMtyXyNKu9TzTueLotJr fTgYKaQmnjhAUpNebWAWN9IYDDAVZJqOskgwhBb7inWuila+7UzzSgcihMd+WEtnUAWzkC5mvqe+5 bzHH3rvbhu98Y2ciab0fUQkEV4beyOzt4S+DsDYzke3SzwdPVwLxB/NS9xRhaKDBQrFjbz/W+GNaE oB0w1MAFuQku+zXF5OLw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sBcma-0000000FimP-1JED; Mon, 27 May 2024 16:01:08 +0000 Received: from wfhigh2-smtp.messagingengine.com ([64.147.123.153]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1sBcmS-0000000Figv-3ebE for linux-arm-kernel@lists.infradead.org; Mon, 27 May 2024 16:01:05 +0000 Received: from compute1.internal (compute1.nyi.internal [10.202.2.41]) by mailfhigh.west.internal (Postfix) with ESMTP id D87B61800114; Mon, 27 May 2024 12:00:51 -0400 (EDT) Received: from imap47 ([10.202.2.97]) by compute1.internal (MEProxy); Mon, 27 May 2024 12:00:52 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=svenpeter.dev; h=cc:cc:content-transfer-encoding:content-type:content-type :date:date:from:from:in-reply-to:in-reply-to:message-id :mime-version:references:reply-to:subject:subject:to:to; s=fm1; t=1716825651; x=1716912051; bh=JR/OInhF2POmXtVueCyiCaZ7btN5anbT nKy85NzXzCk=; b=j9SWpHo0SpO6FNmBkyvG5sFOOUyDv1WMR2oQv2fxilgGy2Cz 4f2tYinh11rf1j9bOmjX3pdmOboFrKCih6tbvrq2nx8TREF8IjTYxCvLHg0i7fCL neay14nDKND8yud/+lr+gNVjXlce+LeLzmvuk1HjVTat/rb0WMfS4AUSiQp4JRB3 0ZGLYRMxvwnq7a02yuCZf4+2Xwl0b/8610IU5RboRdwSwNsLeXju7jiUKjyaQzTa 1ZEI9dGgQGd5breYjYSKzQ0ieeY+uhusK3sf0n4b5BqyZeR3L/OQfQHjudkBM0tw FtRm5HTCYcadMLMnxvacMsSHVXyK90f/8CwgOw== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:cc:content-transfer-encoding :content-type:content-type:date:date:feedback-id:feedback-id :from:from:in-reply-to:in-reply-to:message-id:mime-version :references:reply-to:subject:subject:to:to:x-me-proxy:x-me-proxy :x-me-sender:x-me-sender:x-sasl-enc; s=fm1; t=1716825651; x= 1716912051; bh=JR/OInhF2POmXtVueCyiCaZ7btN5anbTnKy85NzXzCk=; b=o 2Hw3g8UFyT1ekXk3sYpkq0UJ38uCCtoe2QUqKq+b/BjKv/S/YEdtZqAMSmnfMcqu V9/R3YUapZebOpEdQhwCC0kQTNVpJxksWt7oGyKBoiEPZx3bPR0baZR598HD0c0G u2ai5oCETQZNJp4Bko6w7nGNslRgJ0P5mJ06YRQHqunEP5Z3Y9O2UOMG7WNucwE8 26IJz/d4s/lHSNcB3p5UvLoo0SAa965+C2Dqt9Nm84Z5U5YJ7+HoI0C3Ku4EOxU/ GMfMYwwvacfEE6BV12j5dUGQ/xwf2ooyxdrOn/XGZW5GeveB8xmEf+phXyxOOBOu 12r7VP1GO3ImTvdDzChUg== X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedvledrvdejgedgleefucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurhepofgfggfkjghffffhvfevufgtgfesthhqredtreerjeenucfhrhhomhepfdfu vhgvnhcurfgvthgvrhdfuceoshhvvghnsehsvhgvnhhpvghtvghrrdguvghvqeenucggtf frrghtthgvrhhnpeevhedthffgffelhedujefgueduudeutdefleevvdetudelhfeihfdv ffelteeuudenucevlhhushhtvghrufhiiigvpedtnecurfgrrhgrmhepmhgrihhlfhhroh hmpehsvhgvnhesshhvvghnphgvthgvrhdruggvvh X-ME-Proxy: Feedback-ID: i51094778:Fastmail Received: by mailuser.nyi.internal (Postfix, from userid 501) id 68666A60078; Mon, 27 May 2024 12:00:49 -0400 (EDT) X-Mailer: MessagingEngine.com Webmail Interface User-Agent: Cyrus-JMAP/3.11.0-alpha0-491-g033e30d24-fm-20240520.001-g033e30d2 MIME-Version: 1.0 Message-Id: In-Reply-To: <20240527132552.14119-1-ilpo.jarvinen@linux.intel.com> References: <20240527132552.14119-1-ilpo.jarvinen@linux.intel.com> Date: Mon, 27 May 2024 18:00:48 +0200 From: "Sven Peter" To: =?UTF-8?Q?Ilpo_J=C3=A4rvinen?= , "Hector Martin" , "Alyssa Rosenzweig" , "Marcel Holtmann" , "Luiz Augusto von Dentz" , asahi@lists.linux.dev, linux-arm-kernel@lists.infradead.org, linux-bluetooth@vger.kernel.org, linux-kernel@vger.kernel.org Cc: stable@vger.kernel.org Subject: Re: [PATCH 1/1] Bluetooth: hci_bcm4377: Convert PCIBIOS_* return codes to errnos X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240527_090101_450163_794CABFF X-CRM114-Status: UNSURE ( 8.90 ) X-CRM114-Notice: Please train this message. X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org SGksCgoKT24gTW9uLCBNYXkgMjcsIDIwMjQsIGF0IDE1OjI1LCBJbHBvIErDpHJ2aW5lbiB3cm90 ZToKPiBiY200Mzc3X2luaXRfY2ZnKCkgdXNlcyBwY2lfe3JlYWQsd3JpdGV9X2NvbmZpZ19kd29y ZCgpIHRoYXQgcmV0dXJuCj4gUENJQklPU18qIGNvZGVzLiBUaGUgcmV0dXJuIGNvZGVzIGFyZSBy ZXR1cm5lZCBpbnRvIHRoZSBjYWxsaW5nCj4gYmNtNDM3N19wcm9iZSgpIHdoaWNoIGRpcmVjdGx5 IHJldHVybnMgdGhlIGVycm9yIHdoaWNoIGlzIG9mIGluY29ycmVjdAo+IHR5cGUgKGEgcHJvYmUg c2hvdWxkIHJldHVybiBub3JtYWwgZXJybm9zKS4KCkdvb2QgY2F0Y2ghCgo+Cj4gQ29udmVydCBQ Q0lCSU9TXyogcmV0dXJucyBjb2RlIHVzaW5nIHBjaWJpb3NfZXJyX3RvX2Vycm5vKCkgaW50byBu b3JtYWwKPiBlcnJubyBiZWZvcmUgcmV0dXJuaW5nIGl0IGZyb20gYmNtNDM3N19pbml0X2NmZy4g VGhpcyBjb252ZXJzaW9uIGlzIHRoZQo+IGVhc2llc3QgYnkgYWRkaW5nIGEgbGFiZWwgbmV4dCB0 byByZXR1cm4gYW5kIGRvaW5nIHRoZSBjb252ZXJzaW9uIHRoZXJlCj4gb25jZSByYXRoZXIgdGhh biBhZGRpbmcgcGNpYmlvc19lcnJfdG9fZXJybm8oKSBpbnRvIGV2ZXJ5IHNpbmdsZSByZXR1cm4K PiBzdGF0ZW1lbnQuCgpHaXZlbiB0aGF0IGJjbTQzNzdfaW5pdF9jZmcgaXMgb25seSBjYWxsZWQg YXQgb25lIHBsYWNlIGZyb20gYmNtNDM3N19wcm9iZQp3ZSBjb3VsZCBhbHNvIGp1c3QgZG8gc29t ZXRoaW5nIGxpa2UKCglyZXQgPSBiY200Mzc3X2luaXRfY2ZnKGJjbTQzNzcpOwoJaWYgKHJldCkK CQlyZXR1cm4gcGNpYmlvc19lcnJfdG9fZXJybm8ocmV0KTsKCnRoZXJlLCBidXQgZWl0aGVyIHdh eSBpcyBmaW5lIHdpdGggbWUuCgoKQmVzdCwKCgpTdmVuCgoKX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX18KbGludXgtYXJtLWtlcm5lbCBtYWlsaW5nIGxpc3QK bGludXgtYXJtLWtlcm5lbEBsaXN0cy5pbmZyYWRlYWQub3JnCmh0dHA6Ly9saXN0cy5pbmZyYWRl YWQub3JnL21haWxtYW4vbGlzdGluZm8vbGludXgtYXJtLWtlcm5lbAo=