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

Draw Line #3

Open
wants to merge 107 commits into
base: master
Choose a base branch
from
Open

Draw Line #3

wants to merge 107 commits into from

Conversation

simonmcmullon
Copy link

Ive tried using drawline and can only get lines in one direction, that is going from top left to bottom right. If I try to get a line in the other direction, that is top right to bottom left the lines are ether horizontal or vertical.

G4lile0 and others added 30 commits February 17, 2019 18:05
* Add 1-D bitmap array support.

* Fix drawing issue due to wrong _setWindow() args.
* Add _spiWrite16

* Use _spiWrite16 in the drawBitmap functions

* Use HSPI_WRITE_PIXELS in 1D color drawBitmap

* Add _spiWrite16

* 1D drawBitmap should have own description

* j iteratoes over h, so it should be j*w not j*h

* Only actually need one loop

* Add HSPI_WRITE_PIXELS to 2D color drawBitmap

* Fix indentation
…de effects. The (#62)

operations on the port are non-atomic, and can have unwanted and unexpected side effects
on software running on the other CPU core if this software also accesses IO ports.

In addition, FAST_PINIO is also not fast on ESP32. My reference board (T-Beam v1.0)
takes 2.48s to flip a pin 10e7 times with digitalWrite(pin, ON); digitalWrite(pin, OFF).
It takes 2.52s to flip the same pin 10e7 times with *pin |= pinmask; *pin &= ~pinmask.

So this modification causes a slight speed-up and removes serios problems.
Cannot compile with unknown SPI_WRITE16
* remove compiler warnings

* Remove unused variable
Use HSPI_WRITE
I think there is a small type error on line 28 for STM32 boards: uint32 --> uint32_t
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.