Bitbang SPI phase=1 lowest bit is wrong

[gingle]

CircuitPython version

Adafruit CircuitPython 6.1.0 on 2021-01-21; Adafruit QT Py M0 with samd21e18 also
Adafruit CircuitPython 7.0.0-beta.0-161-g65753a1c2 on 2021-09-01; with samd51j19

Code/REPL

# 09/03/2021
# looses both least significant bits
# tested on QT Py and Metro_M4_Express

import board
import digitalio
import adafruit_bitbangio as bitbangio


nEN = digitalio.DigitalInOut(board.A1)    
nEN.direction = digitalio.Direction.OUTPUT

spi = bitbangio.SPI(board.SCK, MOSI=board.MOSI, MISO=board.MISO)

def TestSPI():  
    select = nEN
    select.value = True
    while not spi.try_lock():
        pass
    spi.configure(baudrate=100000, polarity=0, phase=1, bits=8 )
    bytes_read = bytearray(4)
    select.value = False
    bytes_read = bytearray([0x33])  # changes 0x33 to 0x32 
    spi.write(bytes_read)
    bytes_read = bytearray([0x0b])  # changes 0x0b to 0x0a
    spi.write(bytes_read)
    select.value = True
    spi.unlock()

Behavior

Description

No response

Additional information

This works correctly:
from adafruit_bus_device.spi_device import SPIDevice

[dhalbert]

Thanks for the report. A couple of questions:

1. Does your device require phase=1? Is it misbehaving?
2. Did you set the Saleae Logic SPI analyzer to CPHA=1? Screenshot below is from Logic 2.3.36:
   

The operative code is here, and looks OK to me on first glance:
https://github.com/adafruit/circuitpython/blob/d29a12cd81c12b76ce94317074c6396755ccde6c/shared-module/bitbangio/SPI.c#L152-L167

[gingle]

Hi Dan, I first noticed this as I was driving a DDS chip. Using busio the DDS did not set to quite the right frequency. The LSBit is generally not a big impact but for the DDS it was the LSbit in the msb of the frequency set and set the frequency about 10% low over what it should have been. I was using the equivalent of the metro_m4_express. I then did a Busio version which worked OK with the same polarity/phase settings. I then tried the bitbang version on the QT PY which is the waveform that I included with the problem report. I tried both a 16 bit send and 2 eight bit sends. The two 8 bit sends dropped both LSbits. I am including the bitbang/SPI comparison waveforms. [Inline image] Regards, Glen

[dhalbert]

Thanks, your inline image did not come through. Could you post directly in GitHub?
Which DDS chip is it?

[I edited your reply to remove the repetition of my post.]

[gingle]

Hi Dan, I am not well versed in Github unfortunately. One more thing to master. I posted the waveforms and programs on the Adafruit Forum under 'SPI Busio vs Bitbangio' on sept 3. It is under gingle in Adafruit Circuitpython and Micropython. The DDS was an AD9832. Could you send any replies to ***@***.******@***.***>? I am doing most of my work at home and have my files on my home computer. Thanks, Glen

[dhalbert]

I just realized you are using the https://github.com/adafruit/Adafruit_CircuitPython_BitbangIO Python-based library (import adafruit_bitbangio as bitbangio), not the built-in bitbangiomodule. Do you see the same problem when using native import bitbangio?

[dhalbert]

Could you send any replies to ***@***.******@***.***>? I am doing most of my work at home and have my files on my home computer.

We use GitHub so everyone can see the replies. When you get an email from GitHub, don't reply via mail, but instead click the "View it on GitHub" link in the email:

That will bring you directly here so you can reply.

[dhalbert]

Transferred to https://github.com/adafruit/Adafruit_CircuitPython_BitbangIO/

[gingle]

Dan, you are correct. I verified the built in version of bitbangio as working correctly. the Adafruit version has the LSbit problem.

[Linkeor]

Bonjour,
I don't think I'm in the wrong place. I had to test a max31856 and a tft st7735R with bitbangio, on a pico/u2if. I can read but not write in the registers, or at least the values that come out of the register are not at all those written and of course my temperatures are wrong and the screen displays nothing.
After exploring and testing a lot of things I modified def write( in adafruit_bitbangio.py and everything now works (for the max31856 in phase 1 as for the st7735 in phase 0)
This happens on lines 401 and 407 where
401
self._sclk.value = self._polarity
becomes
self._sclk.value = not self._polarity
and 407
self._sclk.value = not self._polarity
becomes
self._sclk.value = self._polarity

I don't know if this is the best way to do things and if it could have side effects, I'm not an experienced developer. But for now this is a modification that works for me.
I hope this can help.

PS sorry, for English via google translate, I read well but I speak English "like a Frenchman" ;-)