'cst816s: failed reading chip id' with Waveshare 1.69" Touch LCD

[ljunquera]

I am getting an error message when trying to use a nRF5340 DK with a Waveshare 1.69inch Touch LCD Module on the Zephyr sample code with minimal modifications.

All the demo source files for other platforms (Arduino, ESP32, Pico, Raspberry Pi, and STM32.) are here https://files.waveshare.com/wiki/1.69inch-Touch-LCD-Module/1.69inch_Touch_LCD_Demo.zip

The error when I flash/reset is:

[00:00:01.167,388] <err> cst816s: failed reading chip id

When I touch the screen I get an error message:

[01:14:39.267,944] <err> cst816s: Could not read x data

I contacted the manufacturer support and they confirmed it was a CST816S touch driver,

Unable to read, most likely entered the sleep state, when initializing the touch screen, you need to turn off the sleep state, otherwise each call, you need to wake up, otherwise the touch screen will enter the sleep mode, the method to disable the sleep mode is: first reset, after the reset is completed, write a non-zero value to the 0xFE register of the slave address 0x15, which can be 0x01 or 0x07 as shown below:

The code looked was:

UBYTE Touch_1IN69_init(UBYTE mode)
{
	UBYTE bRet, Rev;
	
	Touch_1IN69_Reset();
	Touch_1IN69_Stop_Sleep();
}

void Touch_1IN69_Reset()
{
	DEV_Digital_Write(DEV_I2C_RST,0);
	DEV_Delay_ms(100);
	DEV_Digital_Write(DEV_I2C_RST,1);
	DEV_Delay_ms(100);
}

Void Touch_1IN69_Stop_Sleep()
{
	DEV_I2C_Write_Byte(0x15, 0xFE, 0x07);
}

I see the input_cst816s error code, but can't tell if this is how it's implemented

I'm trying to determine if it's a configuration problem or if it is a driver issue.

In case it is relevant my overlay is:

&pinctrl {

    spi4_default: spi4_default {
        group1 {
            psels = <NRF_PSEL(SPIM_SCK, 0, 6)>,
                <NRF_PSEL(SPIM_MOSI, 0, 25)>;
        };
    };

    spi4_sleep: spi4_sleep {
        group1 {
            psels = <NRF_PSEL(SPIM_SCK, 0, 6)>,
                <NRF_PSEL(SPIM_MOSI, 0, 25)>;
            low-power-enable;
        };
    };

    // Set to I2C pins that the touch screen is connected to
    i2c2_default: i2c2_default {
        group1 {
            psels = <NRF_PSEL(TWIM_SDA, 1, 3)>,
                <NRF_PSEL(TWIM_SCL, 1, 2)>;
        };
    };

    i2c2_sleep: i2c2_sleep {
        group1 {
            psels = <NRF_PSEL(TWIM_SDA, 1, 3)>,
                <NRF_PSEL(TWIM_SCL, 1, 2)>;
            low-power-enable;
        };
    };
};

&spi4 {
	compatible = "nordic,nrf-spim";
	status = "okay";
	pinctrl-0 = <&spi4_default>;
	pinctrl-1 = <&spi4_sleep>;
    pinctrl-names = "default", "sleep";
	cs-gpios = <&gpio0 11 GPIO_ACTIVE_LOW>;

    st7789v_st7789v_tl019fqv01: st7789v@0 {
        compatible = "sitronix,st7789v";
        spi-max-frequency = <20000000>;
        reg = <0>;
        cmd-data-gpios = < &gpio1 11 GPIO_ACTIVE_LOW>;
        reset-gpios = < &gpio1 10 GPIO_ACTIVE_LOW>;
        width = <240>;
        height = <280>;
        x-offset = <0>;
        y-offset = <20>;
        vcom = <0x19>;
        gctrl = <0x35>;
        vrhs = <0x12>;
        vdvs = <0x20>;
        mdac = <0x00>;
        gamma = <0x01>;
        colmod = <0x05>;
        lcm = <0x2c>;
        porch-param = [0c 0c 00 33 33];
        cmd2en-param = [5a 69 02 01];
        pwctrl1-param = [a4 a1];
        pvgam-param = [D0 04 0D 11 13 2B 3F 54 4C 18 0D 0B 1F 23];
        nvgam-param = [D0 04 0C 11 13 2C 3F 44 51 2F 1F 1F 20 23];
        ram-param = [00 F0];
        rgb-param = [CD 08 14];
	};
};


&i2c3 {
    compatible = "nordic,nrf-twim";
    status = "okay";
    clock-frequency = <I2C_BITRATE_FAST>;
    pinctrl-0 = <&i2c2_default>;
    pinctrl-1 = <&i2c2_sleep>;
    pinctrl-names = "default", "sleep";

    cst816s: cst816s@15 {
        compatible = "hynitron,cst816s";
        reg = <0x15>;
        irq-gpios = <&gpio1 1 GPIO_ACTIVE_LOW>;
        rst-gpios = <&gpio0 20 GPIO_ACTIVE_LOW>;
    };
};

/ {
	fstab {
		compatible = "zephyr,fstab";
		lfs: lfs {
			compatible = "zephyr,fstab,littlefs";
			mount-point = "/lfs";
			partition = <&littlefs_storage>;
			automount;
            read-size = <16>;
			prog-size = <16>;
			cache-size = <64>;
			lookahead-size = <32>;
			block-cycles = <512>;
		};
	};
};

/delete-node/ &storage_partition;

&mx25r64 {
	partitions {
		compatible = "fixed-partitions";
		#address-cells = <1>;
		#size-cells = <1>;

		littlefs_storage: partition@0 {
			label = "littlefs_storage";
			reg = <0x00000000 0x00200000>;
		};
		lvgl_raw_partition: partition@200000 {
			label = "lvgl_raw_partition";
			reg = <0x00200000 0x00200000>;
		};
        settings_partition: partition@400000 {
            label = "settings_partition";
            reg = <0x400000 0x100000 >;
        };
	};
};

/ {
    chosen {
        nordic,pm-ext-flash = &mx25r64;
        zephyr,display = &st7789v_st7789v_tl019fqv01;
        zephyr,keyboard-scan = &cst816s;
    };

    /* 
    aliases {
        vibrator_pwm = &led0;
    };
    */
    lvgl_pointer_input: lvgl_pointer {
        compatible = "zephyr,lvgl-pointer-input";
        input = <&cst816s>;
        swap-xy;
        invert-x;
    };

    aliases {
        input = &cst816s;
    };

    vib_pwr: vib-pwr-ctrl {
        compatible = "regulator-fixed";
        regulator-name = "vib-pwr-ctrl";
        enable-gpios = <&gpio1 14 GPIO_ACTIVE_HIGH>;
    };
};

And the prj.conf is:

CONFIG_LV_Z_MEM_POOL_SIZE=16384
CONFIG_LV_Z_SHELL=y
CONFIG_MAIN_STACK_SIZE=2048

CONFIG_DISPLAY=y
CONFIG_DISPLAY_LOG_LEVEL_ERR=y
CONFIG_DISPLAY_LOG_LEVEL_INF=y

CONFIG_LOG=y
CONFIG_SHELL=y


CONFIG_LVGL=y
CONFIG_LV_MEM_CUSTOM=y
CONFIG_LV_USE_LOG=y
CONFIG_LV_USE_LABEL=y
CONFIG_LV_USE_BTN=y
CONFIG_LV_USE_ARC=y
CONFIG_LV_USE_IMG=y
CONFIG_LV_USE_MONKEY=y
CONFIG_LV_FONT_MONTSERRAT_16=y


#added to get touch working
CONFIG_INPUT=y
CONFIG_LV_Z_POINTER_INPUT=y
#CONFIG_LV_Z_ENCODER_INPUT=y

I received some diagrams from a manufacturer on the other chips: