Skip to content

Conversation

@rumpelsepp
Copy link
Contributor

No description provided.

@pelwell
Copy link
Contributor

pelwell commented May 30, 2017

That's a good start. Can you:

  1. Keep the sensors in alphabetical order.
  2. Add htu21 to the README (try dtoverlay -h i2c-sensor in a shell). Stick the the strict whitespace rules - no TABs, same indentation.

If you fix those in one or more follow-up commits we can squash them when merging.

@rumpelsepp
Copy link
Contributor Author

Updated.

@pelwell
Copy link
Contributor

pelwell commented May 30, 2017

Your README entry says the default address is 0x40. Can the device have other addresses? If so, we should make the addresses selectable through a parameter. If not, you should remove the word "default".

@rumpelsepp
Copy link
Contributor Author

Your README entry says the default address is 0x40. Can the device have other addresses? If so, we should make the addresses selectable through a parameter. If not, you should remove the word "default".

I will check this and report back, can't remember it atm.

@rumpelsepp
Copy link
Contributor Author

It seems, that the address is hardcoded: https://cdn-shop.adafruit.com/datasheets/1899_HTU21D.pdf

I will remove the address configuration opportunity.

@pelwell
Copy link
Contributor

pelwell commented May 31, 2017

You need the parameter, otherwise there is no way to select your sensor. Put the parameter back, leave everything else as it is and I'll merge it.

@pelwell
Copy link
Contributor

pelwell commented May 31, 2017

The i2c-sensor overlay has now been restructured internally, breaking your PR. I'll patch it up and merge by hand.

@rumpelsepp
Copy link
Contributor Author

Oh nice, thanks! Then I can close this?

@pelwell
Copy link
Contributor

pelwell commented May 31, 2017

I'll do it when I've finished - give me a few minutes.

@rumpelsepp
Copy link
Contributor Author

No need to hurry. :) thanks!

pelwell pushed a commit that referenced this pull request May 31, 2017
See: #2041

Signed-off-by: Phil Elwell <phil@raspberrypi.org>
@pelwell
Copy link
Contributor

pelwell commented May 31, 2017

Merged manually.

@pelwell pelwell closed this May 31, 2017
pelwell pushed a commit that referenced this pull request Jul 10, 2017
See: #2041

Signed-off-by: Phil Elwell <phil@raspberrypi.org>
pelwell pushed a commit that referenced this pull request Jul 10, 2017
See: #2041

Signed-off-by: Phil Elwell <phil@raspberrypi.org>
popcornmix pushed a commit that referenced this pull request Jul 12, 2017
See: #2041

Signed-off-by: Phil Elwell <phil@raspberrypi.org>
popcornmix pushed a commit that referenced this pull request Jul 13, 2017
See: #2041

Signed-off-by: Phil Elwell <phil@raspberrypi.org>
popcornmix pushed a commit that referenced this pull request Jul 13, 2017
See: #2041

Signed-off-by: Phil Elwell <phil@raspberrypi.org>
popcornmix pushed a commit that referenced this pull request Jul 16, 2017
See: #2041

Signed-off-by: Phil Elwell <phil@raspberrypi.org>
popcornmix pushed a commit that referenced this pull request Jul 16, 2017
See: #2041

Signed-off-by: Phil Elwell <phil@raspberrypi.org>
popcornmix pushed a commit that referenced this pull request Jul 17, 2017
See: #2041

Signed-off-by: Phil Elwell <phil@raspberrypi.org>
popcornmix pushed a commit that referenced this pull request Jul 21, 2017
See: #2041

Signed-off-by: Phil Elwell <phil@raspberrypi.org>
popcornmix pushed a commit that referenced this pull request Jul 21, 2017
See: #2041

Signed-off-by: Phil Elwell <phil@raspberrypi.org>
popcornmix pushed a commit that referenced this pull request Jul 27, 2017
See: #2041

Signed-off-by: Phil Elwell <phil@raspberrypi.org>
popcornmix pushed a commit that referenced this pull request Jul 29, 2017
See: #2041

Signed-off-by: Phil Elwell <phil@raspberrypi.org>
popcornmix pushed a commit that referenced this pull request Aug 1, 2017
See: #2041

Signed-off-by: Phil Elwell <phil@raspberrypi.org>
popcornmix pushed a commit that referenced this pull request Aug 8, 2017
See: #2041

Signed-off-by: Phil Elwell <phil@raspberrypi.org>
popcornmix pushed a commit that referenced this pull request Aug 8, 2017
See: #2041

Signed-off-by: Phil Elwell <phil@raspberrypi.org>
popcornmix pushed a commit that referenced this pull request Aug 10, 2017
See: #2041

Signed-off-by: Phil Elwell <phil@raspberrypi.org>
popcornmix pushed a commit that referenced this pull request Aug 13, 2017
See: #2041

Signed-off-by: Phil Elwell <phil@raspberrypi.org>
popcornmix pushed a commit that referenced this pull request Aug 13, 2017
See: #2041

Signed-off-by: Phil Elwell <phil@raspberrypi.org>
popcornmix pushed a commit that referenced this pull request Aug 14, 2017
See: #2041

Signed-off-by: Phil Elwell <phil@raspberrypi.org>
popcornmix pushed a commit that referenced this pull request Aug 23, 2017
See: #2041

Signed-off-by: Phil Elwell <phil@raspberrypi.org>
popcornmix pushed a commit that referenced this pull request Aug 23, 2017
See: #2041

Signed-off-by: Phil Elwell <phil@raspberrypi.org>
popcornmix pushed a commit that referenced this pull request Aug 25, 2017
See: #2041

Signed-off-by: Phil Elwell <phil@raspberrypi.org>
popcornmix pushed a commit that referenced this pull request Aug 30, 2017
See: #2041

Signed-off-by: Phil Elwell <phil@raspberrypi.org>
popcornmix pushed a commit that referenced this pull request Sep 1, 2017
See: #2041

Signed-off-by: Phil Elwell <phil@raspberrypi.org>
popcornmix pushed a commit that referenced this pull request Sep 4, 2017
See: #2041

Signed-off-by: Phil Elwell <phil@raspberrypi.org>
popcornmix pushed a commit that referenced this pull request Sep 8, 2017
See: #2041

Signed-off-by: Phil Elwell <phil@raspberrypi.org>
popcornmix pushed a commit that referenced this pull request Sep 13, 2017
See: #2041

Signed-off-by: Phil Elwell <phil@raspberrypi.org>
popcornmix pushed a commit that referenced this pull request Sep 13, 2017
See: #2041

Signed-off-by: Phil Elwell <phil@raspberrypi.org>
popcornmix pushed a commit that referenced this pull request Sep 14, 2017
See: #2041

Signed-off-by: Phil Elwell <phil@raspberrypi.org>
popcornmix pushed a commit that referenced this pull request Sep 14, 2017
See: #2041

Signed-off-by: Phil Elwell <phil@raspberrypi.org>
popcornmix pushed a commit that referenced this pull request Sep 22, 2017
See: #2041

Signed-off-by: Phil Elwell <phil@raspberrypi.org>
popcornmix pushed a commit that referenced this pull request Oct 29, 2017
See: #2041

Signed-off-by: Phil Elwell <phil@raspberrypi.org>
raspbian-autopush pushed a commit to raspbian-packages/linux-4.9 that referenced this pull request Nov 2, 2017
commit e06a6e0
Author: Stefan Tatschner <rumpelsepp@sevenbyte.org>
Date:   Mon May 29 21:46:16 2017 +0200

    Add device tree config for htu21
    
    See: raspberrypi/linux#2041
    
    Signed-off-by: Phil Elwell <phil@raspberrypi.org>


Gbp-Pq: Topic rpi
Gbp-Pq: Name rpi_1277_e06a6e0a41e70e56887f680e1920327b16839b87.patch
ryncsn pushed a commit to ryncsn/linux-rasp that referenced this pull request Nov 21, 2017
See: raspberrypi#2041

Signed-off-by: Phil Elwell <phil@raspberrypi.org>
raspbian-autopush pushed a commit to raspbian-packages/linux-4.9 that referenced this pull request Apr 7, 2018
commit 1fb3e94
Author: Stefan Tatschner <rumpelsepp@sevenbyte.org>
Date:   Mon May 29 21:46:16 2017 +0200

    Add device tree config for htu21
    
    See: raspberrypi/linux#2041
    
    Signed-off-by: Phil Elwell <phil@raspberrypi.org>


Gbp-Pq: Topic rpi
Gbp-Pq: Name rpi_1279_1fb3e942d0d10ddc3d872cbee380655826a5961c.patch
raspbian-autopush pushed a commit to raspbian-packages/linux-4.9 that referenced this pull request Nov 11, 2018
commit 1fb3e94
Author: Stefan Tatschner <rumpelsepp@sevenbyte.org>
Date:   Mon May 29 21:46:16 2017 +0200

    Add device tree config for htu21
    
    See: raspberrypi/linux#2041
    
    Signed-off-by: Phil Elwell <phil@raspberrypi.org>


Gbp-Pq: Topic rpi
Gbp-Pq: Name rpi_1279_1fb3e942d0d10ddc3d872cbee380655826a5961c.patch
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants