Received: by 2002:a05:7208:9594:b0:7e:5202:c8b4 with SMTP id gs20csp28863rbb; Fri, 23 Feb 2024 10:35:12 -0800 (PST) X-Forwarded-Encrypted: i=3; AJvYcCWaxVJTUedXVni3DPYgycj2X8LIjlYftySjJZv8VGNMULKaT+K4ZnDPrEFycx/Vfb+xMV2hNBLpsYU7VylHnmvYl+nsqvgEuR+CycCofg== X-Google-Smtp-Source: AGHT+IEVfVtRiBPPwFM/jLL5l/22FWhpgMQEIreZEb9PkDH5oysHDYdG/IpMTisTlAlKgwKjITsK X-Received: by 2002:a05:620a:108e:b0:787:a986:f66c with SMTP id g14-20020a05620a108e00b00787a986f66cmr712146qkk.3.1708713311975; Fri, 23 Feb 2024 10:35:11 -0800 (PST) ARC-Seal: i=2; a=rsa-sha256; t=1708713311; cv=pass; d=google.com; s=arc-20160816; b=PDEt+UINbTNGy+J79NuBRKXqa33EPsoFOfNsF94VOXDVaHYvtaZE25bOwpHj9QAcwm VTuoUmFS/D5Mt2iKh6oCOhwApI+6Mn/Mn/T+4AXCSY6MEGvFO3Bx/WWRI+vyEkBkKyEQ ZltTe/WboROHj5dl2h72KALwXJlgikpL9jXtTexMd/YXaz9gVnF2kBJuz6CFY4CbByeu Zqc4tNOV0WmyxFeAdCZj4sNtDaDvGhjmIB6bJhm2B5hKMskOnT5FFjIsgABsGailioL2 ylAwjdg8RniuYjQPi3bI0sRGL5UJpLkdUp5KFYcoL33AhuhbPjUz6F7+SGI10GL8JI7z K+jw== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=in-reply-to:content-disposition:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:references:message-id:subject:cc :to:from:date:dkim-signature; bh=kXGbBe1/xJv+m8+R6qNko85GxYhi9oh20jnc1oOkCyY=; fh=VYZxfVGBVBSY5sEICPGR5Me6RCiMsyTU926U6eN+7WE=; b=mD4/0x9vCF6ESNCHY7MDmLasFuOAg9Zll5nXvVkWn8TLS7fIDCQtgES+RRDstn7fTG 2VlujMXtgFaxOGc4kyCBaIojJh4T2q67DXXxC2YBn2MchIiGo6lkgqbRuja416wjo8jH ClBSVlGEuJB2IvNFjqWEo1Oiaxfq2k1S4nCljSivO8olR+B/y2iTdzPf3uEHy0sUFrHD uMzkLbtE/uDVkHj+P5532xb4uAuzMJbbcvglKHODrEBhrEkLMDhQ/lRGSQbCLJNLNlO6 JK/lP6GNnYubxK/bef+LQd6Z3vroWhqXxQQ0n0N9U7tlsBqecV/zCdscUdfUHRNr8PX/ zqEw==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=rGZ0A8It; arc=pass (i=1 dkim=pass dkdomain=kernel.org); spf=pass (google.com: domain of linux-kernel+bounces-78986-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-78986-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from ny.mirrors.kernel.org (ny.mirrors.kernel.org. [2604:1380:45d1:ec00::1]) by mx.google.com with ESMTPS id pi21-20020a05620a379500b0078726baaca4si15781520qkn.617.2024.02.23.10.35.11 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 23 Feb 2024 10:35:11 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-78986-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=@kernel.org header.s=k20201202 header.b=rGZ0A8It; arc=pass (i=1 dkim=pass dkdomain=kernel.org); spf=pass (google.com: domain of linux-kernel+bounces-78986-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-78986-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 ny.mirrors.kernel.org (Postfix) with ESMTPS id B2EAE1C20987 for ; Fri, 23 Feb 2024 18:35:11 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id DB51F82D90; Fri, 23 Feb 2024 18:34:44 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="rGZ0A8It" 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 074331118D; Fri, 23 Feb 2024 18:34:43 +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=1708713284; cv=none; b=MORTJUYrk3UTpE7Tqu45rf2CdsMrph2XntmWAL2uB/sD95VxkruqC8kmLzh4KsaTB2vMDJRXpmsI3+PadP/3HrecaRjrVKnjS1uClNEKua2JlSvFo6c5EJQD7k3+WuIONC+jcwQzi7ePGbYfySFa2eT7UZT6JjArRqA/j/BacZM= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1708713284; c=relaxed/simple; bh=zlETi33+CFuv5hwQm5WZx5u7wTiSlx0umoUQhthd1h0=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=PTBSu0vAiO41YQF25Vpu4AWKAYwxs9Fv2Lyf6SpfBatypZPO7hXv5Ya8YYp/UtqBv3BX0pkXICt8VCnJbZsiQgS610xmm6G1A2m7x+Kj5p1NlIN1uWNhJWZogeemzaAPc+IO+cFpjMl5Xy7E9UMN+4T3GCuTOzgmXeWqfEyabTs= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=rGZ0A8It; arc=none smtp.client-ip=10.30.226.201 Received: by smtp.kernel.org (Postfix) with ESMTPSA id 7C657C433C7; Fri, 23 Feb 2024 18:34:41 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1708713283; bh=zlETi33+CFuv5hwQm5WZx5u7wTiSlx0umoUQhthd1h0=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=rGZ0A8ItVnGR6BJDHdNbPYcsFSCvlZxjxY1naGW64VV2gZJCv+ym2Ftd9J035b/77 qHrWNpD3fJeO37pLMUDtp/yf1yu2aeMFXNApNM+1qcHHHU5HbUBen1A/WA5p6qrLiH qc2a8GOmToCUIFgejcGnJ8HKhPLeEzfVoIep7Rn93nleAVl36gUjXr2NaNxdX1F73o Tcu165EQGhcd9L4cBidbegOZc1BGsZEF0YY3cyjr6Lwaz4wcdjNLCM0X47VpJXLQon 7uLYMJj8UZnCmpAy1mzV0BN2YJnextedsuIf3oWUtTIvwE9egOFLb4HSWEWLmL/Uif K1hE1pzk8jKig== Date: Fri, 23 Feb 2024 18:34:39 +0000 From: Conor Dooley To: Javier Carrasco Cc: Jonathan Cameron , Lars-Peter Clausen , Rob Herring , Krzysztof Kozlowski , Conor Dooley , linux-iio@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] dt-bindings: iio: light: vishay,veml6075: make vdd-supply required Message-ID: <20240223-niece-shakily-7e18b90f30e7@spud> References: <20240223-veml6075_vdd-v1-1-ac76509b1998@gmail.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="Hgz/JpkIdG8P36dD" Content-Disposition: inline In-Reply-To: <20240223-veml6075_vdd-v1-1-ac76509b1998@gmail.com> --Hgz/JpkIdG8P36dD Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Fri, Feb 23, 2024 at 02:01:33PM +0100, Javier Carrasco wrote: > The VEML6075 requires a single supply to operate. The property already > exists in the bindings and it is used in the example, but it is still > not on the list of required properties. >=20 > Signed-off-by: Javier Carrasco > --- > The driver and bindings for the VEML6075 UV sensor were merged in the > merge window for v6.8 i.e. they have not been released yet beyond rc. >=20 > Applying this patch as a fix now would avoid an ABI break like the > recently discussed here: >=20 > https://lore.kernel.org/linux-iio/8409a5bc71b995e3b738b817a074cfb131c3b2b= 5.camel@gmail.com/ >=20 > On the other hand, from the same discussion it can be concluded that the > major risk would be a potential warning with dtbs_check, in case this > patch is applied during the next merge window. I don't care if it introduces a dtbs_check problem, we can fix those. The device clearly needs power and at least for linux it being missing gets handled perfectly fine. I think it's pretty reasonable to do this. Acked-by: Conor Dooley Cheers, Conor. >=20 > The VEML6075 driver already handles the power supply with > devm_regulator_get_enable(). > --- > Documentation/devicetree/bindings/iio/light/vishay,veml6075.yaml | 1 + > 1 file changed, 1 insertion(+) >=20 > diff --git a/Documentation/devicetree/bindings/iio/light/vishay,veml6075.= yaml b/Documentation/devicetree/bindings/iio/light/vishay,veml6075.yaml > index abee04cd126e..91c318746bf3 100644 > --- a/Documentation/devicetree/bindings/iio/light/vishay,veml6075.yaml > +++ b/Documentation/devicetree/bindings/iio/light/vishay,veml6075.yaml > @@ -21,6 +21,7 @@ properties: > required: > - compatible > - reg > + - vdd-supply > =20 > additionalProperties: false > =20 >=20 > --- > base-commit: 6613476e225e090cc9aad49be7fa504e290dd33d > change-id: 20240223-veml6075_vdd-e2220158ffda >=20 > Best regards, > --=20 > Javier Carrasco >=20 --Hgz/JpkIdG8P36dD Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCZdjlPwAKCRB4tDGHoIJi 0tYTAQC1+0IEYywpMoCMXpgH+lW/sykjfmPF82FwRnlmLuEcYgEAwShTG/cWRfqw evhM0yVbHRI14c70ha4vcyKGg8SStQM= =S2Pw -----END PGP SIGNATURE----- --Hgz/JpkIdG8P36dD--