Skip to content

Conversation

@danieldegrasse
Copy link
Contributor

Add rk043fn02h_ct and rk043fn66hs_ctg shields. These shields both describe Rocktech panels available from NXP, and compatible with several NXP iMX RT EVKs using a FPC interface.

Define nodelabel aliases for these FPC interfaces, and update the RT1060 EVK board definition to no longer define the RK043FN02H_CT as a panel at the board level (and use the shield instead)

Support for additional NXP EVKs (RT1064, RT1050, RT1040) will be added in follow up PRs.

@dleach02
Copy link
Member

@DerekSnell can you verify on your HW?

Copy link
Contributor

@DerekSnell DerekSnell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@danieldegrasse and @dleach02 ,
This PR works well on my hardware. I tested with the following combinations:

  • mimxrt1050_evk board with rk043fn02h_ct shield
  • mimxrt1060_evkb board with rk043fn66hs_ctg shield

The doc page has some typos to fix for the build instructions. Otherwise, this looks good to me.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Set ``-DSHIELD=rk044fn02h_ct`` when you invoke ``west build``. For
Set ``-DSHIELD=rk043fn02h_ct`` when you invoke ``west build``. For

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
:shield: rk044fn02h_ct
:shield: rk043fn02h_ct

@danieldegrasse danieldegrasse force-pushed the feature/rk043-display-shields branch from 70b1a9a to 60ddede Compare May 14, 2024 22:09
@danieldegrasse
Copy link
Contributor Author

@danieldegrasse and @dleach02 , This PR works well on my hardware. I tested with the following combinations:

* `mimxrt1050_evk` board with `rk043fn02h_ct` shield

* `mimxrt1060_evkb` board with `rk043fn66hs_ctg` shield

The doc page has some typos to fix for the build instructions. Otherwise, this looks good to me.

Thanks for catching these- both should be fixed now

DerekSnell
DerekSnell previously approved these changes May 15, 2024
dleach02
dleach02 previously approved these changes May 16, 2024
@dleach02
Copy link
Member

@kartben need your approval

@dleach02
Copy link
Member

@kartben please give this attention or reassign it to someone else (me)

@dleach02 dleach02 added the dev-review To be discussed in dev-review meeting label May 30, 2024
@MaureenHelm MaureenHelm self-requested a review May 30, 2024 15:12
@MaureenHelm MaureenHelm removed the dev-review To be discussed in dev-review meeting label May 30, 2024
MaureenHelm
MaureenHelm previously approved these changes May 30, 2024
@danieldegrasse danieldegrasse dismissed stale reviews from MaureenHelm, dleach02, and DerekSnell via c1573f7 May 31, 2024 14:36
@danieldegrasse danieldegrasse force-pushed the feature/rk043-display-shields branch from 60ddede to c1573f7 Compare May 31, 2024 14:36
@zephyrbot zephyrbot added platform: NXP Drivers NXP Semiconductors, drivers area: Devicetree Binding PR modifies or adds a Device Tree binding labels May 31, 2024
@zephyrbot zephyrbot requested review from butok and galak May 31, 2024 14:37
dleach02
dleach02 previously approved these changes Jun 3, 2024
@dleach02
Copy link
Member

dleach02 commented Jun 3, 2024

@kartben need you to review Daniel's change.

@MaureenHelm @DerekSnell please review as your approvals were dropped when @danieldegrasse addressed @kartben change request.

Copy link
Member

@MaureenHelm MaureenHelm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems we're inconsistent about whether the vendor should be followed by a comma or a dash in the gpio header bindings. But it looks strange not to have a comma

Add bindings for nxp display interfaces. These bindings describe the 6
and 40 pin FPC connectors used for displays on several NXP EVKs using
gpio nexus nodes.

Signed-off-by: Daniel DeGrasse <[email protected]>
Define parallel and i2c FPC display interfaces for the RT1060 EVK. These
interfaces are used by multiple displays for the EVK.

Signed-off-by: Daniel DeGrasse <[email protected]>
Add a shield for the RK043FN02H-CT panel, a Rocktech display panel with
an FT5336 touch controller. This panel uses a 40+6 FPC interface for
parallel displays, which is supported by many NXP iMX RT EVKs.

Signed-off-by: Daniel DeGrasse <[email protected]>
Add a shield for the RK043FN66HS-CTG panel, a Rocktech display panel with
an GT911 touch controller. This panel uses a 40+6 FPC interface for
parallel displays, which is supported by many NXP iMX RT EVKs.

Signed-off-by: Daniel DeGrasse <[email protected]>
Remove display definition for the RT1060 EVK. This display panel is now
supported as the RK043FN02H-CT shield. Add this information to the
documentation for the board, along with other tested shields.

Signed-off-by: Daniel DeGrasse <[email protected]>
Add testcases for rk043fn02h_ct and rk043fn66hs_ctg shields to display
and LVGL samples, so these testcases will be built by CI.

Signed-off-by: Daniel DeGrasse <[email protected]>
@danieldegrasse
Copy link
Contributor Author

It seems we're inconsistent about whether the vendor should be followed by a comma or a dash in the gpio header bindings. But it looks strange not to have a comma

Good point- both bindings have been updated to have a comma

@danieldegrasse danieldegrasse requested a review from kartben June 4, 2024 16:01
@aescolar aescolar merged commit 7b207e3 into zephyrproject-rtos:main Jun 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: Devicetree Binding PR modifies or adds a Device Tree binding area: Display area: Samples Samples area: Shields Shields (add-on boards) platform: NXP Drivers NXP Semiconductors, drivers platform: NXP NXP

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants