Skip to content

Update SPI.h - Add transfer_out #4573

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

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

Conversation

pamribeirox
Copy link

An alternative "output only" transfer that doesn't destroy the source buffer with the bytes returned from the slave device.
Some code simplifications and const enforcement of input variables
Tested in ATMEGA328p (UNO)

  • no speed changes noticed (comparing the the original function)
  • writing/reading to an SPI RAM returns the correct values

An alternative "output only" transfer that doesn't destroy the source buffer with the bytes returned from the slave device.
Some code simplifications and const enforcement of input variables
Tested in ATMEGA328p (UNO)
* no speed changes noticed (comparing the the original function)
* writing/reading to an SPI RAM returns the correct values
An alternative "output only" transfer that doesn't destroy the contents of the source buffer with the bytes returned from the slave device. 
   * Some code simplifications and const enforcement
   * Revision with speedup of about 25% in atmega328p tests
   * NOTE: I can't explain that "*(p++ + 1);" being speedier than "*++p"
Near cosmetic change, in the cast, keep the pointer to const values
Avoid double calls in several methods by changing them to inline
When verbose output is enabled, bossac was run in debug mode, changed to only extra info (-i)
@sandeepmistry sandeepmistry added the Library: SPI The SPI Arduino library label Jul 18, 2016
@CLAassistant
Copy link

CLAassistant commented Apr 9, 2021

CLA assistant check
All committers have signed the CLA.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Library: SPI The SPI Arduino library
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants