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

Indexing error in x axis (SM5166PC Chipset) #434

Closed
Jiwno opened this issue Apr 7, 2023 · 12 comments
Closed

Indexing error in x axis (SM5166PC Chipset) #434

Jiwno opened this issue Apr 7, 2023 · 12 comments

Comments

@Jiwno
Copy link

Jiwno commented Apr 7, 2023

Hi, I have an issue while using the led matrix with the library.

I'm using a 96x48 pixel matrix with SM5166PC chipset and have index error in x axis.

As you can see in the image (0, 0) of the matrix doesn't start from left top corner.

I found that the index of x axis starts with 32-63 and then goes to 0-63.

32 to 63 index are repeated.

Is there any ways to set indexes of the matrix again or any other ways to solve this problem?
스크린샷 2023-04-08 오전 12 32 51

@DarrylStrong
Copy link

Do you have a photo of the rear of the pcb?

@Jiwno
Copy link
Author

Jiwno commented Apr 7, 2023

Do you have a photo of the rear of the pcb?

IMG_2028
Here!!

But when I tested the example code '1_SimpleTestShapes', index seemed fine(after I defined pin e to 32 and I also defined pin e for the plasma example code as well)

Like this.
IMG_2027

@mrcodetastic
Copy link
Owner

So SM5166 compatibility works then? Looks good to me.

@Jiwno
Copy link
Author

Jiwno commented Apr 7, 2023

Yeah, index aligned well on shape test example code but not with the plasma code.

I don't know why but I think it works fine though

@mrcodetastic
Copy link
Owner

It could be a bug with the pattern plasma code to be honest.

@BaiTian6641
Copy link

I have the same issue with 96 x 48 panel and none of example code cannot display properly. However, there is a modify version of ESP32-HUB75-MatrixPanel-DMA library, that library seem work fine, but it only support ESP32.
https://www.waveshare.com/wiki/RGB-Matrix-P2.5-96x48-F#Demo_2

@mrcodetastic
Copy link
Owner

https://www.waveshare.com/wiki/RGB-Matrix-P2.5-96x48-F#Demo_2

Good find. It would have been nice if they could have contributed to this library than make a custom copy.

@BaiTian6641
Copy link

微信图片_20230617004346

So, this is what panel display when I using none modified library. (Example is the BouncingSquares)

@BaiTian6641
Copy link

f8b11884a5fdf1cc771535060417d31

For simple test shape example.

@BaiTian6641
Copy link

I make modify on waveshare's library, using current esp32_i2s_parallel_dma.hpp and esp32_i2s_parallel_dma.cpp. And result is this:
97e25acc074f9c5405c852259e052db
This is SimpleTestShapes example.
e6f40236a45877c3d2dfe0e1c927e1f
This is BouncingSquares example.

@BaiTian6641
Copy link

BaiTian6641 commented Jun 17, 2023

After set the invert_pclk to true in the gdma_lcd_parallel16.hpp, it works on ESP32S3! @mrfaptastic @Jiwno
305d5cac36fac1e74d5bebee7923e98
e2490eb4a7144bd1c65bd6e9d921b36
e65707391fb5c1a8594024cf3815621
1f15217db7d7e5f7330eeecd2e884f2
So, here are some examples. The AuroraDemo example and BouncingSquares example.

Hi, I have an issue while using the led matrix with the library.

I'm using a 96x48 pixel matrix with SM5166PC chipset and have index error in x axis.

As you can see in the image (0, 0) of the matrix doesn't start from left top corner.

I found that the index of x axis starts with 32-63 and then goes to 0-63.

32 to 63 index are repeated.

Is there any ways to set indexes of the matrix again or any other ways to solve this problem? 스크린샷 2023-04-08 오전 12 32 51

mrcodetastic added a commit that referenced this issue Jun 23, 2023
Fix mxconfig.clkphase not being used.

#434
@mrcodetastic
Copy link
Owner

You found a bug that seemingly hasn't been detected for quite some time since I made changes to this library to include ESP32-S3 support.

You shouldn't need to edit the .hpp file to invert the clock (but you had to due toa bug).

You can pass to the mxconfig.clkphase = true; as part of the library config on initialisation.

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

4 participants