marekburiak / ILI9341_due

Arduino Due and AVR (Uno, Mega, Nano, Pro Mini,...) library for interfacing with ILI9341 SPI TFTs
http://marekburiak.github.io/ILI9341_due
99 stars 36 forks source link

_spiSettings uninitialized #6

Open Pablo2048 opened 8 years ago

Pablo2048 commented 8 years ago

Hi Marek. Thank You for Your work, but it seems that I found a problem. In SPI transaction mode there is some initialization of _spiSettings missing in begin(...) method.

marekburiak commented 8 years ago

Are you getting an error or somethings not working?

Pablo2048 commented 8 years ago

I'm trying to port Your library to Arduino ESP8266 platform. My TFT is version with touch panel included, so beacuse small I/O pins count on ESP I have to share SPI between TFT and Touch, so I have to use transactions. I'm actually in testing phase so I found out that _spiSettings are not initialized in .begin() - after studying SPI.h from Arduino ESP8266 there is default setting (in object constructor) for _bitOrder LSBFIRST, but ILI9341 need MSBFIRST.

ppostuma commented 8 years ago

Hi Marek - fantastic work! I too am interested in a ESP8266 version (my new toy of choice is the Digistump Oak, which I hope I'll be working with for quite some time, and using for quite some projects). I have gotten the basic Adafruit library to work but yours looks far more useable and powerful. I would be happy to send you one if you're interested in looking at it. Pablo, if you're reading, have you had any luck?

Pablo2048 commented 8 years ago

Hi ppostuma - well, sort of. I have partially working code, but is way slower than optimized Adafruit variant (AS8).

marekburiak commented 8 years ago

I have a few ESP8266 myself and was thinking to port the library for it but unfortunately I have no time for it at the moment (other projects in the way).

ppostuma commented 8 years ago

understand! thanks again