From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-4321.protonmail.ch (mail-4321.protonmail.ch [185.70.43.21]) (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 C5B7D3DBD4E for ; Mon, 17 Aug 2026 12:13:15 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=185.70.43.21 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786968803; cv=none; b=l8oT6tNgmpLnTnPc3+/0p7dUYyF/u08aSg1GRTb3G+y8qlcGKUkIW4RB6LNtzIn5VSVY6A19Pejne1PLqSMiK9XfwfR0amAuGcKBR5RrK9QG32qxErwdjAYNL9Okb7VXTFrsXZzhGy4XRlKXECdbizdyRr2e77FikFL0Q1DNYjs= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786968803; c=relaxed/simple; bh=AcNiNXsubNzSKo85gtLhu9qj/AfKl1uJKor+uNkkrYY=; h=From:To:Cc:Subject:In-Reply-To:References:Date:Message-ID: MIME-Version:Content-Type; b=rnk03RUzyw1KuQrWkq8GleJh1PvaccuU9T5VY/Y0tlv9y6L2qRKsF+p4bR5KsgiUd0Ce3WBgelNBwlQl9mbk7OtCt/t27msz1yMEfLXhC66gRpQbuXj1x9wqBAxstCDcwa6Iww86YdCDpnZpAZ+Bak1jLizTYg8lNY0WJGjq6nI= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=geanix.com; spf=pass smtp.mailfrom=geanix.com; dkim=pass (2048-bit key) header.d=geanix.com header.i=@geanix.com header.b=zriADBfv; arc=none smtp.client-ip=185.70.43.21 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=geanix.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=geanix.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=geanix.com header.i=@geanix.com header.b="zriADBfv" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=geanix.com; s=protonmail; t=1786967801; x=1787227001; bh=sez7qMxcIbxM4COhVhdmF4Rm74MzzENRelO/rZukLD0=; h=From:To:Cc:Subject:In-Reply-To:References:Date:Message-ID:From:To: Cc:Date:Subject:Reply-To:Feedback-ID:Message-ID:BIMI-Selector; b=zriADBfvzxarerE0m4Oialv9s6n6ZdRWf632sbzNeZAnjuNrankF6WLPGeOmEQyr7 Mbfa7TWZuOMwakb49kJb2P9sNSjRWwHouCv28iLtwEXXdsCAw9I4JdVqHNOmT8sKlI anOKuvH9AnVdJF1e9z6OXQjydMRGeohvyFAne2rguKecdl9kCuw3mbIItZjJThw+qW 2G3YGi5zej7xMQ57XjpQRj3Cke1yfNUsMdtcteOUCqSzSY9mOHFjnd3xPclYhSIwCg dbMDU2I8W+XiH/kJVwldwpikh8Zqi904a+NdjFKKX6qJ/XiGjNPVFJq0++/I1j2hb9 qsg9/GoR9qHNA== X-Pm-Submission-Id: 4hNrsK4yqsz2Scpc From: Esben Haabendal To: "Rob Herring" Cc: "Jonathan Cameron" , "David Lechner" , Nuno =?utf-8?Q?S=C3=A1?= , "Andy Shevchenko" , "Krzysztof Kozlowski" , "Conor Dooley" , "Nikita Travkin" , "Maslov Dmitry" , "Kuppuswamy Sathyanarayanan" , "Thomas Gleixner" , "Pan Chuang" , "Yangtao Li" , , , , "Krzysztof Kozlowski" Subject: Re: [PATCH v4 3/7] dt-bindings: iio: light: ltr501: Add ltr329 compatible In-Reply-To: <20260814175624.GE1981719-robh@kernel.org> (Rob Herring's message of "Fri, 14 Aug 2026 12:56:24 -0500") References: <20260810-liteon-ltr329-v4-0-8bd3dfd15280@geanix.com> <20260810-liteon-ltr329-v4-3-8bd3dfd15280@geanix.com> <20260810155006.GA1815611-robh@kernel.org> <20260814175624.GE1981719-robh@kernel.org> Date: Mon, 17 Aug 2026 13:56:36 +0200 Message-ID: <87bjb1kljf.fsf@geanix.com> User-Agent: Gnus/5.13 (Gnus v5.13) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain "Rob Herring" writes: > On Fri, Aug 14, 2026 at 06:18:55AM +0000, Esben Haabendal wrote: >> On Monday, August 10th, 2026 at 17:50, Rob Herring wrote: >> >> > On Mon, Aug 10, 2026 at 09:11:13AM +0200, Esben Haabendal wrote: >> > > LTR-329ALS-01 is an ambient light sensor similar to LTR-303ALS-01, but >> > > without interrupt support. >> > > >> > > Acked-by: Krzysztof Kozlowski >> > > Acked-by: Kuppuswamy Sathyanarayanan >> > > Signed-off-by: Esben Haabendal >> > > --- >> > > Documentation/devicetree/bindings/iio/light/liteon,ltr501.yaml | 1 + >> > > 1 file changed, 1 insertion(+) >> > >> > sashiko issue should be addressed. >> >> What about the suggestion about adding `proximity-near-level: false` to the condition. I am thinking something like this is more correct >> >> properties: >> proximity-near-level: false >> >> allOf: >> - if: >> properties: >> compatible: >> contains: >> enum: >> - liteon,ltr501 >> - liteon,ltr559 >> then: >> properties: >> proximity-near-level: true > > That does not work. The if/then can only add restrictions, not relax > them. It can do allowed->not-allowed, but not vice-versa as you have. Ok, I will do it the other way around then. /Esben