Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ST7789 - maximum height and width? #142

Open
SimUKdev opened this issue Apr 29, 2021 · 9 comments
Open

ST7789 - maximum height and width? #142

SimUKdev opened this issue Apr 29, 2021 · 9 comments

Comments

@SimUKdev
Copy link

Type of Raspberry Pi

Raspberry Pi 3 A+

Linux Kernel version

Linux pi3aplus-5 5.10.17-v7+ #1403 SMP Mon Feb 22 11:29:51 GMT 2021 armv7l GNU/Linux

Expected behaviour

Expecting the display to show full screen across all 240x320 pixels on the ST7789 2.0 inch display

Actual behaviour

st7789_240x320_lumalcd_partialscreensize

(Note: In the photo only, colours are slightly over-bright and washed out as I've brightened it up to show the hardware and silkscreen more clearly to clarify exactly what sort of device I'm using. The output that is showing correctly, looks correct as per other ST77xx displays)

Only getting the pixels correctly displaying on the top two thirds of the display, with "scrambled data/corruption" showing in the lower third. I have two of these displays, from the same supplier, and both are doing the same, so I think I can rule out the individual display being at "fault".

I've tried adjusting the height and width (swapping 320 and 240 between height and width), also tried rotate which only rotated within the working two-thirds of the screen.

When I run any demo from luma.examples, I note that it shows "240 x 240" for the Dimensions line, despite having an expected output of 240x320 (or 320x240)

python3 colors.py  --conf ../conf/st7789_240x320.conf
Version: luma.lcd 2.9.0 (luma.core 2.3.1)
Display: st7789
Interface: spi
Dimensions: 240 x 240

My st7789_240x320.conf file contains:

--display=st7789
--interface=spi
--spi-bus-speed=48000000
--gpio-reset=24
--gpio-chip-select=8
--gpio-data-command=23
--gpio-backlight=18
--width=240
--height=320
--backlight-active=high

As noted, I've tried --width=320 and --height=240 but that hasn't made any difference to my issue. Also may be worth noting that the content is centring in the middle of the 240x240 size rather than fitting to a 320 sized width or height.

The demos appear to output a Dimensions: 240 x 240 line with the ST7789 no matter what sizes I specify, even far smaller than the actual display's correct sizes.

I've made two other custom .conf files for ST7735 (Open-Smart 160x80) and ST7789 (Open-Smart 240x240) screens this past week (which I plan to submit as PRs once I've worked through my other various ST77xx based displays to confirm they are working with luma.lcd), so feel I am correctly using the conf file and settings.

Is there a maximum height and width for ST7789 displays?

Because the luma.example demo files output a 240 x 240 from the demo_opts even though I'm setting a higher resolution, it makes me wonder if there is a hard-cap being set on the maximum resolution of the display by the repo, possibly even a fixed resolution for the ST7789, so I think my overall question is if there is a maximum height and width for the ST7789 display?

@SimUKdev
Copy link
Author

SimUKdev commented May 2, 2021

Interesting... I appear to have made some progress with this:

st7789_240x320_lumalcd_fullscreen_02may2021

I changed two lines of code in https://github.com/rm-hull/luma.lcd/blob/master/luma/lcd/device.py#L301-L303

    def __init__(self, serial_interface=None, rotate=0, **kwargs):
        super(st7789, self).__init__(luma.lcd.const.st7789, serial_interface, **kwargs)
        self.capabilities(240, 240, rotate, mode="RGB")

to become:

    def __init__(self, serial_interface=None, width=240, height=240, rotate=0, **kwargs):
        super(st7789, self).__init__(luma.lcd.const.st7789, serial_interface, **kwargs)
        self.capabilities(width, height, rotate, mode="RGB")

And it allows my 240x320 ST7789 display to show full screen. It doesn't appear, from looking at various demos in luma.examples, to be stretching the pixels across the screen, as circles appear as far as I can tell, to still be circles.

The examples also output the correct dimensions as specified by the .conf file with those above changes:

python3 colors.py --conf ../conf/st7789_240x320.conf
Version: luma.lcd 2.9.0 (luma.core 2.3.1)
Display: st7789
Interface: spi
Dimensions: 240 x 320

However further down at https://github.com/rm-hull/luma.lcd/blob/master/luma/lcd/device.py#L336-L341 there are some hardcoded references to 240 for height and width.

    def display(self, image):
        w, h = 240, 240
        self.set_window(0, 0, w, h)

        image = self.preprocess(image)
        self.data(list(image.convert("RGB").tobytes()))

If I change those, such as making the second value (h) to be 320, I simply get a black display. Leaving them as 240 each allows my 240x320 ST7789 display to work. I don't understand what that is doing.

I've not yet tried the modified code with a 240x240 ST7789 display, but shall give that a try later. I also note that in the class st7789(backlit_device): code there doesn't appear to be any control of the h-offset or v-offset unlike the class st7735(backlit_device, __framebuffer_mixin):, but maybe the ST7789 doesn't have the same issue of where the display is homed to...?

@SimUKdev
Copy link
Author

SimUKdev commented May 2, 2021

Checked with an Open-Smart 240x240 ST7789 and that is still working correctly with the above code changes.

I've seen there are also 240x135 sized ST7789 displays available, such as:

I don't currently have any 240x135 resolution displays to test with though.

@rm-hull
Copy link
Owner

rm-hull commented May 2, 2021

@matemaciek are you able to help here .. could we change the implementation to not have hard coded width and height please?

@matemaciek
Copy link
Contributor

I'm definitely for this change, I was polishing already existing pull request and those hard coded dimensions were itching me, but I didn't have different displays to test other sizes. The more generic code is, the better (-:

@SimUKdev your changes make total sense. As for changes in display(), can you try setting w to 320? Maybe the dimensions are swapped there. I also like the version from st7735 and I wouldn't be surprised if it would work on st7789.

@matemaciek
Copy link
Contributor

I've ordered 135x240 st7789 display, so I will be able to check on it sooner or later also.

@SimUKdev
Copy link
Author

SimUKdev commented May 2, 2021

@matemaciek

As for changes in display(), can you try setting w to 320? Maybe the dimensions are swapped there.

Just tried both 320, 240 and 240, 320 on w, h = 240, 240 line, and only get a blank black screen when using 320 for either w or h.

Interestingly, if I put 120 for both values, the screen still outputs at what appears to be the same 240x320 (or 320x240) resolution. And just out of curiosity I tried 0, 0 as the numbers being defined there, and that still outputs full screen.

@rm-hull rm-hull closed this as completed May 3, 2021
@rm-hull rm-hull reopened this May 3, 2021
@thijstriemstra
Copy link
Collaborator

rm-hull reopened this on May 3

so, is this a bug?

@Frederic98
Copy link
Contributor

I had the same issue with my display (from waveshare). After looking through the code and ST7789 datasheet, it turns out that the arguments to commands should be sent as data bytes, but the library sends it as command bytes. This causes the display to receive a SWRESET command (software reset), because a resolution of 320px is encoded as 0x01 0x3F.

I solved this in pull request #155 by splitting the command and arguments, and forwarding the argument bytes to self.data()

@omnislash772
Copy link

I am also having issues with a device that seems to follow the setup of this product from adafruit https://www.adafruit.com/product/5206
I've also tried modifying the device.py driver file to change the display capabilities to match it being 280x240
But it seems that it will shift my image down and display garbage at the top
PXL_20230406_141009226 putting the number higher only results in this area being filled with whatever was in the top part of the image. (In this case red)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants