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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 32C76EB64D9 for ; Sat, 17 Jun 2023 07:22:05 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1345949AbjFQHWD (ORCPT ); Sat, 17 Jun 2023 03:22:03 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:44810 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S234065AbjFQHWB (ORCPT ); Sat, 17 Jun 2023 03:22:01 -0400 Received: from mail-ej1-x62c.google.com (mail-ej1-x62c.google.com [IPv6:2a00:1450:4864:20::62c]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 6F3BE26BE for ; Sat, 17 Jun 2023 00:22:00 -0700 (PDT) Received: by mail-ej1-x62c.google.com with SMTP id a640c23a62f3a-970056276acso212128466b.2 for ; Sat, 17 Jun 2023 00:22:00 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1686986519; x=1689578519; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=SjXSmylfvJ9wJt437Uf//TtaIsBaLnBZp/3Ml1w3C3I=; b=yHBWeC24+nXPeU37O1qJgm8rAzoFg76b8+TqcnfZUv5dYdZkOfVs0rhdwVWbC5V0fN q28pRqeLZLRBZNuZtjDwZYy4pPNfLLkqxdblO4oOXUudZdgELeCWRFsuyVtmT7FuQI9/ JtDp4YBcJzfEgPGuBEStRZbbln/t2TPQYzQPQwId3J1JryHGeFM6q1xxSY/kwFinQ6Ot dx0A39gLOvfiUv+mA4YxXRnXaPDGUpdLje0R6pNS80HDKaz+cUkYjjI7msih9jQKeXUN ggDp2ht8pJF0nomgooJ0OAyMMrHnkTyVHSeEugRtAYwW3fbdMg25q9vHiuU+o+99SSLP yUqA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1686986519; x=1689578519; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=SjXSmylfvJ9wJt437Uf//TtaIsBaLnBZp/3Ml1w3C3I=; b=hX5OiJveA8sBDyO/X5sLSR1r0rNAIezkPCiVPtIJl3H1F1ckQA3qOWmIuolpAj7W29 fCq8BGUZcYyHXAdub0uinBRtJzR9D42igISBvXUIbnA4ByHL521UJFS8GZ/cMofUr/Cf SXeQ2tx7qRkUW7sEwFJkF12+/+vPu78y1BUxd8pCreARrHr2LFuP2NlhF+mjT+7ljp/P qg0KlB/xyhSunBo606WtT0NE/J2DtHdgueQg7I643G++NIJtNpGMdPsEtNmJDg8BmcBo zj/2Ck1ZxsNVIi7By6LI7KB6AfBE7N9ExZO2S4UJu4+M/Kbop411o3hI79jXxgMcmYOe GPjw== X-Gm-Message-State: AC+VfDwnjfOv0axTkiKysJlIEIoYULvOuDZfZtRQoenrCU074sYWqelQ AWMbbjq31tLfDScsiSMy7ATTZg== X-Google-Smtp-Source: ACHHUZ4vncMEFaT77el4Fw+e6+SIxuUCOeI/uc/vZiQfRucC3BHCi0YlfIAHeqR+DNo7mKqVjAbKIQ== X-Received: by 2002:a17:906:ef0a:b0:978:337e:c41a with SMTP id f10-20020a170906ef0a00b00978337ec41amr3750891ejs.14.1686986518710; Sat, 17 Jun 2023 00:21:58 -0700 (PDT) Received: from [192.168.1.20] ([178.197.219.26]) by smtp.gmail.com with ESMTPSA id m15-20020a1709060d8f00b00982aa9db473sm2965074eji.55.2023.06.17.00.21.56 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Sat, 17 Jun 2023 00:21:58 -0700 (PDT) Message-ID: Date: Sat, 17 Jun 2023 09:21:55 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.12.0 Subject: Re: [PATCH] arm64: dts: qcom: sdm845-db845c: Move LVS regulator nodes up Content-Language: en-US To: Amit Pundir Cc: Linux regressions mailing list , Mark Brown , Doug Anderson , Bjorn Andersson , Andy Gross , Rob Herring , Konrad Dybcio , Krzysztof Kozlowski , Caleb Connolly , Conor Dooley , linux-arm-msm , dt , lkml References: <20230602161246.1855448-1-amit.pundir@linaro.org> <358c69ad-fa8a-7386-fe75-92369883ee48@leemhuis.info> <0f6c9dcb-b7f6-fff9-6bed-f4585ea8e487@linaro.org> <0ec6c988-d678-c96c-a7a2-af38e6701404@linaro.org> <12d6b687-5e5a-bd7c-ff5c-007a74753edb@linaro.org> From: Krzysztof Kozlowski In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-arm-msm@vger.kernel.org On 16/06/2023 19:09, Amit Pundir wrote: > Hi, > > On Fri, 16 Jun 2023 at 13:57, Krzysztof Kozlowski > wrote: >> >> >> So you have interconnect as module - this is not a supported setup. It >> might work with if all the modules are loaded very early or might not. >> Pinctrl is another driver which should be built-in. >> >> With your defconfig I see regular issue - console and system dies >> because of lack of interconnects, most likely. I don't see your WARNs - >> I just see usual hang. >> >> See: >> https://lore.kernel.org/all/20221021032702.1340963-1-krzysztof.kozlowski@linaro.org/ >> >> If you want them to really be modules, then you need to fix all the >> dependencies (SOFTDEP?), probe ordering glitches. It's not a problem of >> DTS. Just because something can be built as module, does not mean it >> will work. We don't test it, we don't work with them as modules. > > I do somewhat agree with most of your arguments but not this one. If a > driver doesn't work as a module then it shouldn't be allowed to build > as a module. Of course you are right. That's why I am pushing against blindly adding "tristate" by everyone working on GKI. Because such folks like to make them tristate, but not actually test it or work on issues later. That's exactly the case from Google and Samsung patches here: https://lore.kernel.org/all/ac328b6a-a8e2-873d-4015-814cb4f5588e@canonical.com/ and in previous submissions. > I took a quick look at the history of the interconnect > driver and it is tristate from the beginning. And not converted to a > modular build later-on like some of the other drivers to support GKI. OK, maybe it was never actually tested. Or maybe some versions were working on boards where debug serial does not have interconnect, but new chips just followed the pattern without testing? > >> >> It's kind of the same as here: >> https://lore.kernel.org/all/ac328b6a-a8e2-873d-4015-814cb4f5588e@canonical.com/ >> >> I understand that we might have here regression, if these were working >> as modules, but I don't think we ever really committed to it. We can as >> well make it non-module to solve the regression. > > Sure. But since v6.4 is around the corner, can we merge this > workaround for now, while a proper fix is being worked upon. DTS workaround? No. I don't agree. Once it is merged it will not be fixed. I am perfectly fine though with making the interconnect or even rpmh regulator bool instead of tristate. Best regards, Krzysztof