4

I have been using the following link for reference... http://robsraspberrypi.blogspot.co.za/2016/01/raspberry-pi-adding-more-inputs-using.html

// setup SPI for communication
int fd = Spi.wiringPiSPISetup(1, 1000000);
if (fd <= -1) {
      throw new IOException("SPI port setup failed.");
}
byte packet[] = new byte[1];
// send data packet
int i Spi.wiringPiSPIDataRW(0, packet, 1);
techraf
  • 4,319
  • 10
  • 31
  • 42
Brian
  • 51
  • 3
  • Have you contacted the author? – joan Jul 25 '16 at 11:29
  • 1
    What is not working? What is the question you need answered ? – KennetRunner Jul 25 '16 at 11:56
  • The code just returns a 1 when calling the java code above and i dont know why. Does anyone know if this is the correct way of doing it – Brian Jul 25 '16 at 12:44
  • Can you post a schematic as how you have wired it including all power and ground connections. The frizzy picture in your link is incomplete. – Gil Nov 30 '22 at 04:20
  • Consider purchasing one of the inexpensive 24MHz logic analyzers and then you can watch the clocks etc to be sure you are following the data sheet. Some of the software for these will show you the SPI data and convert to Hex, Ascii, Decimal, etc. You then need to compare this to the data sheet. The important thing is the loks and the Latch timing. The latch must remain stable while the bits are shifted out. – Gil Nov 30 '22 at 17:39
  • Not knowing your layout, capacitance, etc try at a much slower bit rate, much less susceptible to errors. you can always raise the bit rate. Before I had a logic analyzer I would use a scope. I even slowed down the clocks to sub herz just to watch the LEDs on the latch and clock pins. If you are not clocking it correctly the data will not be correct. – Gil Nov 30 '22 at 17:39

1 Answers1

1

Okay, so it looks like there a number of issues with your code. I'm not sure it would even run. This line...

int fd = Spi.wiringPiSPISetup(1, 1000000);

...is setting up channel 1 for 1Mhz (which should be fine), but later you are trying to read/write on channel 0 (which has not yet been setup...)

int i Spi.wiringPiSPIDataRW(0, packet, 1);

I also think you need a = in there. int i = Spi.wiringPiSPIDataRW(0, packet, 1); This line...

byte packet[] = new byte[1];

...looks like it declares an array of 1 byte (?? I'm no java expert ??) but it is not initialized with any value ? That means you are sending whatever value happens to be in that byte at the time to the SPI bus and reading back it's output...

Finally - I don't think that you get the result in i (if that's what you were expecting ?), the docs for wiringPiSPI say:

  • int wiringPiSPIDataRW (int channel, unsigned char *data, int len);

This performs a simultaneous write/read transaction over the selected SPI bus. Data that was in your buffer is overwritten by data returned from the SPI bus.

So the data you are trying to read should end up in packet.

I would spend some more time looking at wiringPiSPI and some of the questions/comments here.

Community
  • 1
KennetRunner
  • 1,050
  • 7
  • 13
  • One clarification: In Java this line byte packet[] = new byte[1]; will create an array of length 1 and initialize the contents to 0. That's different than some other languages, where you would get a garbage value in the array at initialization. Rest of this seems right. – Brick Nov 05 '17 at 13:28