Skip to content

Conversation

@pdgendt
Copy link
Contributor

@pdgendt pdgendt commented Sep 7, 2022

Add 2 properties to configure the "any movement" event.

  • Ability to disable the interrupt latch
  • Select movement mode

@pdgendt pdgendt force-pushed the drivers-sensor-lis2dh-anym branch from 51cb602 to edbad12 Compare September 7, 2022 14:41
@pdgendt pdgendt requested a review from MaureenHelm September 7, 2022 14:41
@pdgendt pdgendt force-pushed the drivers-sensor-lis2dh-anym branch from edbad12 to c9119d3 Compare September 14, 2022 07:22
@pdgendt pdgendt force-pushed the drivers-sensor-lis2dh-anym branch from c9119d3 to df043ea Compare September 14, 2022 07:24
Copy link
Contributor

@mbolivar-nordic mbolivar-nordic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry I was unclear, but this is still not done:

#50004 (comment)

Can you please move the YAML comments explaining the values to the description field, so they show up in the bindings index?

Add 2 properties to configure the "any movement" event.
* Ability to disable the interrupt latch
* Select movement mode

Signed-off-by: Pieter De Gendt <[email protected]>
Remove 'required: false' for boolean properties.

Signed-off-by: Pieter De Gendt <[email protected]>
Remove 'required: false' for boolean properties.

Signed-off-by: Pieter De Gendt <[email protected]>
Remove 'required: false' for boolean properties.

Signed-off-by: Pieter De Gendt <[email protected]>
@pdgendt pdgendt force-pushed the drivers-sensor-lis2dh-anym branch from df043ea to 9f716d3 Compare September 15, 2022 07:03
@fabiobaltieri fabiobaltieri added this to the v3.3.0 milestone Sep 23, 2022
@carlescufi carlescufi merged commit 58e8e56 into zephyrproject-rtos:main Oct 3, 2022
@pdgendt pdgendt deleted the drivers-sensor-lis2dh-anym branch October 5, 2022 11:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants