mbox series

[v6,0/3] iio: chemical: atlas-sensor: add DO support

Message ID 20200210001058.7515-1-matt.ranostay@konsulko.com (mailing list archive)
Headers show
Series iio: chemical: atlas-sensor: add DO support | expand

Message

Matt Ranostay Feb. 10, 2020, 12:10 a.m. UTC
Changes from v1:
* combined dt binding docs
* added making interrupt line is optional patchset
* broke out dt changes from DO-SM module patchset

Changes from v2:
* switch docs to YAML format

Changes from v3:
* fix up issues reported with dt-schema

Changes from v4:
* fix logic issue in optional interrupt patchset

Changes from v5:
* Update license on yaml conversion to GPLv2.0+ and BSD clause

Matt Ranostay (3):
  iio: chemical: atlas-sensor: allow probe without interrupt line
  iio: chemical: atlas-sensor: add DO-SM module support
  dt-bindings: iio: chemical: consolidate atlas-sensor docs

 .../bindings/iio/chemical/atlas,ec-sm.txt     | 21 -----
 .../bindings/iio/chemical/atlas,orp-sm.txt    | 21 -----
 .../bindings/iio/chemical/atlas,ph-sm.txt     | 21 -----
 .../bindings/iio/chemical/atlas,sensor.yaml   | 53 +++++++++++
 drivers/iio/chemical/atlas-sensor.c           | 91 +++++++++++++++----
 5 files changed, 125 insertions(+), 82 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/iio/chemical/atlas,ec-sm.txt
 delete mode 100644 Documentation/devicetree/bindings/iio/chemical/atlas,orp-sm.txt
 delete mode 100644 Documentation/devicetree/bindings/iio/chemical/atlas,ph-sm.txt
 create mode 100644 Documentation/devicetree/bindings/iio/chemical/atlas,sensor.yaml

Comments

Jonathan Cameron April 13, 2020, 5:10 p.m. UTC | #1
Hi Matt,

Found this still in my queue and to be honest I'm lost and
can't easily figure out what the status is (I'm too lazy to
read all the past threads).

So what do you think the status of this patch set is?

Thanks,

Jonathan

On Sun,  9 Feb 2020 16:10:55 -0800
Matt Ranostay <matt.ranostay@konsulko.com> wrote:

> Changes from v1:
> * combined dt binding docs
> * added making interrupt line is optional patchset
> * broke out dt changes from DO-SM module patchset
> 
> Changes from v2:
> * switch docs to YAML format
> 
> Changes from v3:
> * fix up issues reported with dt-schema
> 
> Changes from v4:
> * fix logic issue in optional interrupt patchset
> 
> Changes from v5:
> * Update license on yaml conversion to GPLv2.0+ and BSD clause
> 
> Matt Ranostay (3):
>   iio: chemical: atlas-sensor: allow probe without interrupt line
>   iio: chemical: atlas-sensor: add DO-SM module support
>   dt-bindings: iio: chemical: consolidate atlas-sensor docs
> 
>  .../bindings/iio/chemical/atlas,ec-sm.txt     | 21 -----
>  .../bindings/iio/chemical/atlas,orp-sm.txt    | 21 -----
>  .../bindings/iio/chemical/atlas,ph-sm.txt     | 21 -----
>  .../bindings/iio/chemical/atlas,sensor.yaml   | 53 +++++++++++
>  drivers/iio/chemical/atlas-sensor.c           | 91 +++++++++++++++----
>  5 files changed, 125 insertions(+), 82 deletions(-)
>  delete mode 100644 Documentation/devicetree/bindings/iio/chemical/atlas,ec-sm.txt
>  delete mode 100644 Documentation/devicetree/bindings/iio/chemical/atlas,orp-sm.txt
>  delete mode 100644 Documentation/devicetree/bindings/iio/chemical/atlas,ph-sm.txt
>  create mode 100644 Documentation/devicetree/bindings/iio/chemical/atlas,sensor.yaml
>
Matt Ranostay April 28, 2020, 4:30 a.m. UTC | #2
On Mon, Apr 13, 2020 at 10:10 AM Jonathan Cameron <jic23@kernel.org> wrote:
>
>
>
> Hi Matt,
>
> Found this still in my queue and to be honest I'm lost and
> can't easily figure out what the status is (I'm too lazy to
> read all the past threads).
>
> So what do you think the status of this patch set is?

The code is merged but just noticed the documentation patch somehow got dropped.

Thanks,

Matt

>
> Thanks,
>
> Jonathan
>
> On Sun,  9 Feb 2020 16:10:55 -0800
> Matt Ranostay <matt.ranostay@konsulko.com> wrote:
>
> > Changes from v1:
> > * combined dt binding docs
> > * added making interrupt line is optional patchset
> > * broke out dt changes from DO-SM module patchset
> >
> > Changes from v2:
> > * switch docs to YAML format
> >
> > Changes from v3:
> > * fix up issues reported with dt-schema
> >
> > Changes from v4:
> > * fix logic issue in optional interrupt patchset
> >
> > Changes from v5:
> > * Update license on yaml conversion to GPLv2.0+ and BSD clause
> >
> > Matt Ranostay (3):
> >   iio: chemical: atlas-sensor: allow probe without interrupt line
> >   iio: chemical: atlas-sensor: add DO-SM module support
> >   dt-bindings: iio: chemical: consolidate atlas-sensor docs
> >
> >  .../bindings/iio/chemical/atlas,ec-sm.txt     | 21 -----
> >  .../bindings/iio/chemical/atlas,orp-sm.txt    | 21 -----
> >  .../bindings/iio/chemical/atlas,ph-sm.txt     | 21 -----
> >  .../bindings/iio/chemical/atlas,sensor.yaml   | 53 +++++++++++
> >  drivers/iio/chemical/atlas-sensor.c           | 91 +++++++++++++++----
> >  5 files changed, 125 insertions(+), 82 deletions(-)
> >  delete mode 100644 Documentation/devicetree/bindings/iio/chemical/atlas,ec-sm.txt
> >  delete mode 100644 Documentation/devicetree/bindings/iio/chemical/atlas,orp-sm.txt
> >  delete mode 100644 Documentation/devicetree/bindings/iio/chemical/atlas,ph-sm.txt
> >  create mode 100644 Documentation/devicetree/bindings/iio/chemical/atlas,sensor.yaml
> >
>
Jonathan Cameron May 2, 2020, 4:23 p.m. UTC | #3
On Mon, 27 Apr 2020 21:30:30 -0700
Matt Ranostay <matt.ranostay@konsulko.com> wrote:

> On Mon, Apr 13, 2020 at 10:10 AM Jonathan Cameron <jic23@kernel.org> wrote:
> >
> >
> >
> > Hi Matt,
> >
> > Found this still in my queue and to be honest I'm lost and
> > can't easily figure out what the status is (I'm too lazy to
> > read all the past threads).
> >
> > So what do you think the status of this patch set is?  
> 
> The code is merged but just noticed the documentation patch somehow got dropped.
Ah oops. I picked up v7 but missed dropping v6 from my queue.

For docs, it seems to be there..?

J
> 
> Thanks,
> 
> Matt
> 
> >
> > Thanks,
> >
> > Jonathan
> >
> > On Sun,  9 Feb 2020 16:10:55 -0800
> > Matt Ranostay <matt.ranostay@konsulko.com> wrote:
> >  
> > > Changes from v1:
> > > * combined dt binding docs
> > > * added making interrupt line is optional patchset
> > > * broke out dt changes from DO-SM module patchset
> > >
> > > Changes from v2:
> > > * switch docs to YAML format
> > >
> > > Changes from v3:
> > > * fix up issues reported with dt-schema
> > >
> > > Changes from v4:
> > > * fix logic issue in optional interrupt patchset
> > >
> > > Changes from v5:
> > > * Update license on yaml conversion to GPLv2.0+ and BSD clause
> > >
> > > Matt Ranostay (3):
> > >   iio: chemical: atlas-sensor: allow probe without interrupt line
> > >   iio: chemical: atlas-sensor: add DO-SM module support
> > >   dt-bindings: iio: chemical: consolidate atlas-sensor docs
> > >
> > >  .../bindings/iio/chemical/atlas,ec-sm.txt     | 21 -----
> > >  .../bindings/iio/chemical/atlas,orp-sm.txt    | 21 -----
> > >  .../bindings/iio/chemical/atlas,ph-sm.txt     | 21 -----
> > >  .../bindings/iio/chemical/atlas,sensor.yaml   | 53 +++++++++++
> > >  drivers/iio/chemical/atlas-sensor.c           | 91 +++++++++++++++----
> > >  5 files changed, 125 insertions(+), 82 deletions(-)
> > >  delete mode 100644 Documentation/devicetree/bindings/iio/chemical/atlas,ec-sm.txt
> > >  delete mode 100644 Documentation/devicetree/bindings/iio/chemical/atlas,orp-sm.txt
> > >  delete mode 100644 Documentation/devicetree/bindings/iio/chemical/atlas,ph-sm.txt
> > >  create mode 100644 Documentation/devicetree/bindings/iio/chemical/atlas,sensor.yaml
> > >  
> >
Matt Ranostay May 3, 2020, 11:50 p.m. UTC | #4
On Sat, May 2, 2020 at 9:23 AM Jonathan Cameron <jic23@kernel.org> wrote:
>
> On Mon, 27 Apr 2020 21:30:30 -0700
> Matt Ranostay <matt.ranostay@konsulko.com> wrote:
>
> > On Mon, Apr 13, 2020 at 10:10 AM Jonathan Cameron <jic23@kernel.org> wrote:
> > >
> > >
> > >
> > > Hi Matt,
> > >
> > > Found this still in my queue and to be honest I'm lost and
> > > can't easily figure out what the status is (I'm too lazy to
> > > read all the past threads).
> > >
> > > So what do you think the status of this patch set is?
> >
> > The code is merged but just noticed the documentation patch somehow got dropped.
> Ah oops. I picked up v7 but missed dropping v6 from my queue.
>
> For docs, it seems to be there..?

Ah yes it is.. forgot I put it part of "dt-bindings: iio: chemical:
consolidate atlas-sensor docs"

Thanks,

Matt

>
> J
> >
> > Thanks,
> >
> > Matt
> >
> > >
> > > Thanks,
> > >
> > > Jonathan
> > >
> > > On Sun,  9 Feb 2020 16:10:55 -0800
> > > Matt Ranostay <matt.ranostay@konsulko.com> wrote:
> > >
> > > > Changes from v1:
> > > > * combined dt binding docs
> > > > * added making interrupt line is optional patchset
> > > > * broke out dt changes from DO-SM module patchset
> > > >
> > > > Changes from v2:
> > > > * switch docs to YAML format
> > > >
> > > > Changes from v3:
> > > > * fix up issues reported with dt-schema
> > > >
> > > > Changes from v4:
> > > > * fix logic issue in optional interrupt patchset
> > > >
> > > > Changes from v5:
> > > > * Update license on yaml conversion to GPLv2.0+ and BSD clause
> > > >
> > > > Matt Ranostay (3):
> > > >   iio: chemical: atlas-sensor: allow probe without interrupt line
> > > >   iio: chemical: atlas-sensor: add DO-SM module support
> > > >   dt-bindings: iio: chemical: consolidate atlas-sensor docs
> > > >
> > > >  .../bindings/iio/chemical/atlas,ec-sm.txt     | 21 -----
> > > >  .../bindings/iio/chemical/atlas,orp-sm.txt    | 21 -----
> > > >  .../bindings/iio/chemical/atlas,ph-sm.txt     | 21 -----
> > > >  .../bindings/iio/chemical/atlas,sensor.yaml   | 53 +++++++++++
> > > >  drivers/iio/chemical/atlas-sensor.c           | 91 +++++++++++++++----
> > > >  5 files changed, 125 insertions(+), 82 deletions(-)
> > > >  delete mode 100644 Documentation/devicetree/bindings/iio/chemical/atlas,ec-sm.txt
> > > >  delete mode 100644 Documentation/devicetree/bindings/iio/chemical/atlas,orp-sm.txt
> > > >  delete mode 100644 Documentation/devicetree/bindings/iio/chemical/atlas,ph-sm.txt
> > > >  create mode 100644 Documentation/devicetree/bindings/iio/chemical/atlas,sensor.yaml
> > > >
> > >
>