Skip to content
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.

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

reset: ath79: fix reset sequence for AR9344 #3397

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

nrbffs
Copy link
Contributor

@nrbffs nrbffs commented Dec 17, 2024

According to datasheet, on AR9344 the switch and switch analog need to be reset first before initiating a full reset.

This fixes sporadic reboot issues. Instead of rebooting, the device will just hang. Without this patch, I had failed resets after 5, 20 and 250 tries. With this patch I have more than 2200 successful resets.

Other chips like AR7100 do not have this specific requirement in the datasheet and also the meaning for bits 8 and 12 is slightly different, so only perform this sequence on AR9344.

Fixes #2904

According to datasheet, on AR9344 the switch and switch analog need to
be reset first before initiating a full reset.

This fixes sporadic reboot issues. Instead of rebooting, the device will
just hang. Without this patch, I had failed resets after 5, 20 and 250
tries. With this patch I have more than 2200 successful resets.

Other chips like AR7100 do not have this specific requirement in the
datasheet and also the meaning for bits 8 and 12 is slightly different,
so only perform this sequence on AR9344.

Signed-off-by: Nico Boehr <[email protected]>
@blocktrron
Copy link
Member

This PR belongs upstream. The signed off in the commit message is currently missing, apart from that should be fine.

@rotanid rotanid added the 0. type: bugfix This changeset fixes abug label Dec 18, 2024
@rotanid rotanid added the 2. status: waiting-on-upstream Waiting for upstream changes label Dec 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0. type: bugfix This changeset fixes abug 2. status: waiting-on-upstream Waiting for upstream changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[v2022.1+] TP-Link WDR4300 hangs during reboot
4 participants