AVR Freaks

the datasheet of the screen is : https://fr.rs-online.com/web/p/e...

Yeah I need it ! I think there is a problem with my chip select which don't wait all the byte to make the transfer...

Yeah sorry I edit my post now the link is available !

I have an Arduino program which works well, this is why i try to pass on Atmel 7 so I need to rewrite functions already implemented by arduino like SPI.transfer(...)

The problem of my system actually is that the Chip Select don't wait for all the bytes and send it...

I will look at your link thank you man !

Yeah that's correct, it's just weird name of function ! (because CS is active low they put set() for '0' and clear() for '1')

I have somes news. I desactivate the rise of my CS after a transfer thank to the register SPI_CSR 

I initialize my CS like a GPIO to control it thanks to pio_set_pin_low(CS) and pio_set_pin_high(CS). I saw on a forum that it's the only solution to make multi transfer bytes with SPI. Now my CS works well but my MISO still return 0...

I verified that my MOSI, CS and CLK are correct and they are but not my MISO. I have somes screens of my oscilloscope please look at it in attachement! I have correcte oscilloscope that come from Arduino code which start the screen and bad oscilloscope that come from my actual code.

Do you have any idea about that ?

Edit : 

this is my actual function :

static inline uint8_t spi_transmit(uint8_t data)
{
	while ((SPI0->SPI_SR & SPI_SR_TDRE) == 0);

	spi_write(SPI0, data, SPI_CHIP_SEL, 0);

	while ((SPI0->SPI_SR & SPI_SR_RDRF) == 0);

	uint8_t data_read, Chip_Sel_read;

	spi_read(SPI0, &data_read, &Chip_Sel_read);

	return data_read;
}

And this is my SPI initialization :

/* Configure SPI0 pins */
gpio_configure_pin(SPI0_MISO_GPIO, (PIO_PERIPH_A | PIO_DEFAULT));
gpio_configure_pin(SPI0_MOSI_GPIO, (PIO_PERIPH_A | PIO_DEFAULT));
gpio_configure_pin(SPI0_SPCK_GPIO, (PIO_PERIPH_A | PIO_DEFAULT)); 
gpio_configure_pin(SPI0_NPCS0_GPIO, (PIO_OUTPUT_1 | PIO_DEFAULT)); //CS

/* Configure an SPI peripheral. */
spi_enable_clock(SPI0);

spi_disable(SPI0);
spi_reset(SPI0);

//spi_set_lastxfer(SPI0);
spi_set_master_mode(SPI0);

spi_disable_mode_fault_detect(SPI0);
spi_set_variable_peripheral_select(SPI0);
spi_set_peripheral_chip_select_value(SPI0, SPI_CHIP_SEL); //SPI_CHIP_SEL == 0

/* data samples on rising edge because Polarity = Phase = 0 */
spi_set_clock_polarity(SPI0, SPI_CHIP_SEL, SPI_CLK_POLARITY);
spi_set_clock_phase(SPI0, SPI_CHIP_SEL, SPI_CLK_PHASE);

spi_set_bits_per_transfer(SPI0, SPI_CHIP_SEL, SPI_CSR_BITS_8_BIT);
spi_set_baudrate_div(SPI0, SPI_CHIP_SEL, (sysclk_get_peripheral_hz() / gs_ul_spi_clock));

SPI0->SPI_CSR[0] |= SPI_CSR_CSAAT;

spi_enable(SPI0); 

Hello guys,

when I don't initialize my MISO Pin to disconnect it, my MISO signal keep be the same (the CS and the MOSI steel works).

I don't know yet if the screen is correctly setup, I work on this actually.

Yes it is

I don't know why I had a problem with my CS which rise too fast so to fix temporary this problem I put a delay in us in my multi transfer function :

uint8_t EVE_memRead8(uint32_t ftAddress)
{
	uint8_t ftData8;
	
	EVE_cs_set();
	spi_transmit_32(...); //Make 4 spi_transmit(...)
	ftData8 = spi_receive(0x00); /* read data byte by sending another dummy byte */
	
	delay_us(30);
	EVE_cs_clear();

	return ftData8;	/* return byte read */
}

Look at this correct trace of the MISO signal :

And now look at mine :

I don't know why there is no space between my bytes. I need to search on the datasheet of my screen if it's a problem or not !

Yeah my bad you are right ! I change my tests

Yes but when I put my receive part in my spi_transmit(data) function my MISO always return 0 and when I remove it my MISO looks like better...

this is my spi_transmit_32(data) :

static inline void spi_transmit_32(uint32_t data)
{
	spi_transmit((uint8_t)(data));
	spi_transmit((uint8_t)(data >> 8));
	spi_transmit((uint8_t)(data >> 16));
	spi_transmit((uint8_t)(data >> 24));
}

Now my MISO is not correct but by studing my MOSI I notice that i send the information on rising edge and I want to change to send my information on falling edge !

Maybe the error of my MISO come from the bad MOSI

Yeah i only have a 2-channel scope so I can't take a screen of all my signals...

I put a delay before my CS rising because it raised too fast and i change my clock phase and now I have a perfect MISO signal :

I compare with the Arduino one and it's exactly the same.

The only problem now is that my receive function read the 4th bytes (the return value is 67) and not the 5th bytes (the return value is 124)

uint8_t ftData8;
	
EVE_cs_set();
	
spi_transmit_32(...); //Make 4 spi_transmit
ftData8 = spi_receive(0x00); /* read data byte by sending another dummy byte */
	
delay_us(30); /* Delay because CS rise too fast */
EVE_cs_clear();

return ftData8;	/* return byte read */


static inline uint8_t spi_receive(uint8_t data)
{
	return transmission(data, 1);
}


static inline uint8_t transmission(uint8_t data, uint8_t reception)
{
	//A revoir si y a pas un while pour vérifier si il est vide
	spi_write(SPI0, data, SPI_CHIP_SEL, 0);
	
	while ((SPI0->SPI_SR & SPI_SR_TDRE) == SPI_SR_TDRE);
	
	if(reception)
	{
		uint8_t data_read, Chip_Sel_read;
		
		spi_read(SPI0, &data_read, &Chip_Sel_read);
		
		//return 0;
		return data_read;
	}
	else
	{
		return 1;
	}
}