C3000 deactivate the HSUART DMA #476
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
This PR fixes the issue reported in sonic-net/sonic-buildimage#22555 |
|
Can you add the description you have in the PR into the patch? |
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
paulmenzel
suggested changes
Jun 19, 2025
| Subject: [PATCH] C3000 deactivate the HSUART DMA | ||
|
|
||
| Intel DNV CPU report a err，there is a possibility that the serial port function will be abnormal and the CPU will hang. | ||
| So for DNV CPU, HSUART DMA will be deactivated. |
There was a problem hiding this comment.
- reports and an in “Intel C3000 CPU reports an err,“
- Please paste the error.
- Is there an errata for this issue? Have you reported this upstream?
There was a problem hiding this comment.
- done
- done
- will check if some one has reported it in kernel.org, if not, then will reported it. will update soon.
There was a problem hiding this comment.
- Thank you.
- I do not see the copy-pasted error in the commit description (in the added file).
- Sounds great. I am looking forward to your findings.
There was a problem hiding this comment.
- I have reported the patch revision on kernel.org
https://lore.kernel.org/lkml/[email protected]/
| @@ -0,0 +1,45 @@ | |||
| From b63977b0e1421c641495940fafab9502a2a6d659 Mon Sep 17 00:00:00 2001 | |||
| From: philo <[email protected]> | |||
There was a problem hiding this comment.
Please use your full name.
| From b63977b0e1421c641495940fafab9502a2a6d659 Mon Sep 17 00:00:00 2001 | ||
| From: philo <[email protected]> | ||
| Date: Thu, 17 Apr 2025 02:17:18 -0500 | ||
| Subject: [PATCH] C3000 deactivate the HSUART DMA |
There was a problem hiding this comment.
Please add a prefix as is common in the Linux kernel git archive.
| Date: Thu, 17 Apr 2025 02:17:18 -0500 | ||
| Subject: [PATCH] C3000 deactivate the HSUART DMA | ||
|
|
||
| Intel DNV CPU report a err，there is a possibility that the serial port function will be abnormal and the CPU will hang. |
There was a problem hiding this comment.
The comma looks strange: Use ,.
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
…tty-8250-HSUART-DMA-be-deactivated-for-DNV-CPU.patch
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
paulmenzel
reviewed
Jun 21, 2025
| From b63977b0e1421c641495940fafab9502a2a6d659 Mon Sep 17 00:00:00 2001 | ||
| From: Philo Shao <[email protected]> | ||
| Date: Thu, 17 Apr 2025 02:17:18 -0500 | ||
| Subject: [PATCH] tty/8250: The HSUART DMA be deactivated for the DNV CPU. |
There was a problem hiding this comment.
Remove [PATCH], and use a summary/title in imperative mood, and no dot/period at the end.
tty/8250: Deactivate HSUART for Intel DNV CPU
| Subject: [PATCH] C3000 deactivate the HSUART DMA | ||
|
|
||
| Intel DNV CPU report a err，there is a possibility that the serial port function will be abnormal and the CPU will hang. | ||
| So for DNV CPU, HSUART DMA will be deactivated. |
There was a problem hiding this comment.
- Thank you.
- I do not see the copy-pasted error in the commit description (in the added file).
- Sounds great. I am looking forward to your findings.
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
paulmenzel
reviewed
Jul 21, 2025
| Intel DNV CPU reports an error, indicating that there is a possibility of abnormal serial port functionality and the CPU may hang. | ||
| The HSUART DMA will be deactivated for the DNV CPU. | ||
|
|
||
| Signed-off-by: Philo Shao<[email protected]> |
There was a problem hiding this comment.
Please add a space before the <.
| Subject: [PATCH] C3000 deactivate the HSUART DMA | ||
|
|
||
| Intel DNV CPU report a err，there is a possibility that the serial port function will be abnormal and the CPU will hang. | ||
| So for DNV CPU, HSUART DMA will be deactivated. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Why I did it
Intel C3000 CPU report a err，there is a possibility that the serial port function will be abnormal and the CPU will hang.
Work item tracking
How I did it
C3000 deactivate the HSUART DMA
How to verify it
Which release branch to backport (provide reason below if selected)
Tested branch (Please provide the tested image version)
Description for the changelog
Link to config_db schema for YANG module changes
A picture of a cute animal (not mandatory but encouraged)