Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

drivers: sensor: Liteon LTR329 sensor support #85453

Merged
merged 3 commits into from
Apr 1, 2025

Conversation

konrad1s
Copy link
Contributor

@konrad1s konrad1s commented Feb 8, 2025

Hello,
This PR introduces following changes:

  • support for the Liteon LTR329 Digital Ambient Light Sensor
  • vendor prefix update from ltr to liteon. ltr only referes to the light sensors.
  • move drivers to the liteon vendor subdirectory

Datasheet for the sensor is available at the link: datasheet
The ambient light conversion equation is as shown in the following document: LTR-303ALS & LTR-329ALS Appendix A

All changes tested:
img

This PR is related to the #65832. I created new PR since the original one was closed for over 1year, the changes have also been refactored and for some reason I don't have reopen button

@konrad1s konrad1s force-pushed the liteon329-sensor-support branch from cd1ca48 to 1549765 Compare February 9, 2025 22:16
@konrad1s konrad1s requested a review from yperess February 9, 2025 22:21
@yperess
Copy link
Collaborator

yperess commented Feb 18, 2025

Need to rebase this PR

@konrad1s konrad1s force-pushed the liteon329-sensor-support branch from 1549765 to ee85dbb Compare February 23, 2025 11:20
yperess
yperess previously approved these changes Feb 26, 2025
@konrad1s konrad1s force-pushed the liteon329-sensor-support branch 3 times, most recently from cc18882 to b1c9cc9 Compare March 5, 2025 20:46
Copy link
Collaborator

@jeppenodgaard jeppenodgaard left a comment

Choose a reason for hiding this comment

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

Commit message title has a typo: suport

@konrad1s konrad1s force-pushed the liteon329-sensor-support branch from b1c9cc9 to bdf552d Compare March 6, 2025 18:25
@konrad1s konrad1s force-pushed the liteon329-sensor-support branch 2 times, most recently from 90d3c4e to 20bb210 Compare March 29, 2025 13:42
@github-actions github-actions bot added the Release Notes To be mentioned in the release notes label Mar 29, 2025
yperess
yperess previously approved these changes Mar 29, 2025
The vendor prefix has been changed from 'ltr' to 'liteon'.
This change is made to better reflect the actual name of
the vendor, which is LiteOn. The previous prefix 'ltr'
was referring to the LiteOn Optical Sensor family, which
could lead to misunderstandings.

Signed-off-by: Konrad Sikora <kontakt@konradsikora.pl>
Add support for Liteon LTR329 Digital Ambient Light Sensor

Signed-off-by: Konrad Sikora <kontakt@konradsikora.pl>
Organizes liteon sensor drivers by vendor to make consistent
with other drivers.

Signed-off-by: Konrad Sikora <kontakt@konradsikora.pl>
@kartben kartben merged commit c099a01 into zephyrproject-rtos:main Apr 1, 2025
25 checks passed
@konrad1s konrad1s deleted the liteon329-sensor-support branch April 2, 2025 08:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Devicetree area: Sensors Sensors Release Notes To be mentioned in the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants