All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jonathan Cameron <jic23@kernel.org>
To: Rob Herring <robh@kernel.org>
Cc: Stephan Gerhold <stephan@gerhold.net>,
	linux-iio@vger.kernel.org,
	Peter Meerwald-Stadler <pmeerw@pmeerw.net>,
	Lars-Peter Clausen <lars@metafoo.de>,
	Linus Walleij <linus.walleij@linaro.org>,
	devicetree@vger.kernel.org, Rob Herring <robh+dt@kernel.org>
Subject: Re: [PATCH] dt-bindings: iio: accel: bma255: Fix bmc150/bmi055 compatible
Date: Thu, 14 Jan 2021 20:59:37 +0000	[thread overview]
Message-ID: <20210114205937.11e650d0@archlinux> (raw)
In-Reply-To: <20201213132514.00d7ffff@archlinux>

On Sun, 13 Dec 2020 13:25:14 +0000
Jonathan Cameron <jic23@kernel.org> wrote:

> On Wed, 9 Dec 2020 12:17:39 -0600
> Rob Herring <robh@kernel.org> wrote:
> 
> > On Wed, 02 Dec 2020 09:35:51 +0100, Stephan Gerhold wrote:  
> > > The bmc150-accel-i2c.c driver has an "_accel" suffix for the
> > > compatibles of BMC150 and BMI055. This is necessary because BMC150
> > > contains both accelerometer (bosch,bmc150_accel) and magnetometer
> > > (bosch,bmc150_magn) and therefore "bosch,bmc150" would be ambiguous.
> > > 
> > > However, the binding documentation suggests using "bosch,bmc150".
> > > Add the "_accel" suffix for BMC150 and BMI055 so the binding docs
> > > match what is expected by the driver.
> > > 
> > > Cc: Linus Walleij <linus.walleij@linaro.org>
> > > Fixes: 496a39526fce8 ("iio: accel: bmc150-accel: Add DT bindings")
> > > Signed-off-by: Stephan Gerhold <stephan@gerhold.net>
> > > ---
> > >  Documentation/devicetree/bindings/iio/accel/bosch,bma255.yaml | 4 ++--
> > >  1 file changed, 2 insertions(+), 2 deletions(-)
> > >     
> > 
> > Reviewed-by: Rob Herring <robh@kernel.org>  
> Applied to the fixes-togreg branch of iio.git which is now based on stuff
> queued up for the merge window. I'll send a pull not long after rc1.
> 

Not sure why, but the Fixes tag above is invalid. 

As a result I need to rebase this branch anyway so I've also brought it forward
to staging/staging-linus as of today.

Hopefully I'll get pull request out tomorrow.  Sorry for the delay!

Jonathan


> Thanks,
> 
> Jonathan
> 
> 


  reply	other threads:[~2021-01-14 21:00 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-12-02  8:35 [PATCH] dt-bindings: iio: accel: bma255: Fix bmc150/bmi055 compatible Stephan Gerhold
2020-12-02 12:08 ` Linus Walleij
2020-12-02 14:07   ` Stephan Gerhold
2020-12-09 18:17 ` Rob Herring
2020-12-13 13:25   ` Jonathan Cameron
2021-01-14 20:59     ` Jonathan Cameron [this message]
2021-01-15  8:37       ` Stephan Gerhold
2021-01-15 13:56         ` Jonathan Cameron

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=20210114205937.11e650d0@archlinux \
    --to=jic23@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=lars@metafoo.de \
    --cc=linus.walleij@linaro.org \
    --cc=linux-iio@vger.kernel.org \
    --cc=pmeerw@pmeerw.net \
    --cc=robh+dt@kernel.org \
    --cc=robh@kernel.org \
    --cc=stephan@gerhold.net \
    /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.