ugfx/drivers/gdisp/SSD1322/driver.mk missing

[inmarket]

The divide by 2 when calling xyaddr() is incorrect. It is the xyaddr() macro itself that needs the divide by 2. As indicated by the macro name, it is supposed to take (x,y) as it's parameters. 

[AnSc]

On 5/9/2022 at 10:25 AM, inmarket said:

The divide by 2 when calling xyaddr() is incorrect. It is the xyaddr() macro itself that needs the divide by 2. As indicated by the macro name, it is supposed to take (x,y) as it's parameters. 

Hi,

thanks for this tip. As I wrote it was a crude first try.

I changed the macro to this:

#define xyaddr(x, y)        ((x/2) + (y)*SSD1322_ROW_WIDTH)

and did see the same correct behaviour after reverting the code line to:

ram = RAM(g)+xyaddr(x,y);

[Joel Bodenmann]

Would you be able to provide a diff/patch-file (as an attachment) so we can have a look at your "final" changes to the driver? This way we can more easily review them and decide whether the driver should be modified for the next release.

[AnSc]

Hi,

sorry for the delay, was busy elsewhere…

Please find attached the diff of my changes.

gdisp_lld_SSD1322.diff