From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mx0a-0031df01.pphosted.com (mx0a-0031df01.pphosted.com [205.220.168.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 84E4237B027 for ; Mon, 11 May 2026 12:13:53 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=205.220.168.131 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778501634; cv=none; b=k895TCKwRKO5DKOCX9dpj9Q9eN981HN4aNGqOMXafJ3QLH5qwWabyuwPZSuZo6GpYOCvoWLu5IKXZzyQU92CGBp0FNLNp9ZKvGtYPHEFNLZI/LgMqCtihVNHJZlHAYW9LW/zNTIKAs3mKaeMo0nQzcvHOdsVpia5LPuDZ0Nj7bQ= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778501634; c=relaxed/simple; bh=8fqo9qHfK/dEgTotXTuK3of6kfoYb5MbOk81UGuKZuA=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=RiH07sH+vCPsJl107bLuwDDLeziNS/y3tX7Y35PgPR1+w8ChnYofv8FPBInTe40/k8yRhS5M3TblbLFaB5xee1BbXCBse1Ln5Ivo+lR1y2Gs39fQgF/APvadwXTkeKIloQMGtMOX5SsIOpZzHJe93SdIxMPeV/EGviyRmAUqnSg= 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=OcAaV6ju; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b=b3pNYyIz; arc=none smtp.client-ip=205.220.168.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="OcAaV6ju"; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b="b3pNYyIz" Received: from pps.filterd (m0279866.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.18.1.11/8.18.1.11) with ESMTP id 64BC4HeF108021 for ; Mon, 11 May 2026 12:13:53 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= YbyuFd3QVsgjYT9UiU61WRQarqplB3d8903mQnfn4s8=; b=OcAaV6juC49sWjng 91BprsOUVo1QyE85snh+gAfNAWQwepWgLNEmVSYloxL22Orjt2rJX+z+qxVVfuf0 YG38S3CP6qvxUkaSMHuIAV0XibE+kSEp5hdMy3/viWkK7W9Rdss/fUAtXFDKebJc +e5fF/y1EBAyDJh7Jtb3iRVOlCMN3/V92OBb8aH1v0bHpmqkqZbudkZBG5PojduX SBJ6CIHdbySHTqiwyayy1km2NR1MlhtRPXV1KcMgznhoU65ZINdP8aqquULxcgdc 01/o6RORfLYMbUyvqLCgDqbK/hWa8yMIvGUlitdSsO0f7gI1bc5oJy5w7ewoUO4W Hdj48g== Received: from mail-vs1-f69.google.com (mail-vs1-f69.google.com [209.85.217.69]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 4e3esyr12a-1 (version=TLSv1.3 cipher=TLS_AES_128_GCM_SHA256 bits=128 verify=NOT) for ; Mon, 11 May 2026 12:13:52 +0000 (GMT) Received: by mail-vs1-f69.google.com with SMTP id ada2fe7eead31-634396ec8fdso29196137.0 for ; Mon, 11 May 2026 05:13:52 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oss.qualcomm.com; s=google; t=1778501632; x=1779106432; darn=vger.kernel.org; h=content-transfer-encoding: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; bh=YbyuFd3QVsgjYT9UiU61WRQarqplB3d8903mQnfn4s8=; b=b3pNYyIzF3BjlDSiD8Bbj3Qjguk2una0hPh6uCYjH3jMnYzYB2c4V2H8vIZiMRrfG3 +0pubbNTMhgXY6jUy9l3VV7stZ8mD8KEpRahWQAlumgNw79uzUIbm3C1l8qyE5sFbzw3 ok48gx2R1u0PY1B2YwBZqkLY0+WgSDH9gga8lWQrkY5yADc9azwLn0QMbciGcjds5osB rxlTkMT+imcvFN8sG56hd8z7DB0n6/LI69xDScxfAz9p0Dcma+kLLf/lAcJHfEuWRU3X DYqxCOUEIwJJeymNC9Eq8hr6SwHSS8+SR/lUlfzExu8CHmMgV8L8Qr31pksAvCnegKD2 VM0g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1778501632; x=1779106432; h=content-transfer-encoding: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; bh=YbyuFd3QVsgjYT9UiU61WRQarqplB3d8903mQnfn4s8=; b=jjSkXvyL7+BDJ5O3h5vTiqyFoplXRub2TXs7L5dlGI1/5ZUqkkVGx+qH6YTJvipGOl CgBu6SzcwQzu742t/dpMw7KUqlc0VYN14M3yg7x6VZyVWw5g/waUkZrLpGBfxpFljk77 sTxHEMjEk/zRheAB2VtorBWCPAutWyiA8D5EquSYqUhhh0psYOFu90afqMOh0HB/Gq7+ lsczs2e5drTgWmLbiyYc3GKPuZxy/SJzA3Giaw3qSOm/w0ICBX94CN01VSKDd8bl4x5R yaS2Xcmg1YjveQRQ92CSbyMjkymsiRMPiu0BTib2nY70B5iWs2o9plbdLcQ9oqC7FLtE hxgg== X-Forwarded-Encrypted: i=1; AFNElJ9ZAnu7pu0Ueh6Fl4mp2z/MfhrmWkqZjTW09+R2XvV0i9FShDzDqhl7wFdBp80TlHGFqPqzCFyWzPJ9@vger.kernel.org X-Gm-Message-State: AOJu0Yy82szBncC7DaiyvhWY/qCHszOyc2qd8MSaax6M4L/MzHxU1cqU E+ZJq7+wF8vqEIKBnjH7U0OJnSIt/P1zRR9krYWLWOmaaM/8o7Th3xaucdzSy++IdP+3yfxKc7E wLMbNs3fMKGmzXQYdA5TtTBee1HvHoIgd2+h3br6k4F0wXF0WP/v2qgbrqIiesiuv X-Gm-Gg: Acq92OGX067IYa0rqvmBuhuoeP495GdH8hc6BASLzM/ouNbDgMzXNl42mazIpdNv2PC MGTdMHBWOiGPHvQT0kxIRVTW37L2lMGyk6HIsRI2RmppICrQvXvbLQPEu0B3UVHd9RGxUDnMn3j sqUeYGvAyckB8pjhEnIaIA5bq70aCmWI9alZPWvnqMfF5Egwu4R7Ez9TyiXxlnnwzaJZJRf70DH Fvprzdz3//1g/baC2bP1xWTpfE0wGahRDW5ONJXfvnoUntGKjAfoXRkMljkT9gfoclHy4uVNCXY SB+mWw9vjWcBSIuex6sUlAzZUeo+oGTzDHJgrfce32BYTJxsAtLDMwrArEiH4g4HmNDg5GCfuPI sd1FZSalFdYx6v1buI4QqJF5N9U1cWEi15wRleSgHoTn3J0x7Nqr3th3Mdrcqc3rGi/seU/macB D8ghY= X-Received: by 2002:a67:e24b:0:b0:612:13af:f5de with SMTP id ada2fe7eead31-630fad442d5mr3535133137.3.1778501631865; Mon, 11 May 2026 05:13:51 -0700 (PDT) X-Received: by 2002:a67:e24b:0:b0:612:13af:f5de with SMTP id ada2fe7eead31-630fad442d5mr3535117137.3.1778501631385; Mon, 11 May 2026 05:13:51 -0700 (PDT) Received: from [192.168.119.254] (078088045245.garwolin.vectranet.pl. [78.88.45.245]) by smtp.gmail.com with ESMTPSA id a640c23a62f3a-bcac02c832asm498322166b.11.2026.05.11.05.13.48 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 11 May 2026 05:13:50 -0700 (PDT) Message-ID: Date: Mon, 11 May 2026 14:13:47 +0200 Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v8 2/6] platform: arm64: Add driver for EC found on Qualcomm reference devices To: =?UTF-8?Q?Ilpo_J=C3=A4rvinen?= , Anvesh Jain P Cc: Sibi Sankar , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Hans de Goede , Bryan O'Donoghue , Bjorn Andersson , Konrad Dybcio , Randy Dunlap , linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, LKML , platform-driver-x86@vger.kernel.org, Maya Matuszczyk , Dmitry Baryshkov , Akhil P Oommen References: <20260427-add-driver-for-ec-v8-0-702f74e495f7@oss.qualcomm.com> <20260427-add-driver-for-ec-v8-2-702f74e495f7@oss.qualcomm.com> <91523c61-bd03-2d96-823e-78ecd9e3798f@linux.intel.com> <17397c91-53e2-4786-b7c8-88c302821e22@oss.qualcomm.com> <88ce2f85-5d8f-9845-cdb8-77051989e57f@linux.intel.com> Content-Language: en-US From: Konrad Dybcio In-Reply-To: <88ce2f85-5d8f-9845-cdb8-77051989e57f@linux.intel.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-Proofpoint-Spam-Details-Enc: AW1haW4tMjYwNTExMDEzNSBTYWx0ZWRfX4OOsqT+JN34Y vZSngIapuRnaY9PcVckNbUn0Fq+0g0/uyBIWIJD/JEeRycKUvbgPOvzyE4ba9/cUYdtKxY1gjfb JX/0QJ1g3mEALbhL15Bb8eYuWZF4/MBWInQmOA6KnDEuWExpsTsH75JALJy0gSKi2UNrXbKHW6w T49FXEiIKirb+15iCJtSTo2FU53VS3DVuwyTlQZYS8R77qsbC/237kCQrwZMNapn31yM8NGLV39 b6KiKlNL1BLCX/U9Uw1rOedPDWcVQxJx78Toz3lS3TJMnS9LI80Dbp9yj6D04yizUthPWHUZOg7 qjeqjg4nd4Bs2jYx/hWW4nxq7cubIlL9LUrRf1Rk1FDc/GpiG9YpPkh5ZgAP7q8G7v37xhPeXvN Jv1ZqpuTqJbSESQJKD6ADQfure2Jp1nrbv3/KFLWy1t63/2beV5gjV4XoGW80EQdCpf4aWsl1Wn D6stkuAvoZ/qHyrcFbg== X-Authority-Analysis: v=2.4 cv=AI/DAN7A c=1 sm=1 tr=0 ts=6a01c800 cx=c_pps a=5HAIKLe1ejAbszaTRHs9Ug==:117 a=FpWmc02/iXfjRdCD7H54yg==:17 a=IkcTkHD0fZMA:10 a=NGcC8JguVDcA:10 a=s4-Qcg_JpJYA:10 a=VkNPw1HP01LnGYTKEx00:22 a=u7WPNUs3qKkmUXheDGA7:22 a=YMgV9FUhrdKAYTUUvYB2:22 a=VwQbUJbxAAAA:8 a=EUspDBNiAAAA:8 a=ZGoIsIOgen_iXHErvcUA:9 a=3ZKOabzyN94A:10 a=QEXdDO2ut3YA:10 a=gYDTvv6II1OnSo0itH1n:22 X-Proofpoint-GUID: tcXvn8vrwsHE_m3lzTseFFhiJ1ubtj1G X-Proofpoint-ORIG-GUID: tcXvn8vrwsHE_m3lzTseFFhiJ1ubtj1G X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1143,Hydra:6.1.51,FMLib:17.12.100.49 definitions=2026-05-11_03,2026-05-08_02,2025-10-01_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 impostorscore=0 suspectscore=0 adultscore=0 spamscore=0 bulkscore=0 clxscore=1015 phishscore=0 priorityscore=1501 malwarescore=0 lowpriorityscore=0 classifier=typeunknown authscore=0 authtc= authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.22.0-2604200000 definitions=main-2605110135 On 5/11/26 2:07 PM, Ilpo Järvinen wrote: > On Mon, 11 May 2026, Anvesh Jain P wrote: [...] >>>> +static int qcom_ec_sci_evt_control(struct device *dev, bool enable) >>>> +{ >>>> + struct i2c_client *client = to_i2c_client(dev); >>>> + >>>> + return i2c_smbus_write_byte_data(client, EC_SCI_EVT_CONTROL_CMD, !!enable); >>> >>> This converts bool -> u8 using C's implicit conversion rules (plus on top >>> of that does unnecessary !!). Please write the conversion explicitly, you >>> can use ?: syntax for brevity. >>> >> >> The original v4 code used `enable ? 1 : 0` explicitly, but Konrad Dybcio >> suggested switching to `!!enable` during his review [1]. Happy to revert >> to the explicit form if that's the preferred style — just want to flag >> the conflict so everyone is aligned. >> >> [1] >> https://lore.kernel.org/all/6eb3a173-c364-431f-93e4-7bbb7a32431e@oss.qualcomm.com/ > > I prefer to have the boolean to binary conversion done explicitly > (with the platform drivers maintainer hat on). And I won't object if you prefer that Konrad