From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-43172.protonmail.ch (mail-43172.protonmail.ch [185.70.43.172]) (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 BB2B933F8C1 for ; Wed, 12 Aug 2026 12:41:28 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=185.70.43.172 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786538497; cv=none; b=r7n4Iy0uVDtGnp2ssyWvcENP32Tn0/NlHPiQfzzkXF71sVPEmxommRJtIlwxrxFGmQh2eCss7g3n4jFmA9TELRV1g+KhScjkdADwb3IM+NLDZLGXNoMHMlJ6vJuEhDG5K0RJYPARYxrYjBqUKQmoPtFf6fV0+fOW8GbCoMtkt/s= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786538497; c=relaxed/simple; bh=S2iivv0OCUFj7r6dflGwy5lbMAWvtCa62u36edHKWdM=; h=From:To:Cc:Subject:In-Reply-To:References:Date:Message-ID: MIME-Version:Content-Type; b=kFkOzGy7yNvu0Vlo1VKKXKppjTJGErtrAkbJFPVortwtfk+YjqYkJF+i5wVFL2v5u/cDwzuXju8bZm2dQ1nyJM7VrltZpjpmjVXpPbSD5EodvorlOodYa491mFkjpNL8MDrNpvp/5G1U1MTCUYMmb4rh3KR4OzzVmjzZ2x8459A= 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=wphoB4YS; arc=none smtp.client-ip=185.70.43.172 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="wphoB4YS" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=geanix.com; s=protonmail; t=1786538478; x=1786797678; bh=YxWA8tiQCMavv9N3+KTJRnlp3LNFrP04pwpm0fRKjHg=; 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=wphoB4YShsx8O72HP51VDksd28JKGDzkXTJAEW0rZ9DlFVMdrUQKbGkQ05CXUdrNT QMPcvSM1L7kpqKQ4tgu8UWvLKQ1wjhJ/Kl1ZtU+si/Yrdh0IC2wU1W3Lin+4FZCDAj d7rvtymCp0xDl6d3IMNI+nAWwXK+9pw2CLhEoRwHDc7tTnkhDIvE1m/X5dFJzEphyE qkAsrfDh8Eax95YfFlBxLwgebrguUj3T7cuJkXh2qIG8+UcXeewm7obNCCKpMvIwLr vTorsNSyO/zO214WTiA6pdcrBnTiKa6xKiqL/SHkR7hOUrWmPXTVvLcTLVkMkMjlw9 /9rgCOoi/fu5w== X-Pm-Submission-Id: 4hKp5761nrz1DFFG 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: <20260810155006.GA1815611-robh@kernel.org> (Rob Herring's message of "Mon, 10 Aug 2026 10:50:06 -0500") References: <20260810-liteon-ltr329-v4-0-8bd3dfd15280@geanix.com> <20260810-liteon-ltr329-v4-3-8bd3dfd15280@geanix.com> <20260810155006.GA1815611-robh@kernel.org> Date: Wed, 12 Aug 2026 14:41:14 +0200 Message-ID: <871pc3y0id.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 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. I will. I guess it is best to add the new conditional statement under allOf, but as we already have an allOf block at the top, I would have to add it under that, which would place this new conditional statement related to specific compatible strings before the list of those compatible strings. It should work fine, but it looks a bit odd to me. I could also add it directly as an if statement, but that would only work as long as it is the only top level if statement, so that looks like a bad idea. Should I just stick it to the existing allOf list? /Esben