Problem with 1,8" 128x160 TFT Display ST7735 shifted screen #106

karawin · 2018-11-27T18:36:20Z

Some device are ok, some are shifted from one or two pixels in axis x and y
What can be modified in ucg_dev_ic_st7735.c, ucg_dev_tft_128x160_st7735.c to change the offset?
Thanks

sirpete83 · 2018-11-27T20:23:24Z

Hey karawin,
I got the same problem. see #105 . I don't believe that this is a problem with the code or the library. I think it is a problem with the display itself. I ordered a new one from an other distributor. the faulty one I got on aliexpess, only have one.
but I got the same picture like yours.

karawin · 2018-11-27T20:28:56Z

May be, but if i read https://forum.arduino.cc/index.php?topic=487585.0
I think that the soft can be adapted.

sirpete83 · 2018-11-27T21:24:59Z

Yes, but I don't see a solution in this thread.
I think the chip is wrongly positioned on the panel. Pixel 0.0 is outside the panel. The picture is shifted. The panel itself has more pixels than can be addressed. That is perhaps always the case in order to facilitate production. So you have room to apply the chip. but with these chips, the offset just seems to be out of play. These are just my guess. I am all the more pleased when someone has a better education. maybe one that makes the display usable.

karawin · 2018-11-28T09:40:16Z

I think we are in -132 (RGB) x 162
(GM[2:0]= ”000”, DDRAM: 132 x 18-bits x 162) resolution.

sirpete83 · 2018-11-28T14:11:47Z

Hey karawin,
you are right, that modules are in GM 000 Mode, Im working on this! but it can take so time.

karawin · 2018-12-01T17:44:06Z

Modified ucg_dev_ic_st7735.c in CSRC

/*

  ucg_dev_ic_st7735.c
  
  Specific code for the st7735 controller (TFT displays)

  Universal uC Color Graphics Library
  
  Copyright (c) 2014, olikraus@gmail.com
  All rights reserved.

  Redistribution and use in source and binary forms, with or without modification, 
  are permitted provided that the following conditions are met:

  * Redistributions of source code must retain the above copyright notice, this list 
    of conditions and the following disclaimer.
    
  * Redistributions in binary form must reproduce the above copyright notice, this 
    list of conditions and the following disclaimer in the documentation and/or other 
    materials provided with the distribution.

  THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND 
  CONTRIBUTORS "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, 
  INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF 
  MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE 
  DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR 
  CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, 
  SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT 
  NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; 
  LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER 
  CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, 
  STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) 
  ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF 
  ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.  

*/

#include "ucg.h"
#define IX	2
#define IY 1
#define WIDTH	160
#define HEIGHT 	128

const ucg_pgm_uint8_t ucg_st7735_set_pos_seq[] = 
{
  UCG_CS(0),					/* enable chip */
  UCG_C11( 0x036, 0x000),
  UCG_C10(0x02a),	UCG_VARX(0,0x00, 0), UCG_VARX(0,0x0ff, 0), UCG_A2(0x000, 0x07f),					/* set x position */
  UCG_C10(0x02b),	UCG_VARY(0,0x00, 0), UCG_VARY(0,0x0ff, 0), UCG_A2(0x000, 0x09f),		/* set y position */
  UCG_C10(0x02c),							/* write to RAM */
  UCG_DATA(),								/* change to data mode */
  UCG_END()
};


const ucg_pgm_uint8_t ucg_st7735_set_pos_dir0_seq[] = 
{
  UCG_CS(0),					/* enable chip */
  
  /* 0x000 horizontal increment (dir = 0) */
  /* 0x000 vertical increment (dir = 1) */
  /* 0x040 horizontal deccrement (dir = 2) */
  /* 0x080 vertical deccrement (dir = 3) */
  UCG_C11( 0x036, 0x000),
  UCG_C10(0x02a),	UCG_VARX(0,0x00, 0), UCG_VARX(0,0x0ff, 0), UCG_A2(0x000, 0x07f),					/* set x position */
  UCG_C10(0x02b),	UCG_VARY(0,0x00, 0), UCG_VARY(0,0x0ff, 0), UCG_A2(0x000, 0x09f),		/* set y position */

  UCG_C10(0x02c),							/* write to RAM */
  UCG_DATA(),								/* change to data mode */
  UCG_END()
};

const ucg_pgm_uint8_t ucg_st7735_set_pos_dir1_seq[] = 
{
  UCG_CS(0),					/* enable chip */
  /* 0x000 horizontal increment (dir = 0) */
  /* 0x000 vertical increment (dir = 1) */
  /* 0x040 horizontal deccrement (dir = 2) */
  /* 0x080 vertical deccrement (dir = 3) */
  UCG_C11( 0x036, 0x000),
  UCG_C10(0x02a),	UCG_VARX(0,0x00, 0), UCG_VARX(0,0x0ff, 0), UCG_VARX(0,0x00, 0), UCG_VARX(0,0x0ff, 0),					/* set x position */
  UCG_C10(0x02b),	UCG_VARY(0,0x00, 0), UCG_VARY(0,0x0ff, 0), UCG_A2(0x000, 0x09f),		/* set y position */

  UCG_C10(0x02c),							/* write to RAM */
  UCG_DATA(),								/* change to data mode */
  UCG_END()
};

const ucg_pgm_uint8_t ucg_st7735_set_pos_dir2_seq[] = 
{
  UCG_CS(0),					/* enable chip */
  
  /* 0x000 horizontal increment (dir = 0) */
  /* 0x000 vertical increment (dir = 1) */
  /* 0x040 horizontal deccrement (dir = 2) */
  /* 0x080 vertical deccrement (dir = 3) */
  
  UCG_C11( 0x036, 0x040),
  UCG_C11( 0x036, 0x040),			/* it seems that this command needs to be sent twice */
  UCG_C10(0x02a),	UCG_VARX(0,0x00, 0), UCG_VARX(0,0x0ff, 0), UCG_A2(0x000, 0x07f),					/* set x position */
  UCG_C10(0x02b),	UCG_VARY(8,0x00, 0), UCG_VARY(0,0x0ff, 0), UCG_A2(0x000, 0x09f),		/* set y position */

  UCG_C10(0x02c),							/* write to RAM */
  UCG_DATA(),								/* change to data mode */
  UCG_END()
};

const ucg_pgm_uint8_t ucg_st7735_set_pos_dir3_seq[] = 
{
  UCG_CS(0),					/* enable chip */
  
  /* 0x000 horizontal increment (dir = 0) */
  /* 0x000 vertical increment (dir = 1) */
  /* 0x0c0 horizontal deccrement (dir = 2) */
  /* 0x0c0 vertical deccrement (dir = 3) */
  UCG_C11( 0x036, 0x080),
  UCG_C11( 0x036, 0x080),		/* it seems that this command needs to be sent twice */
  UCG_C10(0x02a),	UCG_VARX(0,0x00, 0), UCG_VARX(0,0x0ff, 0), UCG_VARX(0,0x00, 0), UCG_VARX(0,0x0ff, 0),					/* set x position */
  UCG_C10(0x02b),	UCG_VARY(0,0x00, 0), UCG_VARY(0,0x0ff, 0), UCG_A2(0x000, 0x09f),		/* set y position */

  UCG_C10(0x02c),							/* write to RAM */
  UCG_DATA(),								/* change to data mode */
  UCG_END()
};

ucg_int_t ucg_handle_st7735_l90fx(ucg_t *ucg)
{
  uint8_t c[3];
  ucg_int_t tmp;
  if ( ucg_clip_l90fx(ucg) != 0 )
  {
    switch(ucg->arg.dir)
    {
      case 0: 
	ucg_com_SendCmdSeq(ucg, ucg_st7735_set_pos_dir0_seq);	
	break;
      case 1: 
	ucg_com_SendCmdSeq(ucg, ucg_st7735_set_pos_dir1_seq);	
	break;
      case 2: 
	tmp = ucg->arg.pixel.pos.x;
	ucg->arg.pixel.pos.x = 127+-tmp;
	ucg_com_SendCmdSeq(ucg, ucg_st7735_set_pos_dir2_seq);	
	ucg->arg.pixel.pos.x = tmp;
	break;
      case 3: 
      default: 
	tmp = ucg->arg.pixel.pos.y;
	ucg->arg.pixel.pos.y = 159+-tmp;
	ucg_com_SendCmdSeq(ucg, ucg_st7735_set_pos_dir3_seq);	
	ucg->arg.pixel.pos.y = tmp;
	break;
    }
    c[0] = ucg->arg.pixel.rgb.color[0];
    c[1] = ucg->arg.pixel.rgb.color[1];
    c[2] = ucg->arg.pixel.rgb.color[2];
    ucg_com_SendRepeat3Bytes(ucg, ucg->arg.len, c);
    ucg_com_SetCSLineStatus(ucg, 1);		/* disable chip */
    return 1;
  }
  return 0;
}

/*
  L2TC (Glyph Output)
  
*/

/* with CmdDataSequence */ 
ucg_int_t ucg_handle_st7735_l90tc(ucg_t *ucg)
{
  if ( ucg_clip_l90tc(ucg) != 0 )
  {
    uint8_t buf[16];
    ucg_int_t dx, dy;
    ucg_int_t i;
    unsigned char pixmap;
    uint8_t bitcnt;
    ucg_com_SetCSLineStatus(ucg, 0);		/* enable chip */
    ucg_com_SendCmdSeq(ucg, ucg_st7735_set_pos_seq);	

    buf[0] = 0x001;	// change to 0 (cmd mode)
    buf[1] = 0x02a;	// set x
    buf[2] = 0x002;	// change to 1 (arg mode)
    buf[3] = 0x000;	// upper part x
    buf[4] = 0x000;	// no change
    buf[5] = 0x000;	// will be overwritten by x value
    buf[6] = 0x001;	// change to 0 (cmd mode)
    buf[7] = 0x02c;	// write data
    buf[8] = 0x002;	// change to 1 (data mode)
    buf[9] = 0x000;	// red value
    buf[10] = 0x000;	// no change
    buf[11] = 0x000;	// green value
    buf[12] = 0x000;	// no change
    buf[13] = 0x000;	// blue value      
    
    switch(ucg->arg.dir)
    {
      case 0: 
	dx = 1; dy = 0; 
	buf[1] = 0x02a;	// set x
	break;
      case 1: 	
	dx = 0; dy = 1; 
        buf[1] = 0x02b;	// set y
	break;
      case 2: 
	dx = -1; dy = 0; 
        buf[1] = 0x02a;	// set x
	break;
      case 3: 
      default:
	dx = 0; dy = -1; 
        buf[1] = 0x02b;	// set y
	break;
    }
    pixmap = ucg_pgm_read(ucg->arg.bitmap);
    bitcnt = ucg->arg.pixel_skip;
    pixmap <<= bitcnt;
    buf[9] = ucg->arg.pixel.rgb.color[0];
    buf[11] = ucg->arg.pixel.rgb.color[1];
    buf[13] = ucg->arg.pixel.rgb.color[2];
    //ucg_com_SetCSLineStatus(ucg, 0);		/* enable chip */
    
    for( i = 0; i < ucg->arg.len; i++ )
    {
      if ( (pixmap & 128) != 0 )
      {
	if ( (ucg->arg.dir&1) == 0 )
	{
	  buf[5] = ucg->arg.pixel.pos.x;
	}
	else
	{
	  buf[3] = ucg->arg.pixel.pos.y>>8;
	  buf[5] = ucg->arg.pixel.pos.y&255;
	}
	ucg_com_SendCmdDataSequence(ucg, 7, buf, 0);
      }
      pixmap<<=1;
      ucg->arg.pixel.pos.x+=dx;
      ucg->arg.pixel.pos.y+=dy;
      bitcnt++;
      if ( bitcnt >= 8 )
      {
	ucg->arg.bitmap++;
	pixmap = ucg_pgm_read(ucg->arg.bitmap);
	bitcnt = 0;
      }
    }
    ucg_com_SetCSLineStatus(ucg, 1);		/* disable chip */
    return 1;
  }
  return 0;
}


ucg_int_t ucg_handle_st7735_l90se(ucg_t *ucg)
{
  uint8_t i;
  uint8_t c[3];
  ucg_int_t tmp;
  
  /* Setup ccs for l90se. This will be updated by ucg_clip_l90se if required */
  
  for ( i = 0; i < 3; i++ )
  {
    ucg_ccs_init(ucg->arg.ccs_line+i, ucg->arg.rgb[0].color[i], ucg->arg.rgb[1].color[i], ucg->arg.len);
  }
  
  /* check if the line is visible */
  
  if ( ucg_clip_l90se(ucg) != 0 )
  {
    ucg_int_t i;
    switch(ucg->arg.dir)
    {
      case 0: 
	ucg_com_SendCmdSeq(ucg, ucg_st7735_set_pos_dir0_seq);	
	break;
      case 1: 
	ucg_com_SendCmdSeq(ucg, ucg_st7735_set_pos_dir1_seq);	
	break;
      case 2: 
	tmp = ucg->arg.pixel.pos.x;
	ucg->arg.pixel.pos.x = 127-tmp;
	ucg_com_SendCmdSeq(ucg, ucg_st7735_set_pos_dir2_seq);	
	ucg->arg.pixel.pos.x = tmp;
	break;
      case 3: 
      default: 
	tmp = ucg->arg.pixel.pos.y;
	ucg->arg.pixel.pos.y = 159-tmp;
	ucg_com_SendCmdSeq(ucg, ucg_st7735_set_pos_dir3_seq);	
	ucg->arg.pixel.pos.y = tmp;
	break;
    }
    
    for( i = 0; i < ucg->arg.len; i++ )
    {
      c[0] = ucg->arg.ccs_line[0].current;
      c[1] = ucg->arg.ccs_line[1].current; 
      c[2] = ucg->arg.ccs_line[2].current;
      ucg_com_SendRepeat3Bytes(ucg, 1, c);
      ucg_ccs_step(ucg->arg.ccs_line+0);
      ucg_ccs_step(ucg->arg.ccs_line+1);
      ucg_ccs_step(ucg->arg.ccs_line+2);
    }
    ucg_com_SetCSLineStatus(ucg, 1);		/* disable chip */
    return 1;
  }
  return 0;
}


static const ucg_pgm_uint8_t ucg_st7735_power_down_seq[] = {
	UCG_CS(0),					/* enable chip */
	UCG_C10(0x010),				/* sleep in */
	UCG_C10(0x28), 				/* display off */	
	UCG_CS(1),					/* disable chip */
	UCG_END(),					/* end of sequence */
};

ucg_int_t ucg_dev_ic_st7735_18(ucg_t *ucg, ucg_int_t msg, void *data)
{
	ucg->arg.pixel.pos.x += IX;
	ucg->arg.pixel.pos.y += IY;	

  switch(msg)
  {
    case UCG_MSG_DEV_POWER_UP:
      /* setup com interface and provide information on the clock speed */
      /* of the serial and parallel interface. Values are nanoseconds. */
	  	ucg->arg.pixel.pos.x -= IX;
		ucg->arg.pixel.pos.y -= IY;
      return ucg_com_PowerUp(ucg, 100, 66);
    case UCG_MSG_DEV_POWER_DOWN:
      ucg_com_SendCmdSeq(ucg, ucg_st7735_power_down_seq);
	ucg->arg.pixel.pos.x -= IX;
	ucg->arg.pixel.pos.y -= IY;	  
      return 1;
    case UCG_MSG_GET_DIMENSION:
      ((ucg_wh_t *)data)->w = HEIGHT;
      ((ucg_wh_t *)data)->h = WIDTH; //+IY;
	ucg->arg.pixel.pos.x -= IX;
	ucg->arg.pixel.pos.y -= IY;
	return 1;
    case UCG_MSG_DRAW_PIXEL:
      if ( ucg_clip_is_pixel_visible(ucg) !=0 )
      {
	uint8_t c[3];
	ucg_com_SendCmdSeq(ucg, ucg_st7735_set_pos_seq);	
	c[0] = ucg->arg.pixel.rgb.color[0];
	c[1] = ucg->arg.pixel.rgb.color[1];
	c[2] = ucg->arg.pixel.rgb.color[2];
	ucg_com_SendRepeat3Bytes(ucg, 1, c);
	ucg_com_SetCSLineStatus(ucg, 1);		/* disable chip */
      }
	ucg->arg.pixel.pos.x -= IX;
	ucg->arg.pixel.pos.y -= IY;	  
      return 1;
    case UCG_MSG_DRAW_L90FX:
      //ucg_handle_l90fx(ucg, ucg_dev_ic_st7735_18);
      ucg_handle_st7735_l90fx(ucg);
	ucg->arg.pixel.pos.x -= IX;
	ucg->arg.pixel.pos.y -= IY;	  
      return 1;
#ifdef UCG_MSG_DRAW_L90TC
    case UCG_MSG_DRAW_L90TC:
      //ucg_handle_l90tc(ucg, ucg_dev_ic_st7735_18);
      ucg_handle_st7735_l90tc(ucg);
	ucg->arg.pixel.pos.x -= IX;
	ucg->arg.pixel.pos.y -= IY;	  
      return 1;	
#endif /* UCG_MSG_DRAW_L90TC */
#ifdef UCG_MSG_DRAW_L90BF
     case UCG_MSG_DRAW_L90BF:
      ucg_handle_l90bf(ucg, ucg_dev_ic_st7735_18);
	ucg->arg.pixel.pos.x -= IX;
	ucg->arg.pixel.pos.y -= IY;	  
      return 1;
#endif /* UCG_MSG_DRAW_L90BF */
      
    /* msg UCG_MSG_DRAW_L90SE is handled by ucg_dev_default_cb */
    /*
    case UCG_MSG_DRAW_L90SE:
      return ucg->ext_cb(ucg, msg, data);
    */
  }
  	ucg->arg.pixel.pos.x -= IX;
	ucg->arg.pixel.pos.y -= IY;	  
  return ucg_dev_default_cb(ucg, msg, data);  
}

ucg_int_t ucg_ext_st7735_18(ucg_t *ucg, ucg_int_t msg, void *data)
{
	ucg->arg.pixel.pos.x += IX;
	ucg->arg.pixel.pos.y += IY;
  switch(msg)
  {
    case UCG_MSG_DRAW_L90SE:
      //ucg_handle_l90se(ucg, ucg_dev_ic_st7735_18);
      ucg_handle_st7735_l90se(ucg);
      break;
  }
 	ucg->arg.pixel.pos.x -= IX;
	ucg->arg.pixel.pos.y -= IY;	  
 
  return 1;
}

sirpete83 · 2018-12-02T19:42:13Z

Hey,
I also have an solution, I think in yours there are to much OFFSET corrections. can be an workaround for the moment, but i would be only functional for this case.
Maybe I would have the possibility to commit tomorrow.

karawin · 2018-12-02T20:17:00Z

Would be great. My answer is wrong anyway.

sirpete83 · 2018-12-10T19:52:58Z

hey karawin, hey oliver,
in my fork you can find a test branch for the GM00 Mode Display please test it, it works for my display. maybe we can marge it, with some defines to select the "right" mode for the different display around the world.

karawin · 2018-12-10T20:23:05Z

Great, i will test it soon and let you know

karawin · 2018-12-12T18:18:50Z

I took some time to adapt to my code, but it seems to works well.
Congrat!

IAmOrion · 2019-03-25T01:52:51Z

@sirpete83 Just FYI, I just came across this thread after trying out a WaveShare 1.8" 128x160 TFT (https://www.waveshare.com/wiki/1.8inch_LCD_Module) ... I had the same issues as the OP - your "offset_test1_st7735" branch WORKS for me. Thought I'd let you know in case you want to add the Waveshare variant to a list of devices that your fork fixes 👍

a0908896067 · 2020-03-30T04:28:48Z

Some device are ok, some are shifted from one or two pixels in axis x and y
What can be modified in ucg_dev_ic_st7735.c, ucg_dev_tft_128x160_st7735.c to change the offset?
Thanks

Hi, I have the same issue like yours. Where can I download your "offset_test1_st773" ? can you send me the link to the program. Thanks

karawin · 2020-03-31T21:00:59Z

Sources are here
https://github.com/karawin/Ka-Radio32/tree/master/components/ucglib
Integrated in the standat delivery.

arduware · 2022-04-08T10:05:00Z

Hallo - super. Funktioniert bei mir auch. DANKE

karawin mentioned this issue Nov 28, 2018

Ttgo VS1.4 karawin/Ka-Radio32#67

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Problem with 1,8" 128x160 TFT Display ST7735 shifted screen #106

Problem with 1,8" 128x160 TFT Display ST7735 shifted screen #106

karawin commented Nov 27, 2018

sirpete83 commented Nov 27, 2018 •

edited

Loading

karawin commented Nov 27, 2018

sirpete83 commented Nov 27, 2018

karawin commented Nov 28, 2018

sirpete83 commented Nov 28, 2018

karawin commented Dec 1, 2018

sirpete83 commented Dec 2, 2018

karawin commented Dec 2, 2018

sirpete83 commented Dec 10, 2018 •

edited

Loading

karawin commented Dec 10, 2018

karawin commented Dec 12, 2018

IAmOrion commented Mar 25, 2019

a0908896067 commented Mar 30, 2020

karawin commented Mar 31, 2020

arduware commented Apr 8, 2022

Problem with 1,8" 128x160 TFT Display ST7735 shifted screen #106

Problem with 1,8" 128x160 TFT Display ST7735 shifted screen #106

Comments

karawin commented Nov 27, 2018

sirpete83 commented Nov 27, 2018 • edited Loading

karawin commented Nov 27, 2018

sirpete83 commented Nov 27, 2018

karawin commented Nov 28, 2018

sirpete83 commented Nov 28, 2018

karawin commented Dec 1, 2018

sirpete83 commented Dec 2, 2018

karawin commented Dec 2, 2018

sirpete83 commented Dec 10, 2018 • edited Loading

karawin commented Dec 10, 2018

karawin commented Dec 12, 2018

IAmOrion commented Mar 25, 2019

a0908896067 commented Mar 30, 2020

karawin commented Mar 31, 2020

arduware commented Apr 8, 2022

sirpete83 commented Nov 27, 2018 •

edited

Loading

sirpete83 commented Dec 10, 2018 •

edited

Loading