Re: [PATCH v6 4/5] dt-bindings: iio: light: Avago APDS9306

From: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
To: Subhajit Ghosh <subhajit.ghosh@tweaklogic.com>,
	Jonathan Cameron <jic23@kernel.org>,
	Lars-Peter Clausen <lars@metafoo.de>,
	Rob Herring <robh+dt@kernel.org>,
	Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
	Conor Dooley <conor+dt@kernel.org>,
	Matti Vaittinen <mazziesaccount@gmail.com>,
	Andy Shevchenko <andriy.shevchenko@linux.intel.com>,
	Marek Vasut <marex@denx.de>, Anshul Dalal <anshulusr@gmail.com>,
	Javier Carrasco <javier.carrasco.cruz@gmail.com>
Cc: Matt Ranostay <matt@ranostay.sg>,
	Stefan Windfeldt-Prytz <stefan.windfeldt-prytz@axis.com>,
	linux-iio@vger.kernel.org, devicetree@vger.kernel.org,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH v6 4/5] dt-bindings: iio: light: Avago APDS9306
Date: Fri, 9 Feb 2024 08:34:11 +0100	[thread overview]
Message-ID: <5089c549-505f-4342-b3fe-bed8a29b6ce1@linaro.org> (raw)
In-Reply-To: <f7c18fca-2a85-46b2-a671-2409e662520f@tweaklogic.com>

On 08/02/2024 11:51, Subhajit Ghosh wrote:
> Hi Krzysztof,
> 
> On 8/2/24 18:48, Krzysztof Kozlowski wrote:
>> On 06/02/2024 14:00, Subhajit Ghosh wrote:
>>> Driver support for Avago (Broadcom) APDS9306 Ambient Light Sensor.
>>> Extend avago,apds9300.yaml schema file to support apds9306 device.
>>>
>>> this patch depends on patch:
>>> "dt-bindings: iio: light: adps9300: Update interrupt definitions"
>>
>> Drop the paragraph, not helping. There is no dependency here.
> In the submitting patches guide, I read that if one patch depends
> on another, it should be mentioned.

On another patchset! Not on other patch within patchset. Anyway,
submitting patches don't tell to store it in Git forever because it does
not make sense.

Third, there is no dependency.

> If I try to apply this patch with "git am", it fails without
> first applying the patch dependency mentioned above. Is that fine?

It's just context.

Best regards,
Krzysztof