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

From: Conor Dooley <conor@kernel.org>
To: Subhajit Ghosh <subhajit.ghosh@tweaklogic.com>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
	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>,
	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: Thu, 8 Feb 2024 18:52:23 +0000	[thread overview]
Message-ID: <20240208-chaplain-empathy-3955eabb7152@spud> (raw)
In-Reply-To: <f7c18fca-2a85-46b2-a671-2409e662520f@tweaklogic.com>

[-- Attachment #1: Type: text/plain, Size: 1046 bytes --]

On Thu, Feb 08, 2024 at 09:21:17PM +1030, 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.
> If I try to apply this patch with "git am", it fails without
> first applying the patch dependency mentioned above. Is that fine?
> Again, sorry about the silly questions, just don't want to assume stuff!

It's another patch in the same series, I don't think you should bother
mentioning this at all. If there's a dependency on another series, then
you should mention it under the --- line.

Cheers,
Conor.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]