public inbox for devicetree@vger.kernel.org
 help / color / mirror / Atom feed
From: Conor Dooley <conor@kernel.org>
To: Subhajit Ghosh <subhajit.ghosh@tweaklogic.com>
Cc: 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>,
	linux-iio@vger.kernel.org, devicetree@vger.kernel.org,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH] dt-bindings: iio: light: Squash APDS9300 and APDS9960 schemas
Date: Thu, 19 Oct 2023 12:28:20 +0100	[thread overview]
Message-ID: <20231019-hurry-eagle-0ffa95b1a026@spud> (raw)
In-Reply-To: <6b856b0d-4f69-70ac-59a7-237fd21d1a92@tweaklogic.com>

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

On Thu, Oct 19, 2023 at 09:20:12PM +1030, Subhajit Ghosh wrote:
> On 19/10/23 19:21, Conor Dooley wrote:
> > On Thu, Oct 19, 2023 at 06:34:37PM +1030, Subhajit Ghosh wrote:
> > > Squashing Avago (Broadcom) APDS9300 and APDS9960 schemas into one
> > > file and removing the other.
> > 
> > > Link: https://lore.kernel.org/all/4e785d2e-d310-4592-a75a-13549938dcef@linaro.org/
> > 
> > "Yes, they look similar. I will combine them all in a single yaml file in
> > the next revision. Thank you Krzysztof."
> > 
> > Yet this is a follow-up patch, not a version 2. The original patches
> > seem to not have been applied, so I am not sure why you didn't send a
> > v2?
> > 
> > Cheers,
> > Conor.
> 
> Sorry, I should have put a longer description and a longer commit message.
> That patch series adds a new driver - apds9306 which is separate to this
> patch. As per Krzysztof's comments, first operation is to merge the existing
> apds9300 and apds9960 schemas. This patch is the first operation.
> 
> Second operation will be to add apds9306 support on top of that. I will
> explain more on Krzysztof's comments. Thank you for reviewing.

Ahh apologies then. The best course of action would likely be to include
the patch merging the two bindings in your series adding the third user.

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

  reply	other threads:[~2023-10-19 11:28 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-10-19  8:04 [PATCH] dt-bindings: iio: light: Squash APDS9300 and APDS9960 schemas Subhajit Ghosh
2023-10-19  8:51 ` Conor Dooley
2023-10-19  8:55   ` Conor Dooley
2023-10-19 10:54     ` Subhajit Ghosh
2023-10-19 11:21       ` Jonathan Cameron
2023-10-20  7:28         ` Jonathan Cameron
2023-10-20 14:25           ` Conor Dooley
2023-10-21  0:29           ` Subhajit Ghosh
2023-10-19 10:50   ` Subhajit Ghosh
2023-10-19 11:28     ` Conor Dooley [this message]
2023-10-19 12:10       ` Subhajit Ghosh
2023-10-19  9:20 ` Krzysztof Kozlowski
2023-10-19 11:16   ` Subhajit Ghosh
2023-10-19 13:30     ` Krzysztof Kozlowski
2023-10-21  0:38       ` Subhajit Ghosh

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20231019-hurry-eagle-0ffa95b1a026@spud \
    --to=conor@kernel.org \
    --cc=conor+dt@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=jic23@kernel.org \
    --cc=krzysztof.kozlowski+dt@linaro.org \
    --cc=lars@metafoo.de \
    --cc=linux-iio@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=robh+dt@kernel.org \
    --cc=subhajit.ghosh@tweaklogic.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox