Received: by 2002:a05:7412:cfc7:b0:fc:a2b0:25d7 with SMTP id by7csp2166483rdb; Tue, 20 Feb 2024 21:37:17 -0800 (PST) X-Forwarded-Encrypted: i=3; AJvYcCUqbIKlhWC90UeCRo6DHeiDpvOAGc5JjAto3fQVb+VorSlkzcE/FXRNsjjH00l8B/OqOWWRIfJJ/2jFLp/Fyt4d2Ev+C6EoiWuVV7rM9g== X-Google-Smtp-Source: AGHT+IEdxx4EwrgundBHov8ud+fy5E1D3hnJHAWJDuptXZMUxkfCxMn1IXtnuNCU9cS72Ukn1ta2 X-Received: by 2002:a67:f60f:0:b0:46d:30b0:af9f with SMTP id k15-20020a67f60f000000b0046d30b0af9fmr14766786vso.9.1708493837099; Tue, 20 Feb 2024 21:37:17 -0800 (PST) ARC-Seal: i=2; a=rsa-sha256; t=1708493837; cv=pass; d=google.com; s=arc-20160816; b=PFihLbL6f8/jKI3kadw8wKJMm6NnONuBV0dl1FYoHc1YMBfR56K4KUwe/9JEQqLcWR lU0WEpViQ14pkoc2OKMyk6UzPFd+iFZc7ZvZBDb5LtcsSULGl+hzUqTe01B3MC2O08DW T2LZL/KhVVthrrX61h6KIDQA381lXPhLwJwa9tZ0+nuGsn4g85pJdv6c9075dhgwExL8 jcrecsAY3+yiOXGR04IVx0/hXZpMiS5PDapUl0kzLZDg2e/aKOmHZ/ztax1+2HH6FY5k OciZqsKbwb1izNnK8A8pQ81zTUgHIHfwnqN+NXfIcH4BAHYPVVY/uowBORofYR1sc96V s11g== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:date:message-id:dkim-signature; bh=S5n/WpV8JcN0mx2fE3uU/d+OZrGuwRrR1d3N//9CiFA=; fh=5jpJdPUeMq+ohoE/23HtTN0VD5QG71oXLKJyjXqnB6g=; b=Scl04tX8kA65fkYPnE4sUyiFYU5pgmFh5AtYhiImZiyf+2nihNc7+18jz6jT8/PIsY awphmCG9uYbIvQoEpn/+Vu1caBQ9NsLnsyyg5jCowUrMUeSruhx+hfnbJUIZND3W79Am MKMRtdxVAJIS9NhKEPDpRqR4HV4/Y/TXKaRdRkdRRK0c+beX1kSOmWgiEdVnEyXgc87g M56+QdGY7tA8W40DM/xp56D81Ngc59N/JxOzVOq4EgdWDzzMY5xt9M6LhrVKJl5gEFSf ZkAT29uLY8pt52GCj2ba5Q32Kkcu2MM4gcyChBzks1ccoUaXo4dnvTh9CBCIySZyjFs5 kzzw==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@quicinc.com header.s=qcppdkim1 header.b=csqpwUFz; arc=pass (i=1 spf=pass spfdomain=quicinc.com dkim=pass dkdomain=quicinc.com dmarc=pass fromdomain=quicinc.com); spf=pass (google.com: domain of linux-kernel+bounces-74086-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-74086-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=quicinc.com Return-Path: Received: from ny.mirrors.kernel.org (ny.mirrors.kernel.org. [2604:1380:45d1:ec00::1]) by mx.google.com with ESMTPS id h8-20020ac85048000000b0042c4abaf67fsi9544271qtm.498.2024.02.20.21.37.16 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 20 Feb 2024 21:37:17 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-74086-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1; Authentication-Results: mx.google.com; dkim=pass header.i=@quicinc.com header.s=qcppdkim1 header.b=csqpwUFz; arc=pass (i=1 spf=pass spfdomain=quicinc.com dkim=pass dkdomain=quicinc.com dmarc=pass fromdomain=quicinc.com); spf=pass (google.com: domain of linux-kernel+bounces-74086-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-74086-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=quicinc.com 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 ny.mirrors.kernel.org (Postfix) with ESMTPS id B24BE1C208D3 for ; Wed, 21 Feb 2024 05:37:16 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 8A5F13A1D4; Wed, 21 Feb 2024 05:36:51 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=quicinc.com header.i=@quicinc.com header.b="csqpwUFz" 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 BD85E39ADD; Wed, 21 Feb 2024 05:36:48 +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=1708493810; cv=none; b=G3EfME8hhV2k5TRmA4vV6Az8YJS3twJkuXQNut6gXle43zaEmt2i9dvjWJxI762GFQq9xDqY/fgm1JwmzEPXguszxgJXMV8WupWRZOmLgpXO+877xGdBw6JhVH3jALjDpfHwYOlFurbmNuS2/TuNRJCCkMgM7AGv0vng6UryyWQ= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1708493810; c=relaxed/simple; bh=h/VjfafKWnZdt701kQi9hEaeMSuYAmya+J9erv+EEuM=; h=Message-ID:Date:MIME-Version:Subject:To:CC:References:From: In-Reply-To:Content-Type; b=IQ5U1Z3X/g7VApyNW81FVq+3M86IBWYaC1KUglVnbvUduzuAr9bh8w1+pjL50MXHsgI1w6JRsBX0MPUVUE2SSRcSplMkZbc8YkwJN0U2616I2c14YGXyd9Gvqv4iRuWhzCkgMDue1FzZzaJFAU2j/VCf5+anfHLPqUW5dRCaUTE= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=quicinc.com; spf=pass smtp.mailfrom=quicinc.com; dkim=pass (2048-bit key) header.d=quicinc.com header.i=@quicinc.com header.b=csqpwUFz; arc=none smtp.client-ip=205.220.168.131 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=quicinc.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=quicinc.com Received: from pps.filterd (m0279865.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.17.1.24/8.17.1.24) with ESMTP id 41L1jdSL030713; Wed, 21 Feb 2024 05:36:24 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=quicinc.com; h= message-id:date:mime-version:subject:to:cc:references:from :in-reply-to:content-type:content-transfer-encoding; s= qcppdkim1; bh=S5n/WpV8JcN0mx2fE3uU/d+OZrGuwRrR1d3N//9CiFA=; b=cs qpwUFzcl29oiOj+cr6STCjwPDDiknXqXp+BiwsKat9U3dEcTGj5KLCoLVseOafAy 1W1jatIWVkw+PUhTbQOimI693MkpC89cxcVvDZUWN3NUq4D0Zb9gkgO6Xer4m8EI ALJuHSVKkF78jjz7xUrvWtDk7RWS9150zMExJmc3AfsSKDIxWNc3qpUFYKl3RiYm RtnWvktckBz9gHcfsGNN0fEe9lGCTlrxeRSKJ5jlpbeO3WKa7rPKO0rBJB59uIpx ax7IycEeJcNWjVCDKPkuqRjrAEwpoytBSQ+xePQTKMl31kXHH+oBzkDj595/zVQF yXBXHBIjbJxMSF/xm4pQ== Received: from nalasppmta01.qualcomm.com (Global_NAT1.qualcomm.com [129.46.96.20]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 3wd21s924s-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Wed, 21 Feb 2024 05:36:23 +0000 (GMT) Received: from nalasex01b.na.qualcomm.com (nalasex01b.na.qualcomm.com [10.47.209.197]) by NALASPPMTA01.qualcomm.com (8.17.1.5/8.17.1.5) with ESMTPS id 41L5aMKl003639 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Wed, 21 Feb 2024 05:36:22 GMT Received: from [10.218.16.59] (10.80.80.8) by nalasex01b.na.qualcomm.com (10.47.209.197) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.1118.40; Tue, 20 Feb 2024 21:36:11 -0800 Message-ID: <0b9e807d-e0ca-411c-9a2b-3d804bdf168c@quicinc.com> Date: Wed, 21 Feb 2024 11:06:07 +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 v3 2/3] dt-bindings: iio: adc: Add support for QCOM PMIC5 Gen3 ADC To: Krzysztof Kozlowski , , , , , , , , , , CC: , , , , , , , , , , , , , , , , , , , , References: <20231231171237.3322376-1-quic_jprakash@quicinc.com> <20231231171237.3322376-3-quic_jprakash@quicinc.com> <3f812ffa-ec33-448e-b72a-ce698618a8c1@linaro.org> <13f2b558-a50d-44d3-85de-38e230212732@quicinc.com> Content-Language: en-US From: Jishnu Prakash In-Reply-To: Content-Type: text/plain; charset="UTF-8"; format=flowed Content-Transfer-Encoding: 7bit X-ClientProxiedBy: nasanex01b.na.qualcomm.com (10.46.141.250) To nalasex01b.na.qualcomm.com (10.47.209.197) X-QCInternal: smtphost X-Proofpoint-Virus-Version: vendor=nai engine=6200 definitions=5800 signatures=585085 X-Proofpoint-ORIG-GUID: fGcC1wH5IHx3LPFKis3LhLj_W6R4ZpCQ X-Proofpoint-GUID: fGcC1wH5IHx3LPFKis3LhLj_W6R4ZpCQ X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.272,Aquarius:18.0.1011,Hydra:6.0.619,FMLib:17.11.176.26 definitions=2024-02-20_06,2024-02-20_01,2023-05-22_02 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 phishscore=0 impostorscore=0 priorityscore=1501 adultscore=0 spamscore=0 lowpriorityscore=0 bulkscore=0 clxscore=1015 mlxscore=0 suspectscore=0 mlxlogscore=999 malwarescore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.19.0-2402120000 definitions=main-2402210039 Hi Krzysztof, On 2/17/2024 7:43 PM, Krzysztof Kozlowski wrote: > On 16/02/2024 11:39, Jishnu Prakash wrote: >> Hi Krzysztof, >> >> On 1/4/2024 1:48 PM, Krzysztof Kozlowski wrote: >>> On 31/12/2023 18:12, Jishnu Prakash wrote: >>>> For the PMIC5-Gen3 type PMICs, ADC peripheral is present in HW for the >>>> following PMICs: PMK8550, PM8550, PM8550B and PM8550VX PMICs. >>>> >>>> It is similar to PMIC5-Gen2, with SW communication to ADCs on all PMICs >>>> going through PBS(Programmable Boot Sequence) firmware through a single >>>> register interface. This interface is implemented on an SDAM (Shared >>>> Direct Access Memory) peripheral on the master PMIC PMK8550 rather >>>> than a dedicated ADC peripheral. >>>> >>>> Add documentation for PMIC5 Gen3 ADC and macro definitions for ADC >>>> channels and virtual channels (combination of ADC channel number and >>>> PMIC SID number) per PMIC, to be used by clients of this device. >>>> >>>> Changes since v2: >>>> - Moved ADC5 Gen3 documentation into a separate new file. >>> Changelog goes under ---. >>> >>> Why did you do this? What is the rationale? Sorry, this patchset goes >>> nowhere. >> >> >> I'll elaborate this more in the next patchset. There are two main >> reasons for adding this documentation in a new file: > > This was more than a month ago? You reply to my comment with 1.5 months > delay? > > Sorry, I am not in the context and I am not going back to it. I have > many other emails where my questions are addressed faster than 1.5 months. > > The patch is not even in my mailbox, long gone. > Why you are making it so difficult for reviewers? > > You will get answers like I am not in context, sorry. Next time don't > respond after 1.5 months. > You're right - I'll do my best to get back to review comments in a reasonable time frame. > >> >> 1.This device is not exactly like the existing QCOM VADC drivers as it >> now combines VADC functionality (reading ADC channel on client request) >> with ADC_TM functionality (thermal threshold monitoring). > > Does no explain touching bindings. Your drivers don't matter for bindings. > >> >> 2.Adding this device's bindings in the existing qcom,spmi-vadc.yaml file > > No rationale was provided in commit msg. > >> is not possible as it would require updating some of the existing >> top-level constraints. (for the older devices in that file, "reg" and >> "interrupts" can have at most one item, while this device can have more >> than one item under these properties.) > > How is this a problem? In qcom,spmi-vadc.yaml, we have the following top-level constraints for the "reg" and "interrupts" properties: reg: maxItems: 1 interrupts: maxItems: 1 For the ADC5 Gen3 device being added now, these constraints cannot be followed always, as there may be more than one peripheral under one device instance, each with a corresponding interrupt. For example, the above properties could be like this for a ADC5 Gen3 device: reg = <0x9000>, <0x9100>; interrupts = <0x0 0x90 0x1 IRQ_TYPE_EDGE_RISING>, <0x0 0x91 0x1 IRQ_TYPE_EDGE_RISING>; I could not overwrite the top-level constraints for the new device "qcom,spmi-adc5-gen3" alone in qcom,spmi-vadc.yaml, so I tried to remove the constraints from the top level and add them back conditionally for all the device types separately, but you told me not to remove them (full message: https://lore.kernel.org/linux-iio/832053f4-bd5d-4e58-81bb-1a8188e7f364@linaro.org/) Since these constraints cannot be modified for a specific new device or removed, I think the only way to accommodate this new device is to add it in its own new file. Is this a sufficient justification for adding this documentation in a new file or do you have any other suggestions? Thanks, Jishnu > > > Best regards, > Krzysztof >