Для кодирования пикселей достаточно только нижнего цикла while((SPI1->SR & SPI_SR_TXE) == RESET);, верхний для безопасности, это хорошо, но неправильной последовательностью думаю это нельзя назвать. Ок, пусть будет.

И снова не совсем так ) При любом расположении строка

while((SPI1->SR & SPI_SR_TXE) == RESET); 

служит одной цели - убедиться, что предыдущие данные ушли, и можно писать новые.

Если вернуться к оберткам в виде функций для удобства:

/* ---------------------------------------------------------------------------*/
void ST7735_SendByte(uint8_t data)
{
  while((SPI1->SR & SPI_SR_TXE) == RESET);  
  SPI1->DR = data;
}

/* ---------------------------------------------------------------------------*/

/* ---------------------------------------------------------------------------*/
void ST7735_WaitLastData()
{
  while((SPI1->SR & SPI_SR_TXE) == RESET);
  while((SPI1->SR & SPI_SR_BSY) != RESET);
}

/* ---------------------------------------------------------------------------*/

Запись последовательности байт выглядит так:

for (uint32_t i = 0; i < num; i++)
{
  ST7735_SendByte(*data);
  data++;
}

ST7735_WaitLastData();

По шагам это превращается в:

while((SPI1->SR & SPI_SR_TXE) == RESET);  
SPI1->DR = data;
while((SPI1->SR & SPI_SR_TXE) == RESET);  
SPI1->DR = data;
while((SPI1->SR & SPI_SR_TXE) == RESET);  
SPI1->DR = data;
while((SPI1->SR & SPI_SR_TXE) == RESET);  
SPI1->DR = data;

while((SPI1->SR & SPI_SR_TXE) == RESET);
while((SPI1->SR & SPI_SR_BSY) != RESET);

• Проверяем, что можно писать данные в регистр
• Пишем данные в регистр DR
• Проверяем, что эти данные ушли в сдвиговый регистр = можно писать новые данные
• Пишем данные
• ...
• ...
• После последнего байта проверяем и флаг SPI_SR_TXE и флаг SPI_SR_BSY, только после этого (!) можно менять сигналы на DC и т. д.

Соответственно, если пишем только один байт, то последовательность такая:

while((SPI1->SR & SPI_SR_TXE) == RESET);  
SPI1->DR = data;
while((SPI1->SR & SPI_SR_TXE) == RESET);
while((SPI1->SR & SPI_SR_BSY) != RESET);

• Проверяем, что ушли предыдущие данные
• Пишем новые данные
• Проверяем, что ушли новые данные

Но при отправке последовательности байт проверка флага SPI_SR_BSY выполняется только при передаче последнего байта.

От: @stm

HAL овским функциям может было необходимо деактивировать Чип Селект после каждой передачи данных.
Передаче данных через регистры это не нужно.

CS никак не связан с тем, какие библиотеки используются. Этот сигнал нужен для активации/деактивации slave'а на шине, у меня активируется внутри функций отрисовки:

void ST7735_DrawRect(uint16_t cStart, uint16_t rStart, uint16_t cStop, uint16_t rStop, uint16_t color)
{
  HAL_GPIO_WritePin(ST7735_CS_PORT, ST7735_CS_PIN, GPIO_PIN_RESET);
  ST7735_SetColAddr(cStart, cStop - 1);
  ST7735_SetRowAddr(rStart, rStop - 1);
  
  ST7735_SendCommand(ST7735_RAMWR);
  
  uint32_t size = (cStop - cStart) * (rStop - rStart);
  uint8_t colorBytes;
  colorBytes = (color & 0xFF00) >> 8;
  colorBytes = color & 0x00FF;
  
  HAL_GPIO_WritePin(ST7735_DC_PORT, ST7735_DC_PIN, GPIO_PIN_SET);
  
  for (uint32_t i = 0; i < size; i++)
  {
    ST7735_SendByte(colorBytes);
    ST7735_SendByte(colorBytes);
  }
  
  ST7735_WaitLastData();
  HAL_GPIO_WritePin(ST7735_CS_PORT, ST7735_CS_PIN, GPIO_PIN_SET);
}

Я посмотрел, вы не деактивируете Чип Селект после передачи данных.
Я в драйверах видел деактивацию Чип Селекта после передачи данных, поэтому её всегда вставлял в код.

Я просто хотел описать мысль, а не доказать как будет правильнее)

while((SPI1->SR & SPI_SR_TXE) == RESET);

Ок, верхний цикл while((SPI1->SR & SPI_SR_TXE) == RESET);  для безопасности)

SPI1->DR = data;
while((SPI1->SR & SPI_SR_TXE) == RESET);

Это плюс-минус то же самое, что сесть на стул, а только после этого проверить, не занят ли он.

>Плюс у Вас даже в этих двух строках неправильная последовательность.
У вас для передачи данных служат эти функции:

ST7735_SendByte(data);
ST7735_WaitLastData();

Если их раскрыть, то получится:

while((SPI1->SR & SPI_SR_TXE) == RESET);
SPI1->DR = data;
while((SPI1->SR & SPI_SR_TXE) == RESET);
while((SPI1->SR & SPI_SR_BSY) != RESET);

Для чего у вас два раза? while((SPI1->SR & SPI_SR_TXE) == RESET);
-------------------------------------
Попробуйте заменить все эти 3 while просто HAL_Delay()
Ок, это понятно, что флаги лишь ускоряют передачу и по готовности флагов начинается передача.
Дело немного в другом.
Если не ждать нужное время после передачи, то предыдущая передача повреждается и пиксели у меня перестают кодироваться.

Достаточная пауза HAL_Delay() между передачами и while((SPI1->SR & SPI_SR_TXE) == RESET); по сути одно и то же, но вы сказали, что у меня неправильная последовательность

SPI1->DR = data;
while((SPI1->SR & SPI_SR_TXE) == RESET);

Этот цикл как раз и обеспечивает нужную задержку, чтоб предыдущая передача не повреждалась

while((SPI1->SR & SPI_SR_TXE) == RESET);

У вас этот код работает точно по такому же принципу:

while((SPI1->SR & SPI_SR_TXE) == RESET);
SPI1->DR = data;
while((SPI1->SR & SPI_SR_TXE) == RESET);
while((SPI1->SR & SPI_SR_BSY) != RESET);

@stm Я объяснил, почему дисплей ничего не выводит без проверки флага и тонко намекнул на то, что вот эту дичь надо выкинуть раз и навсегда:

for(int i=0;i<10;i++){}

Плюс у Вас даже в этих двух строках:

SPI1->DR = data;
перед
while((SPI1->SR & SPI_SR_TXE) == RESET)

неправильная последовательность.

А так да, просто так сижу, ерунду пишу от нечего делать видимо )

Для читающих - это мой ответ на удаленное сообщение с претензией, что я будто бы перефразировал то же самое, что ТС написал ранее.