diff mbox series

[v2] iio: pressure: bmp280: fix datasheet links

Message ID 20220912222645.377874-1-ang.iglesiasg@gmail.com (mailing list archive)
State Accepted
Headers show
Series [v2] iio: pressure: bmp280: fix datasheet links | expand

Commit Message

Angel Iglesias Sept. 12, 2022, 10:26 p.m. UTC 
Updated links for BMP280 and BME280 datasheets on Bosch website.
Datasheet of BMP180 is no longer available on the manufacturer's website,
changed the link to a copy hosted by a third party.

Changelog in v2:
* Added a notice about the situation with bmp180 datasheet and list
  missing changes from newer versions.

Reported-by: Andy Shevchenko <andy.shevchenko@gmail.com>
Signed-off-by: Angel Iglesias <ang.iglesiasg@gmail.com>
---
 drivers/iio/pressure/bmp280-core.c | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)


base-commit: 2f61ff8272967c9bdcba810aa978170814b08f7c

Comments

Jonathan Cameron Sept. 15, 2022, 1:39 p.m. UTC | #1 
On Tue, 13 Sep 2022 00:26:44 +0200
Angel Iglesias <ang.iglesiasg@gmail.com> wrote:

> Updated links for BMP280 and BME280 datasheets on Bosch website.
> Datasheet of BMP180 is no longer available on the manufacturer's website,
> changed the link to a copy hosted by a third party.
> 
> Changelog in v2:
> * Added a notice about the situation with bmp180 datasheet and list
>   missing changes from newer versions.
Change log needs to be below the ---

We don't want that level of detail in the git logs.

I tidied that up whilst applying. 

Also, it's not a good idea to have new versions as replies
to older ones.  Leads to very deep nesting in email threads
and generally makes things less readable.  Just post
a fresh series each time.

Applied to the togreg branch of iio.git and pushed out as testing 
for 0-day to see if it can find anything (rather unlikely on this
patch!)

Jonathan



> 
> Reported-by: Andy Shevchenko <andy.shevchenko@gmail.com>
> Signed-off-by: Angel Iglesias <ang.iglesiasg@gmail.com>
> ---
>  drivers/iio/pressure/bmp280-core.c | 12 +++++++++---
>  1 file changed, 9 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/iio/pressure/bmp280-core.c b/drivers/iio/pressure/bmp280-core.c
> index fe7aa81e7cc9..01cd32003ca8 100644
> --- a/drivers/iio/pressure/bmp280-core.c
> +++ b/drivers/iio/pressure/bmp280-core.c
> @@ -9,9 +9,15 @@
>   * Driver for Bosch Sensortec BMP180 and BMP280 digital pressure sensor.
>   *
>   * Datasheet:
> - * https://ae-bst.resource.bosch.com/media/_tech/media/datasheets/BST-BMP180-DS000-121.pdf
> - * https://ae-bst.resource.bosch.com/media/_tech/media/datasheets/BST-BMP280-DS001-12.pdf
> - * https://ae-bst.resource.bosch.com/media/_tech/media/datasheets/BST-BME280_DS001-11.pdf
> + * https://cdn-shop.adafruit.com/datasheets/BST-BMP180-DS000-09.pdf
> + * https://www.bosch-sensortec.com/media/boschsensortec/downloads/datasheets/bst-bmp280-ds001.pdf
> + * https://www.bosch-sensortec.com/media/boschsensortec/downloads/datasheets/bst-bme280-ds002.pdf
> + *
> + * Notice:
> + * The link to the bmp180 datasheet points to an outdated version missing these changes:
> + * - Changed document referral from ANP015 to BST-MPS-AN004-00 on page 26
> + * - Updated equation for B3 param on section 3.5 to ((((long)AC1 * 4 + X3) << oss) + 2) / 4
> + * - Updated RoHS directive to 2011/65/EU effective 8 June 2011 on page 26
>   */
>  
>  #define pr_fmt(fmt) "bmp280: " fmt
> 
> base-commit: 2f61ff8272967c9bdcba810aa978170814b08f7c
Angel Iglesias Sept. 15, 2022, 2:59 p.m. UTC | #2 
On jue, 2022-09-15 at 14:39 +0100, Jonathan Cameron wrote:
> On Tue, 13 Sep 2022 00:26:44 +0200
> Angel Iglesias <ang.iglesiasg@gmail.com> wrote:
> 
> > Updated links for BMP280 and BME280 datasheets on Bosch website.
> > Datasheet of BMP180 is no longer available on the manufacturer's website,
> > changed the link to a copy hosted by a third party.
> > 
> > Changelog in v2:
> > * Added a notice about the situation with bmp180 datasheet and list
> >   missing changes from newer versions.
> Change log needs to be below the ---
> 
> We don't want that level of detail in the git logs.
> 
> I tidied that up whilst applying. 
> 
> Also, it's not a good idea to have new versions as replies
> to older ones.  Leads to very deep nesting in email threads
> and generally makes things less readable.  Just post
> a fresh series each time.
> 
> Applied to the togreg branch of iio.git and pushed out as testing 
> for 0-day to see if it can find anything (rather unlikely on this
> patch!)
> 
> Jonathan
> 

OK, I'll keep it on my mind for future contributions. Thanks for your help and
patience!

Kind regards,
Angel

> 
> > 
> > Reported-by: Andy Shevchenko <andy.shevchenko@gmail.com>
> > Signed-off-by: Angel Iglesias <ang.iglesiasg@gmail.com>
> > ---
> >  drivers/iio/pressure/bmp280-core.c | 12 +++++++++---
> >  1 file changed, 9 insertions(+), 3 deletions(-)
> > 
> > diff --git a/drivers/iio/pressure/bmp280-core.c
> > b/drivers/iio/pressure/bmp280-core.c
> > index fe7aa81e7cc9..01cd32003ca8 100644
> > --- a/drivers/iio/pressure/bmp280-core.c
> > +++ b/drivers/iio/pressure/bmp280-core.c
> > @@ -9,9 +9,15 @@
> >   * Driver for Bosch Sensortec BMP180 and BMP280 digital pressure sensor.
> >   *
> >   * Datasheet:
> > - *
> > https://ae-bst.resource.bosch.com/media/_tech/media/datasheets/BST-BMP180-DS000-121.pdf
> > - *
> > https://ae-bst.resource.bosch.com/media/_tech/media/datasheets/BST-BMP280-DS001-12.pdf
> > - *
> > https://ae-bst.resource.bosch.com/media/_tech/media/datasheets/BST-BME280_DS001-11.pdf
> > + * https://cdn-shop.adafruit.com/datasheets/BST-BMP180-DS000-09.pdf
> > + *
> > https://www.bosch-sensortec.com/media/boschsensortec/downloads/datasheets/bst-bmp280-ds001.pdf
> > + *
> > https://www.bosch-sensortec.com/media/boschsensortec/downloads/datasheets/bst-bme280-ds002.pdf
> > + *
> > + * Notice:
> > + * The link to the bmp180 datasheet points to an outdated version missing
> > these changes:
> > + * - Changed document referral from ANP015 to BST-MPS-AN004-00 on page 26
> > + * - Updated equation for B3 param on section 3.5 to ((((long)AC1 * 4 + X3)
> > << oss) + 2) / 4
> > + * - Updated RoHS directive to 2011/65/EU effective 8 June 2011 on page 26
> >   */
> >  
> >  #define pr_fmt(fmt) "bmp280: " fmt
> > 
> > base-commit: 2f61ff8272967c9bdcba810aa978170814b08f7c
>
diff mbox series

Patch

diff --git a/drivers/iio/pressure/bmp280-core.c b/drivers/iio/pressure/bmp280-core.c
index fe7aa81e7cc9..01cd32003ca8 100644
--- a/drivers/iio/pressure/bmp280-core.c
+++ b/drivers/iio/pressure/bmp280-core.c
@@ -9,9 +9,15 @@ 
  * Driver for Bosch Sensortec BMP180 and BMP280 digital pressure sensor.
  *
  * Datasheet:
- * https://ae-bst.resource.bosch.com/media/_tech/media/datasheets/BST-BMP180-DS000-121.pdf
- * https://ae-bst.resource.bosch.com/media/_tech/media/datasheets/BST-BMP280-DS001-12.pdf
- * https://ae-bst.resource.bosch.com/media/_tech/media/datasheets/BST-BME280_DS001-11.pdf
+ * https://cdn-shop.adafruit.com/datasheets/BST-BMP180-DS000-09.pdf
+ * https://www.bosch-sensortec.com/media/boschsensortec/downloads/datasheets/bst-bmp280-ds001.pdf
+ * https://www.bosch-sensortec.com/media/boschsensortec/downloads/datasheets/bst-bme280-ds002.pdf
+ *
+ * Notice:
+ * The link to the bmp180 datasheet points to an outdated version missing these changes:
+ * - Changed document referral from ANP015 to BST-MPS-AN004-00 on page 26
+ * - Updated equation for B3 param on section 3.5 to ((((long)AC1 * 4 + X3) << oss) + 2) / 4
+ * - Updated RoHS directive to 2011/65/EU effective 8 June 2011 on page 26
  */
 
 #define pr_fmt(fmt) "bmp280: " fmt