Talking to LCD via RS232 and Hyperterminal

Blake Ethridge said:

Sounds like we need to get you an updated Avatar :)

Yes, indeed! He needs the best avatar for his contribution! :)

CaEngineer said:

What if I don't use interrupt and would like to send a message by Hyper-Terminal and show it on the display by having while and if as follows?

Not using interrupts makes things more difficult but not impossible. Your main code then needs to implement sort of a state machine. It runs in a loop and only does s mall chunk of work at a time, noting in a variable which part was done and what has to be done next. Then in returns to the main loop, checks for incoming data, stores it, checks whether an incoming command is complete and then does the next chunk of work.

You must ensure that the time between two main loop executions is smaller than the time required for receiving a data byte (so it is only suitable for lower baudrates).

Interrupts are much easier: You receive a byte and write it down in a buffer and set a flag that new data is here. The main code can then check for this flag every now and then and see whether incoming data is complete and action must be taken - If not, it can continue normally

Blake Ethridge said:

Sounds like we need to get you an updated Avatar :)

Ilmars said:

You can't have all :) If you want to do long delays and don't lose received chars then chose one.

Do all what?! I just used Brian's suggestion! Can you please point what part of the code doe snot make sense?

CaEngineer said:

Can you please point what part of the code doe snot make sense?

JMG already said what you shall do in case you don't want to use ISR's

Ilmars said:

JMG already said what you shall do in case you don't want to use ISR's

OK! Thanks for your hint!

Again tested the following code, but the Hyper Terminal does not let me transfer anything through the RS232. How is this possible to send from Hyper to the Display like the following video? I need more help please!

https://www.youtube.com/watch?v=As6VdtigvH4

The Hyper is set as follows.

The code as you guys recommended me is as follows.

#include <msp430f47197.h> 


void DelayMs(int Ms); 
void LCD_cmd(unsigned char cmd); 
void LCD_dat(unsigned char byte); 
void LCD_init(void); 


int main(void) 
{ 
  volatile unsigned int i; 
   WDTCTL = WDTPW+WDTHOLD;           // Stop WDT 
   
  //*** LCD Setups 
      P7DIR = 0x00; 
      P8OUT = 0;    // Before set DIR 
      P8DIR = 0x07; //P8.2=>EN, P8.1=>RW, P8.0=>RS 
       
      DelayMs(100);  
      LCD_init(); 
      DelayMs(50); 
       
       
    // **** Serial communication SETTING
  
  P1SEL |=0x3F;                       // P1.6,7 = USCI_A0 RXD/TXD 
   P1DIR  |= 0x3F  ; // Set as Input 
  UCA1CTL1 |= UCSSEL_2;                   // SMCLK 
  UCA1BR0 = 0x03;                   // 1MHz 115200  --09 and 03 for 9600 // Setting Baud rate  
  UCA1BR1 = 0x00;                 // 1MHz 115200 
  UCA1MCTL = 0x02;                      // Modulation 
  UCA1CTL1 &= ~UCSWRST;                     // 


   while(1)  // infinite loop looking for a received message from the PC
{ 
if(IFG2&UCA1RXIFG)   // If we receive a message from PC!
  { 
  char MES = UCA1RXBUF;                 // Put it into the buffer
  LCD_cmd(0x80); 
  LCD_dat(MES);                  // Show it on the display
   
  __delay_cycles(8388608);                          // wait for a while 
  DelayMs(100);  // OR USING  __delay_cycles() 
  LCD_init();                                  // Clear th edisplay
  DelayMs(50); 
  __delay_cycles(8388608); 
  IFG2 &= ~UCA1TXIFG;         // Change the flag
   
  } 
}
 
}






 
void LCD_cmd(unsigned char cmd) 
{ 
  P8OUT &= 0xF8; // EN=0,RW=0,RS=0 
  P7OUT = cmd; 
   
  P8OUT |= 0x04; // En = 1; 
  P7DIR = 0xFF;  // out P7 
   
  __delay_cycles(2); //Insurance 
   
  P8OUT &= 0xFB; // En = 0; 
  P7DIR = 0;     // float P7 
   
   
  DelayMs(3); 
} 
void LCD_dat(unsigned char byte) 
{ 
  P8OUT &= 0xF8; // EN=0,RW=0,RS=0 
  P8OUT |= 0x01; // change to RS=1 
  P7DIR = 0xFF;  // P7 out 
   
  P7OUT = byte; 
   
  P8OUT |= 0x04; // EN=1 
   
   
  __delay_cycles(2); // Insurance 
   
  P8OUT &= 0xFB;  // EN=0 
  P7DIR = 0;     //  P7 float 
   
   
  DelayMs(3); 
} 
void LCD_init(void) 
{ 
  LCD_cmd(0x38);    // 8-bit mode 5*8 dots ==> Page 17 of the LCD datasheet: Function Set 
  LCD_cmd(0x0C);    // Blinking display   ==> Page 16 of the LCD datasheet: Display ON/OFF Control 
  LCD_cmd(0x06);    // Blinking cursor   ==> Page 15 of NMTC-S0802XRGHS: Entry Mode Set 
  LCD_cmd(0x01);    // Clear Display ==> Page 15 of the LCD 
} 
void DelayMs(int Ms) 
{ 
  volatile int i; // To counteract "optimization" 
  while(Ms>0) 
  { 
    for(i=0;i<100;i++); 
    Ms--; 
  } 
}

CaEngineer said:

Hyper Terminal does not let me transfer anything through the RS232

It's not the Hyper Terminal for sure. It's your code. First thing you shall learn: echo characters on HyperTerminal without any LCD. When your UART code works - only then add LCD.

CaEngineer said:

  char MES = UCA1RXBUF;                 // Put it into the buffer

  LCD_cmd(0x80);

  LCD_dat(MES);                  // Show it on the display

   

  __delay_cycles(8388608);                          // wait for a while

  DelayMs(100);  // OR USING  __delay_cycles()

  LCD_init();                                  // Clear th edisplay

  DelayMs(50);

  __delay_cycles(8388608);

  IFG2 &= ~UCA1TXIFG;         // Change the flag

This is very funny code you are running in the ISR. It seems that you don't listen at all what others are teaching you. Do you have any programming courses around you can attend?

Ilmars said:

This is very funny code you are running in the ISR. It seems that you don't listen at all what others are teaching you. Do you have any programming courses around you can attend?

You better learn how to behave people here. Rather than make fun of them! Learn how people like Gens-Michael teach and treat people ...humble and helpful. Not like you rude and disrespectful!

If you need more point to get contribution prize in TI Forum go talk to them to grant you with ... don't reply to my posts. I do not need your misleads and your DADY advise! I had enough course. You better learn how to treat people and respect them! You need a course to teach you how to behave people!

I have had enough of your sarcasms!

CaEngineer said:

I have had enough of your sarcasms!

You once more confirm what I said by NOT paying attention to part of my reply where I actually teach you:

Ilmars said:

First thing you shall learn: echo characters on HyperTerminal without any LCD. When your UART code works - only then add LCD.

About funny code: rather than (again) going into pointless discussion about my behavior you would rather ask question about why I said what I said. I would comment your questions.

As a teacher I am highly disappointed seeing ISR code where you put delay of 8 million CPU cycles disregarding advice all around this forum- run ISR code as fast as you can.

You don't only put long delays in your ISR, you write character on LCD and after that - clear it! This is not good if I can say so.

Not to mention that following code clearly shows that you don't pay attention to what you actually write in your code or simply don't care:

Ilmars said:

  DelayMs(50);

  __delay_cycles(8388608);

Other people are donating their time to help you, in return you shall show some respect - show that you are trying. Show that you care and you are working hard by posting clean code, showing that you debugged it, tested. - Then you will get much better responses. Of course, often problems are not msp430-related, but if you repeatedly come with questions which are out of forum scope in form of "this code does not work - take look and fix it", then readers of your posts could become... how to say... disappointed.

That's why I said what I said :)

I got the echo from MSP to my Terminal on PC but the opposite does not work!

I used while as follows.

Any idea?

#include  <msp430f47197.h>

char MES[];
//const char numbers[]={0x30,0x31,0x32,0x33,0x34,0x35,0x36,0x37,0x38,0x39};

void LCD_cmd(unsigned char cmd);
void LCD_dat(unsigned char byte);
void LCD_init(void);
void DelayMs(int Ms);


void delay(unsigned int ms)
{
unsigned int i, j;


    //*** LCD Setups
      P7DIR = 0x00; // Redundant
      P8OUT = 0;    // Before set DIR
      P8DIR = 0x07; //P8.2=>EN, P8.1=>RW, P8.0=>RS
      
      DelayMs(100);  // OR USING  __delay_cycles()
      LCD_init();
      DelayMs(50);
      
      
      

for (i = 0; i <= ms; i++)
{
for (j = 0; j<=255; j++);
}
}

void main(void)
{

  WDTCTL = WDTPW + WDTHOLD;                 // Stop WDT
 
  P1SEL |= 0xC0;                            // P1.6,7 = USART0 TXD/RXD
  
  UCA1CTL1 |= UCSSEL_1;                     // CLK = ACLK
  UCA1BR0 = 0x03;                           // 32k/9600 - 3.41
  UCA1BR1 = 0x00;                           //
  UCA1MCTL = 0x06;                          // Modulation
  UCA1CTL1 &= ~UCSWRST;                     // **Initialize USCI state machine**

  
  while (1)
  {
    
     for (int i =0 ; i <=7; i++)
     {
      LCD_cmd(0xc0);
      LCD_dat(MES[i]);
  }
      while(!(IFG2&UCA1RXIFG));
       delay(100);
     char MES = UCA1RXBUF ;
      delay(100);
      
    
    
 
}
//  _BIS_SR(LPM3_bits + GIE);                 // Enter LPM3 w/ interrupt - does not reach here
}



void LCD_cmd(unsigned char cmd)
{
  P8OUT &= 0xF8; // EN=0,RW=0,RS=0
  P7OUT = cmd;
  
  P8OUT |= 0x04; // En = 1;
  P7DIR = 0xFF;  // out P7
  
  __delay_cycles(2); //Insurance
  
  P8OUT &= 0xFB; // En = 0;
  P7DIR = 0;     // float P7
  
  
  DelayMs(3);
}


void LCD_dat(unsigned char byte)
{
  P8OUT &= 0xF8; // EN=0,RW=0,RS=0
  P8OUT |= 0x01; // change to RS=1
  P7DIR = 0xFF;  // P7 out
  
  P7OUT = byte;
  
  P8OUT |= 0x04; // EN=1
  
  
  __delay_cycles(2); // Insurance
  
  P8OUT &= 0xFB;  // EN=0
  P7DIR = 0;     //  P7 float
  
  
  DelayMs(3);
}



void LCD_init(void)
{
  LCD_cmd(0x38);    // 8-bit mode 5*8 dots ==> Page 17 of the LCD datasheet: Function Set 
  LCD_cmd(0x0C);    // Blinking display   ==> Page 16 of the LCD datasheet: Display ON/OFF Control
  LCD_cmd(0x06);    // Blinking cursor   ==> Page 15 of NMTC-S0802XRGHS: Entry Mode Set 
  LCD_cmd(0x01);    // Clear Display ==> Page 15 of the LCD 
}


void DelayMs(int Ms)
{
  volatile int i; // To counteract "optimization"
  while(Ms>0)
  {
    for(i=0;i<100;i++);
    Ms--;
  }
}

Finally got this to work. But I would like to know how to send a string and show it on the display instead of one character at a time. The terminal shows the character repeatably! How can I stop them?

#include  <msp430f47197.h>


void LCD_cmd(unsigned char cmd);
void LCD_dat(unsigned char byte);
void LCD_init(void);
void DelayMs(int Ms);



void main(void)
{

  
  WDTCTL = WDTPW + WDTHOLD;                 // Stop WDT
 
      //*** LCD Setups
      P7DIR = 0x00; // Redundant
      P8OUT = 0;    // Before set DIR
      P8DIR = 0x07; //P8.2=>EN, P8.1=>RW, P8.0=>RS
      
      DelayMs(100);  // OR USING  __delay_cycles()
      LCD_init();
      DelayMs(50);

      
  P1SEL |= 0xC0;                            // P1.6,7 = USART0 TXD/RXD

  
  UCA1CTL1 |= UCSSEL_1;                  // CLK = ACLK
  UCA1BR0 = 0x03;                           // 32k/9600 - 3.41
  UCA1BR1 = 0x00;                           //
  UCA1MCTL = 0x06;                          // Modulation
  UCA1CTL1 &= ~UCSWRST;                     // **Initialize USCI state machine**
  
  while(1)
  {

     if(!(IFG2&UCA1RXIFG)  )
      {
     
    UCA1TXBUF = UCA1RXBUF ;
      DelayMs(100);
      

        
      LCD_cmd(0x80);
      LCD_dat(UCA1RXBUF);
   
   
  
   }
  
  }

}



void LCD_cmd(unsigned char cmd)
{
  P8OUT &= 0xF8; // EN=0,RW=0,RS=0
  P7OUT = cmd;
  
  P8OUT |= 0x04; // En = 1;
  P7DIR = 0xFF;  // out P7
  
  __delay_cycles(2); //Insurance
  
  P8OUT &= 0xFB; // En = 0;
  P7DIR = 0;     // float P7
  
  
  DelayMs(3);
}


void LCD_dat(unsigned char byte)
{
  P8OUT &= 0xF8; // EN=0,RW=0,RS=0
  P8OUT |= 0x01; // change to RS=1
  P7DIR = 0xFF;  // P7 out
  
  P7OUT = byte;
  
  P8OUT |= 0x04; // EN=1
  
  
  __delay_cycles(2); // Insurance
  
  P8OUT &= 0xFB;  // EN=0
  P7DIR = 0;     //  P7 float
  
  
  DelayMs(3);
}



void LCD_init(void)
{
  LCD_cmd(0x38);    // 8-bit mode 5*8 dots ==> Page 17 of the LCD datasheet: Function Set 
  LCD_cmd(0x0C);    // Blinking display   ==> Page 16 of the LCD datasheet: Display ON/OFF Control
  LCD_cmd(0x06);    // Blinking cursor   ==> Page 15 of NMTC-S0802XRGHS: Entry Mode Set 
  LCD_cmd(0x01);    // Clear Display ==> Page 15 of the LCD 
}


void DelayMs(int Ms)
{
  volatile int i; // To counteract "optimization"
  while(Ms>0)
  {
    for(i=0;i<100;i++);
    Ms--;
  }
}

CaEngineer said:

If you need more point to get contribution prize in TI Forum

Soon you could get TI prize as forum user who earned most points by contributing just questions.

CaEngineer said:

don't reply to my posts.

Thank you - I know. Apologies for this one ;)