updated SPI API & impl to match current WiringPi version

This commit is contained in:
Robert Savage 2017-08-11 11:00:47 -04:00
parent 03a8ddeadd
commit 73fb0d8434
2 changed files with 31 additions and 14 deletions

View File

@ -1,7 +1,7 @@
/* /*
* wiringPiSPI.c: * wiringPiSPI.c:
* Simplified SPI access routines * Simplified SPI access routines
* Copyright (c) 2012 Gordon Henderson * Copyright (c) 2012-2015 Gordon Henderson
*********************************************************************** ***********************************************************************
* This file is part of wiringPi: * This file is part of wiringPi:
* https://projects.drogon.net/raspberry-pi/wiringpi/ * https://projects.drogon.net/raspberry-pi/wiringpi/
@ -28,6 +28,7 @@
#include <errno.h> #include <errno.h>
#include <string.h> #include <string.h>
#include <sys/ioctl.h> #include <sys/ioctl.h>
#include <asm/ioctl.h>
#include <linux/spi/spidev.h> #include <linux/spi/spidev.h>
#include "wiringPi.h" #include "wiringPi.h"
@ -38,11 +39,10 @@
// The SPI bus parameters // The SPI bus parameters
// Variables as they need to be passed as pointers later on // Variables as they need to be passed as pointers later on
const static char *spiDev0 = "/dev/spidev0.0" ; static const char *spiDev0 = "/dev/spidev0.0" ;
const static char *spiDev1 = "/dev/spidev0.1" ; static const char *spiDev1 = "/dev/spidev0.1" ;
const static uint8_t spiMode = 0 ; static const uint8_t spiBPW = 8 ;
const static uint8_t spiBPW = 8 ; static const uint16_t spiDelay = 0 ;
const static uint16_t spiDelay = 0 ;
static uint32_t spiSpeeds [2] ; static uint32_t spiSpeeds [2] ;
static int spiFds [2] ; static int spiFds [2] ;
@ -75,6 +75,11 @@ int wiringPiSPIDataRW (int channel, unsigned char *data, int len)
channel &= 1 ; channel &= 1 ;
// Mentioned in spidev.h but not used in the original kernel documentation
// test program )-:
memset (&spi, 0, sizeof (spi)) ;
spi.tx_buf = (unsigned long)data ; spi.tx_buf = (unsigned long)data ;
spi.rx_buf = (unsigned long)data ; spi.rx_buf = (unsigned long)data ;
spi.len = len ; spi.len = len ;
@ -87,16 +92,17 @@ int wiringPiSPIDataRW (int channel, unsigned char *data, int len)
/* /*
* wiringPiSPISetup: * wiringPiSPISetupMode:
* Open the SPI device, and set it up, etc. * Open the SPI device, and set it up, with the mode, etc.
********************************************************************************* *********************************************************************************
*/ */
int wiringPiSPISetup (int channel, int speed) int wiringPiSPISetupMode (int channel, int speed, int mode)
{ {
int fd ; int fd ;
channel &= 1 ; mode &= 3 ; // Mode is 0, 1, 2 or 3
channel &= 1 ; // Channel is 0 or 1
if ((fd = open (channel == 0 ? spiDev0 : spiDev1, O_RDWR)) < 0) if ((fd = open (channel == 0 ? spiDev0 : spiDev1, O_RDWR)) < 0)
return wiringPiFailure (WPI_ALMOST, "Unable to open SPI device: %s\n", strerror (errno)) ; return wiringPiFailure (WPI_ALMOST, "Unable to open SPI device: %s\n", strerror (errno)) ;
@ -105,12 +111,10 @@ int wiringPiSPISetup (int channel, int speed)
spiFds [channel] = fd ; spiFds [channel] = fd ;
// Set SPI parameters. // Set SPI parameters.
// Why are we reading it afterwriting it? I've no idea, but for now I'm blindly
// copying example code I've seen online...
if (ioctl (fd, SPI_IOC_WR_MODE, &spiMode) < 0) if (ioctl (fd, SPI_IOC_WR_MODE, &mode) < 0)
return wiringPiFailure (WPI_ALMOST, "SPI Mode Change failure: %s\n", strerror (errno)) ; return wiringPiFailure (WPI_ALMOST, "SPI Mode Change failure: %s\n", strerror (errno)) ;
if (ioctl (fd, SPI_IOC_WR_BITS_PER_WORD, &spiBPW) < 0) if (ioctl (fd, SPI_IOC_WR_BITS_PER_WORD, &spiBPW) < 0)
return wiringPiFailure (WPI_ALMOST, "SPI BPW Change failure: %s\n", strerror (errno)) ; return wiringPiFailure (WPI_ALMOST, "SPI BPW Change failure: %s\n", strerror (errno)) ;
@ -119,3 +123,15 @@ int wiringPiSPISetup (int channel, int speed)
return fd ; return fd ;
} }
/*
* wiringPiSPISetup:
* Open the SPI device, and set it up, etc. in the default MODE 0
*********************************************************************************
*/
int wiringPiSPISetup (int channel, int speed)
{
return wiringPiSPISetupMode (channel, speed, 0) ;
}

View File

@ -28,6 +28,7 @@ extern "C" {
int wiringPiSPIGetFd (int channel) ; int wiringPiSPIGetFd (int channel) ;
int wiringPiSPIDataRW (int channel, unsigned char *data, int len) ; int wiringPiSPIDataRW (int channel, unsigned char *data, int len) ;
int wiringPiSPISetupMode (int channel, int speed, int mode) ;
int wiringPiSPISetup (int channel, int speed) ; int wiringPiSPISetup (int channel, int speed) ;
#ifdef __cplusplus #ifdef __cplusplus