All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jonathan Cameron <Jonathan.Cameron@Huawei.com>
To: Philipp Jungkamp <p.jungkamp@gmx.net>
Cc: srinivas pandruvada <srinivas.pandruvada@linux.intel.com>,
	Jiri Kosina <jikos@kernel.org>,
	Jonathan Cameron <jic23@kernel.org>, <linux-iio@vger.kernel.org>
Subject: Re: [PATCH v4 0/4] IIO: More HID custom sensors
Date: Wed, 14 Dec 2022 09:27:07 +0000	[thread overview]
Message-ID: <20221214092707.0000046e@Huawei.com> (raw)
In-Reply-To: <cc35bdc25daedb32dbb5949f99559485a7f83080.camel@gmx.net>

On Tue, 13 Dec 2022 23:51:50 +0100
Philipp Jungkamp <p.jungkamp@gmx.net> wrote:

> Hello,
> 
> Is there something I still need to do for this to be merged in the next
> merge window?
> 
> I don't quite understand what would happen next. I'm curious and hope I
> don't bother in the busy times when approaching a merge window.
Given most of this is in HID, up to Jiri to pick it up when happy with it.

Has all the other responses I think it needs.  You are absolutely correct
to ping it given it's been a few weeks.

Jonathan
> 
> Regards,
> Philipp Jungkamp
> 
> On Sun, 2022-11-27 at 09:33 -0800, srinivas pandruvada wrote:
> > On Fri, 2022-11-25 at 00:38 +0100, Philipp Jungkamp wrote:  
> > > Hello,
> > > 
> > > I'm sorry for the confusion caused by me sending revisions as
> > > replies.
> > > I've now addresses the points raised. As I'm unfamiliar with
> > > mailing
> > > lists
> > > any recommendation on making the process more seamless is welcome.
> > > 
> > > Thank you for your time!
> > >   
> > 
> > For the series
> > 
> > Acked-by: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
> > 
> >   
> > > v2:
> > > - Patch 1/4:
> > >   - Remove unnecessary comments.
> > >   - Change return type of functions to bool. (introduced an error,
> > > fixed in v4)
> > >   - Swallows return value of a function. (fixed in v4)
> > > - Patch 2/4:
> > >   - Create this 'noop' patch that just adds the LISS sensors to the
> > > custom
> > >     sensor match table.
> > > - Patch 3/4:
> > >   - Remove some unnecessary newlines in function calls.
> > > - Patch 4/4:
> > >   - Allow for 1 byte human presence reports.
> > > 
> > > v3:
> > > - Patch 2/4
> > >   - Add missing 'Signed-Off-By'
> > > 
> > > v4:
> > > - Patch 1/4:
> > >   - Fix return value and error codes on property query.
> > >   - Document and ignore return values for queries of optionally
> > > matched
> > >     properties.
> > >   - Don't modify *known in _get_known, return error when not
> > > matched.
> > >   - Remove comment on newly added sensors from commit message.
> > > - Patch 4/4:
> > >   - Clean up switch statement.
> > > 
> > > Regards,
> > > Philipp Jungkamp
> > > 
> > > Philipp Jungkamp (4):
> > >   HID: hid-sensor-custom: Allow more custom iio sensors
> > >   HID: hid-sensor-custom: Add LISS custom sensors
> > >   IIO: hid-sensor-als: Use generic usage
> > >   IIO: hid-sensor-prox: Use generic usage
> > > 
> > >  drivers/hid/hid-sensor-custom.c     | 233 +++++++++++++++++++-----
> > > ----
> > >  drivers/iio/light/hid-sensor-als.c  |  27 ++--
> > >  drivers/iio/light/hid-sensor-prox.c |  37 +++--
> > >  include/linux/hid-sensor-ids.h      |   1 +
> > >  4 files changed, 199 insertions(+), 99 deletions(-)
> > > 
> > > --
> > > 2.38.1
> > >   
> > 
> >   
> 


  reply	other threads:[~2022-12-14  9:27 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-11-24 23:38 [PATCH v4 0/4] IIO: More HID custom sensors Philipp Jungkamp
2022-11-24 23:38 ` [PATCH v4 1/4] HID: hid-sensor-custom: Allow more custom iio sensors Philipp Jungkamp
2022-11-26 19:11   ` Jonathan Cameron
2022-11-24 23:38 ` [PATCH v4 2/4] HID: hid-sensor-custom: Add LISS custom sensors Philipp Jungkamp
2022-11-26 19:11   ` Jonathan Cameron
2022-11-24 23:38 ` [PATCH v4 3/4] IIO: hid-sensor-als: Use generic usage Philipp Jungkamp
2022-11-26 19:05   ` Jonathan Cameron
2022-11-24 23:38 ` [PATCH v4 4/4] IIO: hid-sensor-prox: " Philipp Jungkamp
2022-11-26 19:04   ` Jonathan Cameron
2022-11-27 17:33 ` [PATCH v4 0/4] IIO: More HID custom sensors srinivas pandruvada
2022-12-13 22:51   ` Philipp Jungkamp
2022-12-14  9:27     ` Jonathan Cameron [this message]
2022-12-14 17:20     ` Jiri Kosina
2022-12-15 16:21       ` Jonathan Cameron
2022-12-20 14:26         ` Jiri Kosina

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=20221214092707.0000046e@Huawei.com \
    --to=jonathan.cameron@huawei.com \
    --cc=jic23@kernel.org \
    --cc=jikos@kernel.org \
    --cc=linux-iio@vger.kernel.org \
    --cc=p.jungkamp@gmx.net \
    --cc=srinivas.pandruvada@linux.intel.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.