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 99354CFB44D for ; Mon, 7 Oct 2024 16:48:45 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: Content-Type:In-Reply-To:From:References:Cc:To:Subject:MIME-Version:Date: Message-ID:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=S9CjsSMRTcKaZhkjpOoFNmBuBgqEpMntZNfMT8VgpNk=; b=NzDGr0YfxVEJDqvPBSzX4dARHK I1F4cCuvlHH2/Xi3VWHJf5Sb/WKJi/+2i4dBzATVz4thPMMp3oLQwRgWc/ANIS1Nsub93AbvdVQ11 n+Vk3VcPCo0ISS70GrrXKrtVwHVK+Wq8B9fUVV8Fp1/ZIUobD2Ii1Y7JSEb7+5hFYMoQ4hRv3nJj4 NBd2TQflGVjccQ91Wl6T2jkKXiJs6b4Wq17mgnPOk0xk40iGx+5BCbUE3+UrE4kAZ0uMBpbj9mvSE zZZRckBh48lwubWf3mYOJWJxJLZqJIYhU3wyuf0cvTtdgrXHtrDocM/o25tE4mvzPs7jYHuRsrf/D TYPnBRuw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1sxquR-00000003Dnl-3HY9; Mon, 07 Oct 2024 16:48:35 +0000 Received: from mail-qv1-xf31.google.com ([2607:f8b0:4864:20::f31]) by bombadil.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1sxqt6-00000003DPZ-2UXm for linux-arm-kernel@lists.infradead.org; Mon, 07 Oct 2024 16:47:14 +0000 Received: by mail-qv1-xf31.google.com with SMTP id 6a1803df08f44-6cb2aaf4a73so48587356d6.2 for ; Mon, 07 Oct 2024 09:47:11 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=broadcom.com; s=google; t=1728319630; x=1728924430; darn=lists.infradead.org; h=content-transfer-encoding:in-reply-to:autocrypt:from :content-language:references:cc:to:subject:user-agent:mime-version :date:message-id:from:to:cc:subject:date:message-id:reply-to; bh=S9CjsSMRTcKaZhkjpOoFNmBuBgqEpMntZNfMT8VgpNk=; b=UDnAInfcP0YPumY2AgZ4R1YIMf+yUSk40OF02LLFt56+ua+BHlpxQFdI1GIDriIrZm 5+CCvQdgZXLINcWn6RzQoAwC8QoGoeH4IVGX21PTqLyGq3RkwYvlS5pizkyhpq4aslHI eHMeKa4leo0xm6tR6kHlG5vXVs/T2tekXwpQU= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1728319630; x=1728924430; h=content-transfer-encoding:in-reply-to:autocrypt:from :content-language:references:cc:to:subject:user-agent:mime-version :date:message-id:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=S9CjsSMRTcKaZhkjpOoFNmBuBgqEpMntZNfMT8VgpNk=; b=RnycYZJ894nr6gfcDmt4r0dfxNAVbXJ1Hig+kWbuux+b40em15g2OecbW72yBFhiCc MxhhDS7idJlfn0lGH4zMBPlSkD4jvJVgehbZ+H/uGPin4JgUXL95E6vIblRrvEs2EpDV Q6IMhQBMmfJ3wzaQtHcj3Xmk7QSXPYyTSTHrZr4wRYjee5Y0LFx6m+8hLo4dnwtBxid/ 0LdxPiHnU9KWQZMJAW+6xi9qArzKmbPh1o+/AWTAuzlhj+Um43OGxQPiKNOjSSa/370J 6J/n2gkpAkImXW5GvkphPZgFF9Ne/tfer8cUqjuGkLSkAbNWhx7uYCB7EOdRyxRdOcBk E78g== X-Forwarded-Encrypted: i=1; AJvYcCUhy7gmCOMncZr5nKUEo4t3hWBdA8j/c2/mHR1lM/68kda4N3XhQyFIKTOhMk9gmCmr5ZH0tulVfXMJNTXnvQA9@lists.infradead.org X-Gm-Message-State: AOJu0YxIl2rbHaAN6jZvErO1+I+W3R9W4iPXa0mj7FsbdN74ZIMNAetE 6ahjeYpa6HxKh3fxKZ88Y1LMCEqvaDTag4miw8y9GAMN8+sqDdu7ePWttzeD9g== X-Google-Smtp-Source: AGHT+IGkzDBDte/43eYOd/aigjOKOeHrGPpTVqz8vfZIpAXJNx38FHSgFi1MXbXtmwfxAGzq41Ah1A== X-Received: by 2002:a05:6214:4a87:b0:6cb:b7ea:2072 with SMTP id 6a1803df08f44-6cbb7ea20a2mr28174736d6.1.1728319630398; Mon, 07 Oct 2024 09:47:10 -0700 (PDT) Received: from [10.67.48.245] ([192.19.223.252]) by smtp.gmail.com with ESMTPSA id 6a1803df08f44-6cba475142dsm26725486d6.102.2024.10.07.09.47.07 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 07 Oct 2024 09:47:09 -0700 (PDT) Message-ID: <17593cd5-e188-4146-8e4b-c87ce48e1140@broadcom.com> Date: Mon, 7 Oct 2024 09:47:06 -0700 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH] firmware: arm_scmi: Give SMC transport precedence over mailbox To: Sudeep Holla Cc: linux-arm-kernel@lists.infread.org, Rob Herring , Krzysztof Kozlowski , Conor Dooley , Cristian Marussi , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , open list , "open list:SYSTEM CONTROL & POWER/MANAGEMENT INTERFACE" , "moderated list:SYSTEM CONTROL & POWER/MANAGEMENT INTERFACE" , justin.chen@broadcom.com, opendmb@gmail.com, kapil.hali@broadcom.com, bcm-kernel-feedback-list@broadcom.com, Arnd Bergmann References: <20241006043317.3867421-1-florian.fainelli@broadcom.com> Content-Language: en-US From: Florian Fainelli Autocrypt: addr=florian.fainelli@broadcom.com; keydata= xsBNBFPAG8ABCAC3EO02urEwipgbUNJ1r6oI2Vr/+uE389lSEShN2PmL3MVnzhViSAtrYxeT M0Txqn1tOWoIc4QUl6Ggqf5KP6FoRkCrgMMTnUAINsINYXK+3OLe7HjP10h2jDRX4Ajs4Ghs JrZOBru6rH0YrgAhr6O5gG7NE1jhly+EsOa2MpwOiXO4DE/YKZGuVe6Bh87WqmILs9KvnNrQ PcycQnYKTVpqE95d4M824M5cuRB6D1GrYovCsjA9uxo22kPdOoQRAu5gBBn3AdtALFyQj9DQ KQuc39/i/Kt6XLZ/RsBc6qLs+p+JnEuPJngTSfWvzGjpx0nkwCMi4yBb+xk7Hki4kEslABEB AAHNMEZsb3JpYW4gRmFpbmVsbGkgPGZsb3JpYW4uZmFpbmVsbGlAYnJvYWRjb20uY29tPsLB IQQQAQgAywUCZWl41AUJI+Jo+hcKAAG/SMv+fS3xUQWa0NryPuoRGjsA3SAUAAAAAAAWAAFr ZXktdXNhZ2UtbWFza0BwZ3AuY29tjDAUgAAAAAAgAAdwcmVmZXJyZWQtZW1haWwtZW5jb2Rp bmdAcGdwLmNvbXBncG1pbWUICwkIBwMCAQoFF4AAAAAZGGxkYXA6Ly9rZXlzLmJyb2FkY29t Lm5ldAUbAwAAAAMWAgEFHgEAAAAEFQgJChYhBNXZKpfnkVze1+R8aIExtcQpvGagAAoJEIEx tcQpvGagWPEH/2l0DNr9QkTwJUxOoP9wgHfmVhqc0ZlDsBFv91I3BbhGKI5UATbipKNqG13Z TsBrJHcrnCqnTRS+8n9/myOF0ng2A4YT0EJnayzHugXm+hrkO5O9UEPJ8a+0553VqyoFhHqA zjxj8fUu1px5cbb4R9G4UAySqyeLLeqnYLCKb4+GklGSBGsLMYvLmIDNYlkhMdnnzsSUAS61 WJYW6jjnzMwuKJ0ZHv7xZvSHyhIsFRiYiEs44kiYjbUUMcXor/uLEuTIazGrE3MahuGdjpT2 IOjoMiTsbMc0yfhHp6G/2E769oDXMVxCCbMVpA+LUtVIQEA+8Zr6mX0Yk4nDS7OiBlvOwE0E U8AbwQEIAKxr71oqe+0+MYCc7WafWEcpQHFUwvYLcdBoOnmJPxDwDRpvU5LhqSPvk/yJdh9k 4xUDQu3rm1qIW2I9Puk5n/Jz/lZsqGw8T13DKyu8eMcvaA/irm9lX9El27DPHy/0qsxmxVmU pu9y9S+BmaMb2CM9IuyxMWEl9ruWFS2jAWh/R8CrdnL6+zLk60R7XGzmSJqF09vYNlJ6Bdbs MWDXkYWWP5Ub1ZJGNJQ4qT7g8IN0qXxzLQsmz6tbgLMEHYBGx80bBF8AkdThd6SLhreCN7Uh IR/5NXGqotAZao2xlDpJLuOMQtoH9WVNuuxQQZHVd8if+yp6yRJ5DAmIUt5CCPcAEQEAAcLB gQQYAQIBKwUCU8AbwgUbDAAAAMBdIAQZAQgABgUCU8AbwQAKCRCTYAaomC8PVQ0VCACWk3n+ obFABEp5Rg6Qvspi9kWXcwCcfZV41OIYWhXMoc57ssjCand5noZi8bKg0bxw4qsg+9cNgZ3P N/DFWcNKcAT3Z2/4fTnJqdJS//YcEhlr8uGs+ZWFcqAPbteFCM4dGDRruo69IrHfyyQGx16s CcFlrN8vD066RKevFepb/ml7eYEdN5SRALyEdQMKeCSf3mectdoECEqdF/MWpfWIYQ1hEfdm C2Kztm+h3Nkt9ZQLqc3wsPJZmbD9T0c9Rphfypgw/SfTf2/CHoYVkKqwUIzI59itl5Lze+R5 wDByhWHx2Ud2R7SudmT9XK1e0x7W7a5z11Q6vrzuED5nQvkhAAoJEIExtcQpvGagugcIAJd5 EYe6KM6Y6RvI6TvHp+QgbU5dxvjqSiSvam0Ms3QrLidCtantcGT2Wz/2PlbZqkoJxMQc40rb fXa4xQSvJYj0GWpadrDJUvUu3LEsunDCxdWrmbmwGRKqZraV2oG7YEddmDqOe0Xm/NxeSobc MIlnaE6V0U8f5zNHB7Y46yJjjYT/Ds1TJo3pvwevDWPvv6rdBeV07D9s43frUS6xYd1uFxHC 7dZYWJjZmyUf5evr1W1gCgwLXG0PEi9n3qmz1lelQ8lSocmvxBKtMbX/OKhAfuP/iIwnTsww 95A2SaPiQZA51NywV8OFgsN0ITl2PlZ4Tp9hHERDe6nQCsNI/Us= In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20241007_094712_770907_98516A1F X-CRM114-Status: GOOD ( 21.11 ) 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: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 10/7/24 06:13, Sudeep Holla wrote: > On Sat, Oct 05, 2024 at 09:33:17PM -0700, Florian Fainelli wrote: >> Broadcom STB platforms have for historical reasons included both >> "arm,scmi-smc" and "arm,scmi" in their SCMI Device Tree node compatible >> string. >> > > I assume in the same order. That is correct, in that exact order indeed. > >> After the commit cited in the Fixes tag and with a kernel >> configuration that enables both the SCMI and the Mailbox transports, we > > ^^^^^ s/SCMI/SMC ? Yes, this should read "SMC" here. > >> would probe the mailbox transport, but fail to complete since we would >> not have a mailbox driver available. >> > > I always assumed the node compatible match happens from the more specific > compatible(on the left) to the more generic ones(on the right) from the > compatible property list. Looks like that was a wrong assumption then ? This is the correct assumption, and this worked very well, and we were utilizing that as long as all of the transports where "sub" entities within the common and single arm_scmi platform device. When breaking up the transports into individual platform drivers, now each one is responsible for matching, and if they are all built-into the kernel, they are matching in the order in which they have been linked into the kernel. > >> By keeping the SMC transport objects linked first, we can let the >> platform driver, match the compatible string and probe successfully with >> no adverse effects on platforms using the mailbox transport. >> > > I don't have strong objection to the patch itself, happy to get it merged. > Just curious if my understanding of the issue is correct. I think Cristian > has more detailed query, so just responding to that will suffice. Sounds good, thanks. > >> Fixes: b53515fa177c ("firmware: arm_scmi: Make MBOX transport a standalone driver") >> Signed-off-by: Florian Fainelli >> Change-Id: I8e348e3e0deabdc5c1d596929d7f9134793f346e > > Spurious from internal gerrit repo ? Indeed, will post v2 with the typo you highlighted and that remove, and any additional explanation Cristian deems necessary to add, thanks! -- Florian