From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mx0b-0031df01.pphosted.com (mx0b-0031df01.pphosted.com [205.220.180.131]) (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 8264B407CCC for ; Wed, 12 Aug 2026 10:50:02 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=205.220.180.131 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786531804; cv=none; b=Cl7KU6qfc4sMw42BYsXIhB9x9FrssFR33vurvPt4W1rQ5CPNGSdX594ilypH3+bi0pEjm3uhKQi+g2miMcom5MJ41zkWzlzpHGCSP+DVJZUkn+IaZ1lahH5G0Ys+b9h5bHoo221hgnxfmAWhCkHF2I+LFW4XcFwuhZeTim33ALM= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786531804; c=relaxed/simple; bh=29uv9rNNOe8cJpC4RVfKo9oF63UjVRrTJfBnFmSK9d0=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=FmkrUn8TFmz54OhI1hBrC7X3QxpiIDfA13M5Kl7m988ztpf5zmAWHNOqLFr3RRoDVO59WYu+RP9uGmfEuz+5fPMy/TklyvmqimDXXbp2GxQ0AhqG1nqpFKyOmperEWlZa2tu6f7mPCkrJGcwFyOxyBfEwEF4I0Txa1O8KMlPyEY= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=oss.qualcomm.com; spf=pass smtp.mailfrom=oss.qualcomm.com; dkim=pass (2048-bit key) header.d=qualcomm.com header.i=@qualcomm.com header.b=QqC/Esph; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b=d6c9Boz8; arc=none smtp.client-ip=205.220.180.131 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=oss.qualcomm.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=oss.qualcomm.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=qualcomm.com header.i=@qualcomm.com header.b="QqC/Esph"; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b="d6c9Boz8" Received: from pps.filterd (m0279869.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.18.1.11/8.18.1.11) with ESMTP id 67C9gJUe2212249 for ; Wed, 12 Aug 2026 10:50:01 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=qualcomm.com; h= cc:content-transfer-encoding:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to; s=qcppdkim1; bh= UbG04XwAubXPmna3vE6vThuJ/BN7DybPLAShgSUkaz0=; b=QqC/EsphNBDPZwQi hmdOM4XHdWRL1y0oej7g++dzUdx/2Ymb3Q7wQiy8FrftGJsa8ZBrLB5acbltG3A0 IRNx4mvCekL3YU8frd76WjRnF+z5LXsFcwCBfsawEkNTZaJrq7BaFww8NtydBTTF v/R/EzFt03ofNMzDiIljgufl2GEM+Oc6L79H5J9pCQ5eUv6gmVB95KGqxx1l2PUy Z4cJvuKVYa8LJyDKNmET9iNKkseb/w5IRkVfqM7z/Dgz7q1gcco0zLRPMn5h3vQ8 dsotr9efOn/3Kgy/7U+mx5gMpwoAbDoKSZx31h2cqgyodwMBRXf4g9wTB6UHYRk5 zn/nfA== Received: from mail-pj1-f70.google.com (mail-pj1-f70.google.com [209.85.216.70]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 4g0hfw1er2-1 (version=TLSv1.3 cipher=TLS_AES_128_GCM_SHA256 bits=128 verify=NOT) for ; Wed, 12 Aug 2026 10:50:01 +0000 (GMT) Received: by mail-pj1-f70.google.com with SMTP id 98e67ed59e1d1-38f5ac7416eso1021119a91.3 for ; Wed, 12 Aug 2026 03:50:01 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oss.qualcomm.com; s=google; t=1786531797; x=1787136597; darn=vger.kernel.org; h=content-transfer-encoding:content-type:in-reply-to:from :content-language:references:cc:to:subject:user-agent:mime-version :date:message-id:from:to:cc:subject:date:message-id:reply-to :content-type; bh=UbG04XwAubXPmna3vE6vThuJ/BN7DybPLAShgSUkaz0=; b=d6c9Boz8c80UM32bSniEcMksrAhA1aKuvAa4Esx/uQjpP/PLWHBNYnzjxq+77bZZgv vRrCNQEJ8hBPJVN25++2jGjYmFz4g8PFk+pQlVx83E0iQHxQ1AjtR4SSzf37bhD4bHMs qWpuIJTTL37vinkOaJczc/NyDdBwvrHImjPex7SInkN2cSlRRr8aOUHe0jNNdT7el893 qBR64VPTPrbS7GD3zCvMsymwS1/nHHgJBrpz5jKfVyowEOCO6XifIYMK/+FO7vobM+lu MG/+tmy9q9Hh39jX88zhxCsun9iNcxCz9Clez/0cVVsag0eifWXnskir5/Dp++8NJvQe jzBg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1786531797; x=1787136597; h=content-transfer-encoding:content-type:in-reply-to:from :content-language:references:cc:to:subject:user-agent:mime-version :date:message-id:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to:content-type; bh=UbG04XwAubXPmna3vE6vThuJ/BN7DybPLAShgSUkaz0=; b=mhMY1OVtORK4qUtnoVm9QJuXAX9eQhJ34/DQrlUJEWeIrDBprS0N9t2egI4KtvV+cC RQkO582WmeF+6xPecGtAiphuxIsUfnuxONja5oUISdsNbD2kphG9weAu7uUiT4ZjCDHh iHtenANvfwxSQlpKlnFRGhl6pNElvDgsWPfN8T1NR4fPMjEfqQTyHRZKTPemC5P4fBCz Yk2vgArAEdJANS10bHfvAy3eWWgaLaYhjPIs+Ws0hZjAiF+u1VQBELFNP6u373nZWKWu Anks3ttve3FHz3ZXtxlCcEnQr8FoPTidGeC526TEWlrk0fD14CgontOWVnAFXy/dnVQv j4Ig== X-Forwarded-Encrypted: i=1; AHgh+RpBmarfvEwJdC6/4VbUWXJ44xfx51qWjsa9NmIy+gkuA4ilEo+p2ZEF7rNMd426IC7MrwyyeQKEN8ByR9k=@vger.kernel.org X-Gm-Message-State: AOJu0YzlqMTGXxGbuosZfHcKkGOhklFnng+sww7n5zqa7WuGB1/vRye2 RMqIZ88lbzZVD8QNDUt1CzFe2+J5UG1FycMTISWiMnk150n0jPh5bF7+e4HNoskIqaBBlg3yRvl YX45AkcfJsoSYLxtZAwwtxMNaHDcgO42ncY4YJSCvMWlWuW0bwPFvMTYpF03NfFit13I= X-Gm-Gg: AR+sD11tlORJvu7zYVZIDZZmY0wc5H9NQEb7oJbMTEofB4Ufaj3rjcgL2i1qOkXPFjc Z1mNwMARCTQxNkCkoOh7l4L2W3doDCHXOT+dfxVEy8iP8Ye3n5YUyMIXn0BR4Rs15wAgsFz5ePW aJLbGCk1fi2TiyrVd5w6D8vcTKdKCyaazL9vaamZwZb1ONPRZfhj1qNwP6RDNDrrW/AY8ylKrwJ hUuYYgKI5Jc6YaNVmeydTLBmxP9FsLHYwUg7cgsQ6lvQrWquq6+OKVkGVIxD/RwwdNjimDB/l0G 3r6DnbevIxicoNgF401H/AOS+3ZfoG1lFgL7RHANdNiiSc/fpHvvgGfiNYfo8ipcrQslqFiumy5 z/axnewVfKUDWqgJb4uz9yTPwDAMtYHci X-Received: by 2002:a17:90b:164b:b0:37f:b0a9:1a68 with SMTP id 98e67ed59e1d1-3930128c13cmr3916866a91.7.1786531796747; Wed, 12 Aug 2026 03:49:56 -0700 (PDT) X-Received: by 2002:a17:90b:164b:b0:37f:b0a9:1a68 with SMTP id 98e67ed59e1d1-3930128c13cmr3916838a91.7.1786531796248; Wed, 12 Aug 2026 03:49:56 -0700 (PDT) Received: from [10.217.217.28] ([202.46.22.19]) by smtp.gmail.com with ESMTPSA id 98e67ed59e1d1-392f8e3f565sm3191597a91.13.2026.08.12.03.49.49 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 12 Aug 2026 03:49:55 -0700 (PDT) Message-ID: <6978a578-05ed-49ad-a9dd-814d518cea8a@oss.qualcomm.com> Date: Wed, 12 Aug 2026 16:19:48 +0530 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v4 0/3] Introduce EC driver for Snapdragon X1E based Dell XPS 13 9345 To: Aleksandrs Vinarskis Cc: linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, platform-driver-x86@vger.kernel.org, Abel Vesa , Tobias Heider , Val Packett , Neil Armstrong , Stephan Gerhold , linux-hardening@vger.kernel.org, Krzysztof Kozlowski , Bjorn Andersson , Konrad Dybcio , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Hans de Goede , =?UTF-8?Q?Ilpo_J=C3=A4rvinen?= , Bryan O'Donoghue , Kees Cook , "Gustavo A. R. Silva" References: <20260809-dell-xps-9345-ec-v4-0-88306dbbc7aa@vinarskis.com> Content-Language: en-US From: Jishnu Prakash In-Reply-To: <20260809-dell-xps-9345-ec-v4-0-88306dbbc7aa@vinarskis.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-Proofpoint-Spam-Info: AW1haW4tMjYwODEyMDA4NyBTYWx0ZWRfXwpjtCoWUzC1b ELGaG8ofS5VKrVXiaElrVfx+zYADfSaup8/uZicwMVBf9ryriCzIrWIItz+mwPo7DEhdgym0VJM npDWcQJ/LRRrQWUHBCO+swuzy+op4zI= X-Proofpoint-GUID: JUPh1FEZK4mPIi2WxjptKngIGiq10Pvy X-Authority-Analysis: v=2.4 cv=fpDsol4f c=1 sm=1 tr=0 ts=6a7c4fd9 cx=c_pps a=0uOsjrqzRL749jD1oC5vDA==:117 a=fChuTYTh2wq5r3m49p7fHw==:17 a=IkcTkHD0fZMA:10 a=Sv0fKeRqtYgA:10 a=s4-Qcg_JpJYA:10 a=VkNPw1HP01LnGYTKEx00:22 a=u7WPNUs3qKkmUXheDGA7:22 a=_glEPmIy2e8OvE2BGh3C:22 a=VwQbUJbxAAAA:8 a=EUspDBNiAAAA:8 a=I76Qk8w-AAAA:8 a=FXFtw1A4AdT0iKmZFxQA:9 a=QEXdDO2ut3YA:10 a=mQ_c8vxmzFEMiUWkPHU9:22 a=vUPM0Wvl0xcrLs4nqPIT:22 X-Proofpoint-ORIG-GUID: JUPh1FEZK4mPIi2WxjptKngIGiq10Pvy X-Proofpoint-Spam-Details-Enc: AW1haW4tMjYwODEyMDA4NyBTYWx0ZWRfX9oZQVSxxJjAo 3Y7rScdoCYIEIBWhh+ynUGy+sQ9W8akIGV3TFpJKG5GEnd7TFVoC/qKk6RomDmbx+4EysO2QZtO zXuRU54nMKy48yzPht0dwyVXJzD8Kl/oiSoZnETR4NDn5kP3Bca9pse+kcrM5JyKyaqAjMlxIbT cuzOS3Jrf5ddTZcqlFH6mv4TQukVF+wfjn/S6Z+PCIRP8HOBuW7TnnLtVc/qeTdK39H3saMogSa Ye32ytoUf0BsD46VkhK0sXMOAT1dPK8m1NO1rlQiHr9R4DBbKH/7WU2vwX3oqtCDshneF9m7FL7 l4vSBffq/5O/+xlTf4GAOXYsOqYSAXU70KtWzwhKj8kakQ16nmjwE7SlRKU8Nnj8Yvm5xYtucy3 vIYjIsyza4bVmJ1SBMa3AJXzuZhHg7nBrckk2vRO5DTyiBtlvMW61NULlZ7EbCFTB7bz3xIqcvq o7l2fisADZPyGCu0ZaA== X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1176,Hydra:6.1.134,FMLib:17.12.100.49 definitions=2026-08-12_03,2026-08-10_03,2025-10-01_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 clxscore=1015 priorityscore=1501 spamscore=0 impostorscore=0 lowpriorityscore=0 suspectscore=0 adultscore=0 malwarescore=0 phishscore=0 bulkscore=0 classifier=typeunknown authscore=0 authtc= authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.22.0-2606150000 definitions=main-2608120087 On 8/9/2026 7:02 PM, Aleksandrs Vinarskis wrote: > This series adds Embedded Controller (EC) driver for Dell XPS 13 9345. > While EC appears to control most of device's peripherals, particular > driver addresses power and thermal managment issues. Key operational > principle involves initial thermistor constants configuration followed > by a periodic reporting of these onboard thermistor values from across > the motherboard to the EC. The latter then handles fan ramp-up/ > ramp-down internally. Suspend/Resume must be likewise propagated to EC > for power management. > > The driver was developed primarily by analyzing ACPI DSDT's _DSM and > i2c dumps of communication between SoC and EC during various stages of > operation (bootup, suspend, resume). > > With EC driver in place, the following issues are addressed: > 1. Fans were not properly cooling the laptop, would kick in late and > spin lazily, resulting in heavy throttling. With EC driver fans > start sooner and hit high RPM under heavy load. > 2. Fans were not stopping once SoC temperature dropped, they would keep > slowly spinning irrespective of suspend and/or closed lid until the > next powercycle. With EC driver shortly after SoC temperature drops, > thermistors temperature drops, and fans ramp-down. > 3. Keyboard and touch row backlight were not turning off during > suspend - only lid close would power off the touch row. With EC > driver behavior matches that of Windows, suspending device with lid > open powers off the peripherals. > > As thermistor readout depends on pmic's ADCs, this series introduces > EC driver and its schema, adds missing ADC to hamoa-pmics, and finally > adds thermistor and EC nodes to x1e80100-dell-xps13-9345.dts. > > Additional findings: > - Max fan speed depends on Dell's power mode settings, configurable in > BIOS or using Windows app (relies on ACPI-WMI). It appears best > cooling performance is achieved under 'Ultra Performance' profile. > - When the said power mode is changed using Windows app, EC IRQ is > triggered. Windows performs what appears to be thermistor contants > readout, though its not obvious what it is used for. > - Given similarities between Dell XPS 13 series (codename 'tributo') > and Snapdragon-based Latitude, Inspiron ('thena'), including matching > EC address and response to suspend/resume command the EC driver can > be likely used for both, though in-depth testing on 'thena' is > required. > Hi Alex, > This series depends on QCOM SPMI PMIC5 Gen3 ADC [1], which was added to > `linux-next` for v7.1, but must be picked if this series to be backported. > > [1] https://lore.kernel.org/all/20260209105438.596339-1-jishnu.prakash@oss.qualcomm.com/ > Just a minor point - this link points to V11 of the patch series, but please note V10 was the version of the series actually picked in the kernel. The difference between V10 and V11 is minimal though. Thanks, Jishnu > Signed-off-by: Aleksandrs Vinarskis > --- > Changes in v4: