Received: by 2002:a05:6500:1b8f:b0:1fa:5c73:8e2d with SMTP id df15csp130756lqb; Tue, 28 May 2024 10:42:41 -0700 (PDT) X-Forwarded-Encrypted: i=3; AJvYcCWYe50Th/xOtkwmSEIfj64frIKdC10rce/GM9GHfD105EJaX6XWP+eyhvmDDzn0DmoQ1KmLFGUryb0MyUQOQwexXRU6PlSDnLdjO68gtw== X-Google-Smtp-Source: AGHT+IGYx/yJsrwMYnZOnUtUUtCZE4CFWyhU8/w1hEU+hGLG/ZoYFEc95O1+iXuCMBt4PRSm0/bP X-Received: by 2002:a17:903:32ce:b0:1f3:488a:bc54 with SMTP id d9443c01a7336-1f448d3510bmr158710785ad.36.1716918161572; Tue, 28 May 2024 10:42:41 -0700 (PDT) ARC-Seal: i=2; a=rsa-sha256; t=1716918161; cv=pass; d=google.com; s=arc-20160816; b=bK03Px+WcwKLGN7NldWS+4iUdAGi0xpwO5pVSZT87gcKmSgL/CfsPil0FZsLirPnYP R/fqzr+Od06SvmkUuRmXElkC72H0rOrN4aa/S/xD8ROqW760VWP/wRs7eYVJuKu6u1YU cuwZySJKbYggfL1mKP0TN+yfQifZb9Vi1AbgfWK8fksUfx5EJsgWWUHXUoMm02khbvr9 P7YjE/QMkJD/3b4voVZXAGvu3mnzBFDlmR2LWNEYNv9iDGVCo2endvcGDbvprusPjT2d BVAHKaF7TkMrmTpfwEEium9+ETKLzTPGlLKXIGIO9iWX4rC1K1ZOcGZ3WFVNJRQspdtl azRg== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=subject:message-id:references:in-reply-to:cc:to:from:mime-version :list-unsubscribe:list-subscribe:list-id:precedence :content-transfer-encoding:date:dkim-signature; bh=EKUG5roaaBLZjBPWh9Lv+sElSIBZH7ge1b1QVr3XYT0=; fh=jLiww682WpgBnlwuYSDnuDQ2LlGEXoPQ1IhPkgTVDB0=; b=qISjbjVrOjraoxEjrNpO8dxogyhqyR9A9ULQsFaV4SCiaFNbXRwUqGRut16nEnoWKH 09hW8rVXB+GbB4ShW8xLN001QEp0THtnkUFVeC9sru9y9T/byWETbSijY8vWymQbCAip fIdu2D1rfqdoUAwlPn2vHrOpzeFS/PL606kGmHZd+imy9wJAC5QSMs+RoWIU8RNXoK3h ydQ/SYddEGMOtFO0VHOrd8Xm5MiPDM16JzpLFG1m+/8rnKhm6B2J8TSmxnefVUdEK5UX rr3S9tnKCdbpWo+n4tHoGh2+KT7VqnazAr6qrCHPqZOUh/kP77rFNWMYo4Y5D7OhoDO+ RLOQ==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b="I/Xp5bRw"; arc=pass (i=1 dkim=pass dkdomain=kernel.org); spf=pass (google.com: domain of linux-kernel+bounces-192818-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) smtp.mailfrom="linux-kernel+bounces-192818-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from sv.mirrors.kernel.org (sv.mirrors.kernel.org. [139.178.88.99]) by mx.google.com with ESMTPS id d9443c01a7336-1f44c78b45csi81980795ad.5.2024.05.28.10.42.41 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 28 May 2024 10:42:41 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel+bounces-192818-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b="I/Xp5bRw"; arc=pass (i=1 dkim=pass dkdomain=kernel.org); spf=pass (google.com: domain of linux-kernel+bounces-192818-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) smtp.mailfrom="linux-kernel+bounces-192818-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by sv.mirrors.kernel.org (Postfix) with ESMTPS id 3B675286324 for ; Tue, 28 May 2024 17:42:41 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id E9646171084; Tue, 28 May 2024 17:41:46 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="I/Xp5bRw" Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (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 DA32A47F60; Tue, 28 May 2024 17:41:45 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1716918106; cv=none; b=DfPtZk7f2FpMs2XZFAHU1CZsaHn5PKXjaY+sSOVA0JVN2nlGTwxye7XDxDMv3aDkfHOJpqwWbesN1098YJ1k+darZvk6H79S7Y5QWOWp6pdepg/+TiY56/p8El8rSc4PXlsqlRKWwKJYPpjHQTFRM3b3gBrwJ4w5CfsZp+tEF6Y= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1716918106; c=relaxed/simple; bh=RLYbWmtB1JSsSqTY7dPzl/obl0aq3fUopxD+P3sLiE8=; h=Date:Content-Type:MIME-Version:From:To:Cc:In-Reply-To:References: Message-Id:Subject; b=I01LRYJOUO4uqDTLrGj9TbFBsc8BSL3M4J7QaQuTDzOvW/NRX499p6PrUMgNsGJ1w5ACLwNeReNmBEaSbmKDJTdnAoi4hft2yIAtF7gHy1plaekBeUzZNZhfao0r/hy/fbhv590ZHcwxi2o25ddxglF/VYkJi26tkCBvz3tU8cI= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=I/Xp5bRw; arc=none smtp.client-ip=10.30.226.201 Received: by smtp.kernel.org (Postfix) with ESMTPSA id 2F78EC32782; Tue, 28 May 2024 17:41:45 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1716918105; bh=RLYbWmtB1JSsSqTY7dPzl/obl0aq3fUopxD+P3sLiE8=; h=Date:From:To:Cc:In-Reply-To:References:Subject:From; b=I/Xp5bRwm2CmB1DjEPopS7cv+o9A3G5woq4FBaf3o60k1PKxJ9QSoeWj3e9TfOpxC vwNP6Ne0lfNrOp1cetrrdIDRMIvHzK9GfEJurIMsSsqZEZkEq274cNH6vLRfsQs82B 2QnrAO5gtjo+YZoG+W91mS1EdgX2f6Pnmwai/k28Rnof9n/GAdHnVen9Vq3WRCDaex rXB77GDTMq8CIRDIsPzkbzVDxaLdrwAWbCXH9OPXWLVX7Qh22I2tMRePCP6PmHRQ6L Wb1dKsMEDxQ2vrPyGNaWyn47q0MCzC1h8vXIOj2E8w50nFiy7ECKKr83zgXJaPhf5G Abxys47sO/qRw== Date: Tue, 28 May 2024 12:41:44 -0500 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 From: "Rob Herring (Arm)" To: Dmitry Baryshkov Cc: Bjorn Andersson , Greg Kroah-Hartman , Bryan O'Donoghue , Heikki Krogerus , Conor Dooley , linux-kernel@vger.kernel.org, Sebastian Reichel , Konrad Dybcio , Hans de Goede , devicetree@vger.kernel.org, linux-pm@vger.kernel.org, Nikita Travkin , Krzysztof Kozlowski , Krzysztof Kozlowski , =?utf-8?q?Ilpo_J=C3=A4rvinen?= , linux-usb@vger.kernel.org, platform-driver-x86@vger.kernel.org, linux-arm-msm@vger.kernel.org In-Reply-To: <20240527-yoga-ec-driver-v3-0-327a9851dad5@linaro.org> References: <20240527-yoga-ec-driver-v3-0-327a9851dad5@linaro.org> Message-Id: <171691793418.1180687.12564348124831502092.robh@kernel.org> Subject: Re: [PATCH v3 0/6] power: supply: Lenovo Yoga C630 EC On Mon, 27 May 2024 13:03:45 +0300, Dmitry Baryshkov wrote: > This adds binding, driver and the DT support for the Lenovo Yoga C630 > Embedded Controller, to provide battery information. > > Support for this EC was implemented by Bjorn, who later could not work > on this driver. I've picked this patchset up and updated it following > the pending review comments. > > DisplayPort support is still not a part of this patchset. It uses EC > messages to provide AltMode information rather than implementing > corresponding UCSI commands. However to have a cleaner uAPI story, the > AltMode should be handled via the same Type-C port. > > Merge strategy: the driver bits depend on the platform/arm64 patch, > which adds interface for the subdrivers. I'd either ask to get that > patch merged to the immutable branch, which then can be picked up by > power/supply and USB trees or, to make life simpler, ack merging all > driver bits e.g. through USB subsystem (I'm biased here since I plan to > send more cleanups for the UCSI subsystem, which would otherwise result > in cross-subsystem conflicts). > > --- > Changes in v3: > - Split the driver into core and power supply drivers, > - Added UCSI driver part, handling USB connections, > - Fixed Bjorn's address in DT bindings (Brian Masney) > - Changed power-role for both ports to be "dual" per UCSI > - Link to v2: https://lore.kernel.org/linux-arm-msm/20230205152809.2233436-1-dmitry.baryshkov@linaro.org/ > > Changes in v2: > - Dropped DP support for now, as the bindings are in process of being > discussed separately, > - Merged dt patch into the same patchseries, > - Removed the fixed serial number battery property, > - Fixed indentation of dt bindings example, > - Added property: reg and unevaluatedProperties to the connector > bindings. > - Link to v1: https://lore.kernel.org/linux-arm-msm/20220810035424.2796777-1-bjorn.andersson@linaro.org/ > > --- > Bjorn Andersson (2): > dt-bindings: power: supply: Add Lenovo Yoga C630 EC > arm64: dts: qcom: c630: Add Embedded Controller node > > Dmitry Baryshkov (4): > platform: arm64: add Lenovo Yoga C630 WOS EC driver > usb: typec: ucsi: add Lenovo Yoga C630 glue driver > power: supply: lenovo_yoga_c630_battery: add Lenovo C630 driver > arm64: dts: qcom: sdm845: describe connections of USB/DP port > > .../bindings/power/supply/lenovo,yoga-c630-ec.yaml | 83 ++++ > arch/arm64/boot/dts/qcom/sdm845.dtsi | 53 ++- > .../boot/dts/qcom/sdm850-lenovo-yoga-c630.dts | 76 ++++ > drivers/platform/arm64/Kconfig | 14 + > drivers/platform/arm64/Makefile | 1 + > drivers/platform/arm64/lenovo-yoga-c630.c | 279 ++++++++++++ > drivers/power/supply/Kconfig | 9 + > drivers/power/supply/Makefile | 1 + > drivers/power/supply/lenovo_yoga_c630_battery.c | 476 +++++++++++++++++++++ > drivers/usb/typec/ucsi/Kconfig | 9 + > drivers/usb/typec/ucsi/Makefile | 1 + > drivers/usb/typec/ucsi/ucsi_yoga_c630.c | 189 ++++++++ > include/linux/platform_data/lenovo-yoga-c630.h | 42 ++ > 13 files changed, 1232 insertions(+), 1 deletion(-) > --- > base-commit: 8314289a8d50a4e05d8ece1ae0445a3b57bb4d3b > change-id: 20240527-yoga-ec-driver-76fd7f5ddae8 > > Best regards, > -- > Dmitry Baryshkov > > > My bot found new DTB warnings on the .dts files added or changed in this series. Some warnings may be from an existing SoC .dtsi. Or perhaps the warnings are fixed by another series. Ultimately, it is up to the platform maintainer whether these warnings are acceptable or not. No need to reply unless the platform maintainer has comments. If you already ran DT checks and didn't see these error(s), then make sure dt-schema is up to date: pip3 install dtschema --upgrade New warnings running 'make CHECK_DTBS=y qcom/sdm850-lenovo-yoga-c630.dtb' for 20240527-yoga-ec-driver-v3-0-327a9851dad5@linaro.org: arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dtb: pinctrl@3400000: ec-int-state: 'oneOf' conditional failed, one must be fixed: 'bias-disable', 'function', 'input-enable', 'pins' do not match any of the regexes: '-pins$', 'pinctrl-[0-9]+' False schema does not allow True from schema $id: http://devicetree.org/schemas/pinctrl/qcom,sdm845-pinctrl.yaml#