Skip to content

Kernel: Kernel_Drivers:Display: Device Tree Node: Added comments about necessity of HPD pin for J721S2 #2

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.

Sign up for GitHub

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

Closed
h-shenoy wants to merge 1 commit into from
Closed

Kernel: Kernel_Drivers:Display: Device Tree Node: Added comments about necessity of HPD pin for J721S2 #2

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions source/linux/Foundational_Components/Kernel/Kernel_Drivers/Display/DSS7.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -362,6 +362,9 @@ Device Tree Node
Documentation for tidss device tree node and its properties can be found in linux kernel device tree bindings in below directory
``Documentation/devicetree/bindings/display/ti/``. Seperate binding files are present for different version of the DSS controller.

.. ifconfig:: CONFIG_part_variant in ('J721S2')

- Pin designated as HPD for DP should not be used for other purpose, irrespective of whether it is connected as HPD or not.
Copy link
Contributor

@nmenon nmenon Oct 28, 2024

Choose a reason for hiding this comment

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

is this covered in TRM? if so, point out chapter. DO NOT make s/w SDK documentation the goto place for careabouts of the SoC.

Copy link
Member

@StaticRocket StaticRocket Oct 31, 2024

Choose a reason for hiding this comment

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

@h-shenoy could you verify this and resolve this conversation if it is actually covered in the TRM?

Copy link
Author

@h-shenoy h-shenoy Oct 31, 2024

Choose a reason for hiding this comment

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

No, I did not find information regarding HPD pin connection status in TRM , let me know if information regarding the can be made part of SDK ?

Copy link
Contributor

@nmenon nmenon Oct 31, 2024

Choose a reason for hiding this comment

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

ideally, you wont document SoC features in S/w documentation - you should file a internal ticket asking it to be documented in TRM and discuss with applications team how to handle it best.

Copy link
Author

@h-shenoy h-shenoy Nov 4, 2024

Choose a reason for hiding this comment

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

@uditkumarti , should this be documented in TRM and not in SDK docs as mentioned in ticket?

Copy link
Member

@StaticRocket StaticRocket Dec 3, 2024

Choose a reason for hiding this comment

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

@uditkumarti

Copy link
Collaborator

@uditkumarti uditkumarti Dec 3, 2024

Choose a reason for hiding this comment

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

needs to be part of TRM . Please reject this PR


Driver Usage
============
Expand Down
Loading