From patchwork Wed Mar 13 17:53:51 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Bryan O'Donoghue X-Patchwork-Id: 13591696 Received: from mail-wm1-f47.google.com (mail-wm1-f47.google.com [209.85.128.47]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id A45A25823C for ; Wed, 13 Mar 2024 17:53:55 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.47 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1710352438; cv=none; b=j5EEfe5lSCzUwwH/MfxAhsX9Hl7tGvjKM2gdt1rqs5hU0+dZfvN7yV94qoS7Gmymz/+5+ERfRPowe02+1cRl2icURJDV/Q6DqgGxDnwTZic1Y4DTZzpU7RiU8+5a2zo180at0fp1nBcrVnYgBXnTj6o0XK7jUfV41ooT1TepyBI= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1710352438; c=relaxed/simple; bh=m4MoUO1mvIPqt4fC1lB6nKiiGjCp4eGwXz6oqUc/OR0=; h=From:Subject:Date:Message-Id:MIME-Version:Content-Type:To:Cc; b=IZatGXcWQyzFnriGPbBq1qfZG4i+Sh7ntDw+KP7fO1WiPnJP9qAT8iNUA0FFd+BS07X7A5DFmDWw5epbuLk2U/l1dZjw2LXu355xnggS5XVSuK72ua5EMYshvxXw3kginOawfITzjYMFpTMZzkRIHTHupAZBzRPvyvvVwpWUv10= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=YtilXYlL; arc=none smtp.client-ip=209.85.128.47 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="YtilXYlL" Received: by mail-wm1-f47.google.com with SMTP id 5b1f17b1804b1-41324a16c9eso832335e9.0 for ; Wed, 13 Mar 2024 10:53:55 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1710352434; x=1710957234; darn=vger.kernel.org; h=cc:to:content-transfer-encoding:mime-version:message-id:date :subject:from:from:to:cc:subject:date:message-id:reply-to; bh=izT/XW0GrmGVL6Tl66ecjkPXN9CJyhGJnGTtICjCD3M=; b=YtilXYlLZ883k/3wcu1/eYrS6FgOJOBJfTT34FcrCWVND4zDBllNsKYffjUUaZ3OoQ 3U/BCFzqc+FQ4YPfFIMFBR9ZdnoFQA+5WSX/gRkhIGISb30IXHKZSFn21uR1LIaDoQvY 3FpDM+aNiRTznYxitnKSEcCkP3ZGLSi/LDVjzMUQoqrPhbVTSs05ZOJFLtC7vF63CTUV OcugCGa62B1IyvOG6ppdQJ+cJ+C7kJyjnH4dIDlojuDaz+qrduBAUFSubRdccMg/OYPL crD/KZMJZUR4UXVAjElqNxSE14T3IKWkOkfSOgR9TNbMbYy25+SzzVxcI6w4RsMEYPkj tFqQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1710352434; x=1710957234; h=cc:to:content-transfer-encoding:mime-version:message-id:date :subject:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=izT/XW0GrmGVL6Tl66ecjkPXN9CJyhGJnGTtICjCD3M=; b=S1mYHd+GhDcmKlEduHl9AJvWtoBmCGsXrp1rh48cbNg2w9kyWCIqwi/gkfVmpN+xP/ xBIBHxqpmixxSsXHR9pFFxIdbWTIvTqKFjA2xTHfK/C9F+e1lU9S/W/mc762X0mj6fg0 b2+i/zk1GH/yICXoZ02+67i5cTYn7kAbq9RXL7eSzQu3C7Zgz4yRxyqxhpgUoY9V7wMz 5Jos3CpcZvtmedihW6GCUJHwq9Pu9FPePGBssSsWuFbVKvx0dIUh3cqGlCoFv5jJ154f 41Gk5f2MQd2hU+II6Kf7+ciPoVDrThKauaYU7oQ8jgJYCsp6vNUUFfJwnh0EXaZ/PsGw hlaA== X-Gm-Message-State: AOJu0YyfnKmmUIPYl49CPBqcL/CodGZXtlwrJrvzTsKouOI1SDJ+LaNL vTfx2447x6BMN4bLOoC1BPjKwr6+QdNLxOdOj1ShdOjZii7wBkg4D8ywX3yW+V4= X-Google-Smtp-Source: AGHT+IHaKy+zzPYoBf25HdN7N6UQcsD5sQuS2StMY4WAgYUHdmUfCSSe9fFJKEDwU62nzkDMfpNiMA== X-Received: by 2002:a05:600c:4e4b:b0:412:ed3f:1b4b with SMTP id e11-20020a05600c4e4b00b00412ed3f1b4bmr474221wmq.11.1710352434035; Wed, 13 Mar 2024 10:53:54 -0700 (PDT) Received: from [127.0.0.1] ([176.61.106.68]) by smtp.gmail.com with ESMTPSA id u12-20020a5d6acc000000b0033e7a499deasm9914482wrw.109.2024.03.13.10.53.53 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 13 Mar 2024 10:53:53 -0700 (PDT) From: Bryan O'Donoghue Subject: [PATCH v2 0/2] qcom: clk: camcc: Fix power-domain definitions on x1e80100 Date: Wed, 13 Mar 2024 17:53:51 +0000 Message-Id: <20240313-linux-next-camcc-fixes-v2-0-9426da94ae37@linaro.org> Precedence: bulk X-Mailing-List: linux-arm-msm@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-B4-Tracking: v=1; b=H4sIAC/o8WUC/3WNSw6CMBCGr0Jm7ZjOgERceQ/DopYRJtHWtEgwh LtbSVy6/P7nAkmiSoJTsUCUSZMGn4F3BbjB+l5Qu8zAhitTksG7+teMXuYRnX04hzedJaGtjyV zQ3XFBLn8jLIZuXtpMw+axhDf289EX/U3Sf8mJ0KDHXFtxVyb8mDOOWhj2IfYQ7uu6weicQ7mv QAAAA== To: Bjorn Andersson , Konrad Dybcio , Michael Turquette , Stephen Boyd , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Vladimir Zapolskiy , Abel Vesa , Rajendra Nayak Cc: linux-arm-msm@vger.kernel.org, linux-clk@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Bryan O'Donoghue X-Mailer: b4 0.13-dev-26615 There are two problems with the upstream camcc implementation at the moment which this series addresses. 1. The camcc block has two power-domains MXC and MMCX however, the yaml description doesn't include MXC. 2. The code for the GDSC definitions for x1e80100 camcc fails to list the titan_top_gdsc as the parent GDSC of the other GDSCs. This series addresses both of those bugs. There is currently no upstream camcc dtsi for x1e80100 so the yaml change won't affect the ABI. Signed-off-by: Bryan O'Donoghue --- Changes in v2: - Incorporates Krzysztof's comment on removing "phandle" description - Added Konrad's RB - Moves the location of the allOf Editing the dtsi to test the yaml constraints showed the power-domains were not being picked up but putting the allOf before the required fixed that. - Listed the 8450, 8560 and 8280 under "-items" as testing shows adding too many power-domains to the dtsi was not being picked up by the automated checkers. - Tested with Too many power-domains in sm8450 Zero power-domains in 8550 Too few power-domains in x1e80100 - Link to v1: https://lore.kernel.org/r/20240311-linux-next-camcc-fixes-v1-0-d126ae0b9350@linaro.org --- Bryan O'Donoghue (2): dt-bindings: clock: qcom: Fix x1e80100 camcc power-domain declaration clk: qcom: camcc-x1e80100: Set titan_top_gdsc as the parent GDSC of subordinate GDSCs .../bindings/clock/qcom,sm8450-camcc.yaml | 37 ++++++++++++++++++---- drivers/clk/qcom/camcc-x1e80100.c | 7 ++++ 2 files changed, 38 insertions(+), 6 deletions(-) --- base-commit: 8ffc8b1bbd505e27e2c8439d326b6059c906c9dd change-id: 20240310-linux-next-camcc-fixes-a68322916421 Best regards,