Mks-Robin TFT LCD V1.0 ILI9328 Support. Is it possible? #15871

3Dangel · 2019-11-11T05:32:12Z

(Sorry, not sure why the pull request is saying merge request...If not correspond, Pls delete or close)

Description

The Marlin bugfix - 2.0 contain 2 supported TFT LCD ILI9413 and st7789v.
it is noted in the comment section that the ILI9328 is not supported.
Would it be possible to support the ILI9328?

In the early stage of the development, jmz52's branch had these two ILI9328 and st7789v supported and the coded for.
after the merge, not sure why the ILI9328 was left out and not been supported.

Benefits

It would enable owners of such STM32's Mian board combo kit to be able to use Marlin 2.

Related Issues

here is jmz52's repository of his code. tested o my MksRobin V2.1 TFT 1.0 and it works.

When using the nowadays' (Nov-10-19) it will compile correctly but I get "booting" on the screen.
I'm able to connect via ponterface and access eeprom's data by M503.

I'm not a programmer, so I'm not able to write codes.
hopping the team could help me out if possible and time allow.

Thanks

https://github.com/jmz52/Marlin/tree/MKS-Robin-Touch-POC

partial code from : Marlin\src\lcd\dogm\u8g_dev_tft_320x240_upscale_from_128x64.cpp

static const uint16_t ili9328_init[] = {
ESC_REG(0x0001), 0x0100,
ESC_REG(0x0002), 0x0400,
ESC_REG(0x0003), 0x1038,
ESC_REG(0x0004), 0x0000,
ESC_REG(0x0008), 0x0202,
ESC_REG(0x0009), 0x0000,
ESC_REG(0x000A), 0x0000,
ESC_REG(0x000C), 0x0000,
ESC_REG(0x000D), 0x0000,
ESC_REG(0x000F), 0x0000,
ESC_REG(0x0010), 0x0000,
ESC_REG(0x0011), 0x0007,
ESC_REG(0x0012), 0x0000,
ESC_REG(0x0013), 0x0000,
ESC_REG(0x0007), 0x0001,
ESC_DELAY(200),
ESC_REG(0x0010), 0x1690,
ESC_REG(0x0011), 0x0227,
ESC_DELAY(50),
ESC_REG(0x0012), 0x008C,
ESC_DELAY(50),
ESC_REG(0x0013), 0x1500,
ESC_REG(0x0029), 0x0004,
ESC_REG(0x002B), 0x000D,
ESC_DELAY(50),
ESC_REG(0x0050), 0x0000,
ESC_REG(0x0051), 0x00EF,
ESC_REG(0x0052), 0x0000,
ESC_REG(0x0053), 0x013F,
ESC_REG(0x0020), 0x0000,
ESC_REG(0x0021), 0x0000,
ESC_REG(0x0060), 0x2700,
ESC_REG(0x0061), 0x0001,
ESC_REG(0x006A), 0x0000,
ESC_REG(0x0080), 0x0000,
ESC_REG(0x0081), 0x0000,
ESC_REG(0x0082), 0x0000,
ESC_REG(0x0083), 0x0000,
ESC_REG(0x0084), 0x0000,
ESC_REG(0x0085), 0x0000,
ESC_REG(0x0090), 0x0010,
ESC_REG(0x0092), 0x0600,
ESC_REG(0x0007), 0x0133,
ESC_REG(0x0022),
ESC_END

Fix #8890

[1.1.x] Extra insurance against endstop false positives

Fix message to long for 12864 display

The registers within the Max7219 can get corrupted a number of ways. This allows the Max7219 to be reset and fully initialized. And... it also allows the user to see the initialization pattern.

See #5597 (comment)

According to #9917 (comment)

* Fix planner/stepper race condition Co-Authored-By: ejtagle <[email protected]> Co-Authored-By: AnHardt <[email protected]> * Fix stepper pulse timing Co-Authored-By: ejtagle <[email protected]>

Fixed for cycle used to initialize default values

Co-Authored-By: gloomyandy <[email protected]>

Co-Authored-By: ejtagle <[email protected]>

- Allow planner to alter the deceleration phase of the currently executing block. - Remove BUSY flag, as it is NON ATOMIC to set bits in the Stepper ISR and Planner at the same time. Co-Authored-By: ejtagle <[email protected]>

To allow for the case where a Z endstop (e.g., hall effect) is higher than the probe trigger point, don't check the Z min endstop during probe moves. See #11134

See #12981

@Ludy87

Followup to #13136 by @Ludy87

V3 of the BLTouch has an [Alarm Release & Touch mode](https://docs.wixstatic.com/ugd/f5a1c8_77037f55e5d542309d9fc178165c9f3f.pdf) to activate after the probe is deployed. If this mode isn't activated, the BLTouch keeps deploying, ends up detecting a collision and gets itself in error mode. Tested on a Creality Ender-3.

Counterpart to #13595 Co-Authored-By: swilkens <[email protected]>

Co-Authored-By: InsanityAutomation <[email protected]>

see PR #14857 for details

This reverts commit 73f4027.

Co-Authored-By: thisiskeithb <[email protected]>

Make the configs zip file requirement more prominent

tpruvot · 2019-11-11T07:18:10Z

~~the 9328 is already ok ... in marlin 2.0 branch~~

3Dangel · 2019-11-11T07:52:21Z

Unless I’m doing something wrong. Or the board v2.1 is a bit different ( don’t have the pin out from MKS)
But I don’t believe I did, since flashing the Jmz52’s works and fresh Marlin2 won’t.
Will be uploading my Config files for your review if needed to be.

Thank you all again!!!
.
I’m very sorry that if this is not considered as pull request.
I though since it was “not supported” then I would need to request a pull or feature request.

tpruvot · 2019-11-11T08:18:45Z

cp config/examples/Mks/Robin/Configuration* Marlin/

Please close this PR... tx

The FSMC connector to drive the LCD is the same for all boards

3Dangel · 2019-11-11T13:00:40Z

Closing PR as requested.
Then the initialization is different for The ILI9328.
And in the coding only has 2 LCD’s initialization and none of them is for the ILI9238.

In this case will I need to open bug report to report, right?

tpruvot · 2019-11-11T13:06:49Z

https://drive.google.com/file/d/19zjFjk2XoG93TN7go9lv5q_XIunrDD6u/view?usp=sharing

I know its the v2.4 but the FSMC pins are dedicated on the STM chip... and its a 9328... not 9238

3Dangel · 2019-11-11T13:58:25Z

You are right!
9328
It was very late night. Sorry for that.
Yes, I know the board should work. But different LCD controller wouldn’t need different initialization sequences?
The coding in Marlin only provide support for 2 LCD type initialization.
Hence mine is not in there. That is why I was having trouble and seeking help here.

No, mine does not work yet. :-(

3Dangel · 2019-11-13T02:19:44Z

re-open my request.
after digging into old and jmz52's coding, I found more information that the ILI9328 is not the same as ILI9341.

would it be possible to add codings for the ILI9328 TFT controller?
the main differences is that the initialization is totally different.
my screen stays at booting....

really thank you for your time....
if we cannot support this ILI9328 chipset, please let me know, then I will close my request for good.....
again thank you very much !!!!!

u8g_dev_tft_320x240_upscale_from_128x64.zip

thinkyhead · 2019-11-13T03:26:07Z

You might be able to do what you want. But before we can review your PR you will need to start over and try again. Be sure to start with an up-to-date bugfix-2.0.x before making your changes, and be sure to target bugfix-2.0.x when creating your PR.

See http://marlinfw.org/docs/development/getting_started_pull_requests.html for full instructions.

3Dangel · 2019-11-13T03:45:23Z

Understood SCOTT. Thank you very much. Will do it again. The last download was Nov 11 2019 Will re download it tonight.

…

On Tue, Nov 12, 2019 at 20:26 Scott Lahteine ***@***.***> wrote: You might be able to do what you want. But before we can review your PR you will need to start over and try again. Be sure to start with an up-to-date bugfix-2.0.x before making your changes, and be sure to target bugfix-2.0.x when creating your PR. See http://marlinfw.org/docs/development/getting_started_pull_requests.html for full instructions. — You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub <#15871?email_source=notifications&email_token=AFYWWXIJJMQRR722JGVYV4TQTNXXXA5CNFSM4JLQZGMKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOED4X55Y#issuecomment-553221879>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AFYWWXNLY3R2ZZZIB3SLUSDQTNXXXANCNFSM4JLQZGMA> .

3Dangel · 2019-12-31T16:10:30Z

Just wanted to cross post the solution provided by Jmz52 in the below post.
Confirmed to work beautifully.

#16159

thinkyhead and others added 30 commits June 12, 2018 22:53

Fix coolstep_min_speed / sensorless homing

cc4e515

Fix #8890

Extra insurance against endstop false positives

af1721b

Merge pull request #11012 from thinkyhead/bf1_ensure_endstop

c0e4a7a

[1.1.x] Extra insurance against endstop false positives

[1.1.x] reset bltouch on home (#11010)

e601cfb

Auto Build for Visual Studio Code (#11015)

c28d2e9

Unsigned eeprom mesh addresses

a85f60a

Fix critical section start/end

35023b0

[1.1.x] Tweaks and additions for MAX7219 (#11033)

dc087b1

[1.1.x] Fix Italian msg len (#11057)

7cdaf69

Fix message to long for 12864 display

Tiny improvement to M7219

3ca4f09

The registers within the Max7219 can get corrupted a number of ways. This allows the Max7219 to be reset and fully initialized. And... it also allows the user to see the initialization pattern.

Don't reset stepper timeout for all g-codes

e0b4abd

Use Tool 0 for Dual X Axis G29

3bb332b

See #5597 (comment)

Fix value editing with callback

b05fcad

Edit Z fade as whole numbers

bd10791

add the move_z_after_probing() functionality to UBL's G29 J

7632df6

Merge pull request #11074 from thinkyhead/bf1_value_editing

9478711

Filter endstops state at all times (#11076)

52f5efb

Fix LIN_ADVANCE max_e_jerk_factor

5374544

According to #9917 (comment)

[1.1.x] Fix stepper/planner race condition, Stepper pulse timer (#11084)

dd9c65d

* Fix planner/stepper race condition Co-Authored-By: ejtagle <[email protected]> Co-Authored-By: AnHardt <[email protected]> * Fix stepper pulse timing Co-Authored-By: ejtagle <[email protected]>

Clear Max7219 without test pattern

8600cbb

[1.1.x] Fix advanced pause init values (#11096)

0a2d3ec

Fixed for cycle used to initialize default values

Make max_e_jerk an array to save CPU (#11121)

4d5c655

Co-Authored-By: gloomyandy <[email protected]>

Add minimum step dir delay option, A4988 pulse note (#11122)

04fc715

Co-Authored-By: ejtagle <[email protected]>

Endstop patch followup

e352c1e

Poll all endstops, even when stationary (#11125)

cff2201

Co-Authored-By: ejtagle <[email protected]>

Eliminate stepper.* signed/unsigned warnings

5e03ded

Fix up microsteps conditionals

5788a39

Ignore Z min endstop with separate Z probe

485d50e

To allow for the case where a Z endstop (e.g., hall effect) is higher than the probe trigger point, don't check the Z min endstop during probe moves. See #11134

Fix SKEW_CORRECTION compile error

6747537

thinkyhead and others added 16 commits September 2, 2019 05:38

Remove dead code in get_pid_output, consolidate

6a45d52

See #12981

German patch

9180235

Followup to #13136 by @Ludy87

Fix pin assignment to KILL_PIN in MKS_MINI_12864 (#13397)

cc6d781

Fix missing quotation marks (#13424)

8d9380b

[1.1.x] Fix German language (#13438)

5f783b4

Reduce XY homing feedrate in Creality configs

a5fe0ed

Counterpart to #13595 Co-Authored-By: swilkens <[email protected]>

[1.1.x] BLTOUCH_V3 parity with 2.0.x (#13682)

65ff804

Co-Authored-By: InsanityAutomation <[email protected]>

move PR #14857 Autobuild update into bugfix 1.1.x

fb276f6

see PR #14857 for details

Revert "move PR #14857 Autobuild update into bugfix 1.1.x"

1ab59d6

This reverts commit 73f4027.

[1.1.x] BLTouch 3.0 - 3.1 (#14839)

37bfc5a

[1.1.x] Autobuild formatting (#14858)

e5a4f19

Fix #else extra tokens (#15013)

f3faaed

Fix PT-BR strings (#15023)

909c015

Fix Creality bed thermistor

0110faa

Co-Authored-By: thisiskeithb <[email protected]>

Update bug_report.md

014ec3e

Make the configs zip file requirement more prominent

3Dangel closed this Nov 11, 2019

3Dangel reopened this Nov 13, 2019

thinkyhead closed this Nov 13, 2019

jmz52 mentioned this pull request Dec 31, 2019

Add support for MKS Robin ILI9328-based TFT display #16401

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Mks-Robin TFT LCD V1.0 ILI9328 Support. Is it possible? #15871

Mks-Robin TFT LCD V1.0 ILI9328 Support. Is it possible? #15871

3Dangel commented Nov 11, 2019 •

edited

Loading

tpruvot commented Nov 11, 2019 •

edited

Loading

3Dangel commented Nov 11, 2019

tpruvot commented Nov 11, 2019 •

edited

Loading

3Dangel commented Nov 11, 2019 •

edited

Loading

tpruvot commented Nov 11, 2019 •

edited

Loading

3Dangel commented Nov 11, 2019 •

edited

Loading

3Dangel commented Nov 13, 2019

thinkyhead commented Nov 13, 2019

3Dangel commented Nov 13, 2019 via email

3Dangel commented Dec 31, 2019

Mks-Robin TFT LCD V1.0 ILI9328 Support. Is it possible? #15871

Mks-Robin TFT LCD V1.0 ILI9328 Support. Is it possible? #15871

Conversation

3Dangel commented Nov 11, 2019 • edited Loading

Description

Benefits

Related Issues

tpruvot commented Nov 11, 2019 • edited Loading

3Dangel commented Nov 11, 2019

tpruvot commented Nov 11, 2019 • edited Loading

3Dangel commented Nov 11, 2019 • edited Loading

tpruvot commented Nov 11, 2019 • edited Loading

3Dangel commented Nov 11, 2019 • edited Loading

3Dangel commented Nov 13, 2019

thinkyhead commented Nov 13, 2019

3Dangel commented Nov 13, 2019 via email

3Dangel commented Dec 31, 2019

3Dangel commented Nov 11, 2019 •

edited

Loading

tpruvot commented Nov 11, 2019 •

edited

Loading

tpruvot commented Nov 11, 2019 •

edited

Loading

3Dangel commented Nov 11, 2019 •

edited

Loading

tpruvot commented Nov 11, 2019 •

edited

Loading

3Dangel commented Nov 11, 2019 •

edited

Loading