Message ID | 20241016213956.4508-1-everestkc@everestkc.com.np (mailing list archive) |
---|---|
State | Accepted |
Commit | 8f66f649846348871148413dcb6af69d9adf760a |
Headers | show |
Series | [V5,next] Bluetooth: btintel_pcie: Remove structurally deadcode | expand |
Context | Check | Description |
---|---|---|
tedd_an/pre-ci_am | success | Success |
tedd_an/CheckPatch | success | CheckPatch PASS |
tedd_an/GitLint | fail | WARNING: I3 - ignore-body-lines: gitlint will be switching from using Python regex 'match' (match beginning) to 'search' (match anywhere) semantics. Please review your ignore-body-lines.regex option accordingly. To remove this warning, set general.regex-style-search=True. More details: https://jorisroovers.github.io/gitlint/configuration/#regex-style-search 14: B3 Line contains hard tab characters (\t): " - Removed Fixes tag as this is a code cleanup" 17: B3 Line contains hard tab characters (\t): " - Removed the link to Coverity Scan report" 21: B2 Line has trailing whitespace: " " |
tedd_an/SubjectPrefix | success | Gitlint PASS |
tedd_an/BuildKernel | success | BuildKernel PASS |
tedd_an/CheckAllWarning | success | CheckAllWarning PASS |
tedd_an/CheckSparse | success | CheckSparse PASS |
Hello: This patch was applied to bluetooth/bluetooth-next.git (master) by Luiz Augusto von Dentz <luiz.von.dentz@intel.com>: On Wed, 16 Oct 2024 15:39:55 -0600 you wrote: > The switch case statement has a default branch. Thus, the return > statement at the end of the function can never be reached. > Fix it by removing the return statement at the end of the > function. > > This issue was reported by Coverity Scan. > > [...] Here is the summary with links: - [V5,next] Bluetooth: btintel_pcie: Remove structurally deadcode https://git.kernel.org/bluetooth/bluetooth-next/c/8f66f6498463 You are awesome, thank you!
diff --git a/drivers/bluetooth/btintel_pcie.c b/drivers/bluetooth/btintel_pcie.c index e4ae8c898dfd..660496e55276 100644 --- a/drivers/bluetooth/btintel_pcie.c +++ b/drivers/bluetooth/btintel_pcie.c @@ -391,7 +391,6 @@ static inline char *btintel_pcie_alivectxt_state2str(u32 alive_intr_ctxt) default: return "unknown"; } - return "null"; } /* This function handles the MSI-X interrupt for gp0 cause (bit 0 in