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.

Sign up for GitHub

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

Jump to bottom

Neopixel support #177

Merged
merged 4 commits into from
Aug 23, 2024
Merged

Neopixel support #177

merged 4 commits into from
Aug 23, 2024

Conversation

IFX-Anusha
Copy link

  • Support for most common neopixel timing sequences {400, 850, 800, 450}, {300, 900, 600, 600}, {800, 1700, 1600, 900}.
  • Neopixel documentation

@ederjc ederjc force-pushed the neopixel-support branch 2 times, most recently from ffdade1 to c44d3a9 Compare August 21, 2024 10:17
jaenrig-ifx
jaenrig-ifx approved these changes Aug 21, 2024
View reviewed changes
Copy link
Member

@jaenrig-ifx jaenrig-ifx left a comment

Choose a reason for hiding this comment

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

All good 👯
Just moving the neopixel docs to the agreed level.

ederjc
ederjc approved these changes Aug 21, 2024
View reviewed changes
Copy link
Member

@ederjc ederjc left a comment

Choose a reason for hiding this comment

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

Perfect, given that the section will be moved 👍

NikhitaR-IFX
NikhitaR-IFX approved these changes Aug 21, 2024
View reviewed changes
Copy link
Member

@NikhitaR-IFX NikhitaR-IFX left a comment

Choose a reason for hiding this comment

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

Same.. Nice!

ramya-subramanyam
ramya-subramanyam approved these changes Aug 21, 2024
View reviewed changes
Copy link

@ramya-subramanyam ramya-subramanyam left a comment

Choose a reason for hiding this comment

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

good :)

@jaenrig-ifx
Copy link
Member

Yep, all properly parsing: https://ifx-micropython.readthedocs.io/en/neopixel-support/psoc6/quickref.html

@IFX-Anusha IFX-Anusha force-pushed the neopixel-support branch 2 times, most recently from 2c35c51 to 4d6fc7f Compare August 23, 2024 05:32
IFX-Anusha and others added 4 commits August 23, 2024 11:05
@IFX-Anusha
ports/psoc6: Support for WS218B and SK6812.
5f18156 
Signed-off-by: IFX-Anusha <Anusha.TR@infineon.com>
@IFX-Anusha
docs/psoc6: Neopixel documentation.
1b78e1e 
Signed-off-by: IFX-Anusha <Anusha.TR@infineon.com>
@ederjc @IFX-Anusha
docs/psoc6: Fix wording/typos in docs.
845ff63 
Signed-off-by: Eder Julian <Julian.Eder@infineon.com>
@IFX-Anusha
docs/psoc6: Documentation rearrangement.
a0c8f10 
Signed-off-by: IFX-Anusha <Anusha.TR@infineon.com>
@IFX-Anusha IFX-Anusha merged commit c66f067 into ports-psoc6-main Aug 23, 2024
30 of 31 checks passed
@IFX-Anusha IFX-Anusha deleted the neopixel-support branch August 26, 2024 06:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ramya-subramanyam ramya-subramanyam ramya-subramanyam approved these changes

@NikhitaR-IFX NikhitaR-IFX NikhitaR-IFX approved these changes

@jaenrig-ifx jaenrig-ifx jaenrig-ifx approved these changes

@ederjc ederjc ederjc approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@IFX-Anusha @jaenrig-ifx @ederjc @NikhitaR-IFX @ramya-subramanyam