drm/amd/display: Remove unnecessary assignments
[WHAT & HOW] TimeForFetchingMetaPTE, TimeForFetchingRowInVBlank and LinesToRequestPrefetchPixelData are local variables. They are freed when CalculatePrefetchSchedule() ends and need not clearing explicitly. This fixes 21 UNUSED_VALUE issues reported by Coverity. Reviewed-by: Nevenko Stupar <nevenko.stupar@amd.com> Signed-off-by: Alex Hung <alex.hung@amd.com> Signed-off-by: Rodrigo Siqueira <rodrigo.siqueira@amd.com> Tested-by: Daniel Wheeler <daniel.wheeler@amd.com> Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
This commit is contained in:
@@ -785,12 +785,9 @@ static bool CalculatePrefetchSchedule(
|
||||
|
||||
if (MyError) {
|
||||
*PrefetchBandwidth = 0;
|
||||
TimeForFetchingMetaPTE = 0;
|
||||
TimeForFetchingRowInVBlank = 0;
|
||||
*DestinationLinesToRequestVMInVBlank = 0;
|
||||
*DestinationLinesToRequestRowInVBlank = 0;
|
||||
*DestinationLinesForPrefetch = 0;
|
||||
LinesToRequestPrefetchPixelData = 0;
|
||||
*VRatioPrefetchY = 0;
|
||||
*VRatioPrefetchC = 0;
|
||||
*RequiredPrefetchPixDataBW = 0;
|
||||
|
||||
@@ -845,12 +845,9 @@ static bool CalculatePrefetchSchedule(
|
||||
|
||||
if (MyError) {
|
||||
*PrefetchBandwidth = 0;
|
||||
TimeForFetchingMetaPTE = 0;
|
||||
TimeForFetchingRowInVBlank = 0;
|
||||
*DestinationLinesToRequestVMInVBlank = 0;
|
||||
*DestinationLinesToRequestRowInVBlank = 0;
|
||||
*DestinationLinesForPrefetch = 0;
|
||||
LinesToRequestPrefetchPixelData = 0;
|
||||
*VRatioPrefetchY = 0;
|
||||
*VRatioPrefetchC = 0;
|
||||
*RequiredPrefetchPixDataBW = 0;
|
||||
|
||||
@@ -1049,12 +1049,9 @@ static bool CalculatePrefetchSchedule(
|
||||
|
||||
if (MyError) {
|
||||
*PrefetchBandwidth = 0;
|
||||
TimeForFetchingMetaPTE = 0;
|
||||
TimeForFetchingRowInVBlank = 0;
|
||||
*DestinationLinesToRequestVMInVBlank = 0;
|
||||
*DestinationLinesToRequestRowInVBlank = 0;
|
||||
*DestinationLinesForPrefetch = 0;
|
||||
LinesToRequestPrefetchPixelData = 0;
|
||||
*VRatioPrefetchY = 0;
|
||||
*VRatioPrefetchC = 0;
|
||||
*RequiredPrefetchPixDataBWLuma = 0;
|
||||
|
||||
@@ -1280,12 +1280,9 @@ static bool CalculatePrefetchSchedule(
|
||||
|
||||
if (MyError) {
|
||||
*PrefetchBandwidth = 0;
|
||||
TimeForFetchingMetaPTE = 0;
|
||||
TimeForFetchingRowInVBlank = 0;
|
||||
*DestinationLinesToRequestVMInVBlank = 0;
|
||||
*DestinationLinesToRequestRowInVBlank = 0;
|
||||
*DestinationLinesForPrefetch = 0;
|
||||
LinesToRequestPrefetchPixelData = 0;
|
||||
*VRatioPrefetchY = 0;
|
||||
*VRatioPrefetchC = 0;
|
||||
*RequiredPrefetchPixDataBWLuma = 0;
|
||||
|
||||
@@ -1444,12 +1444,9 @@ static bool CalculatePrefetchSchedule(
|
||||
|
||||
if (MyError) {
|
||||
*PrefetchBandwidth = 0;
|
||||
TimeForFetchingMetaPTE = 0;
|
||||
TimeForFetchingRowInVBlank = 0;
|
||||
*DestinationLinesToRequestVMInVBlank = 0;
|
||||
*DestinationLinesToRequestRowInVBlank = 0;
|
||||
*DestinationLinesForPrefetch = 0;
|
||||
LinesToRequestPrefetchPixelData = 0;
|
||||
*VRatioPrefetchY = 0;
|
||||
*VRatioPrefetchC = 0;
|
||||
*RequiredPrefetchPixDataBWLuma = 0;
|
||||
|
||||
@@ -1461,12 +1461,9 @@ static bool CalculatePrefetchSchedule(
|
||||
|
||||
if (MyError) {
|
||||
*PrefetchBandwidth = 0;
|
||||
TimeForFetchingMetaPTE = 0;
|
||||
TimeForFetchingRowInVBlank = 0;
|
||||
*DestinationLinesToRequestVMInVBlank = 0;
|
||||
*DestinationLinesToRequestRowInVBlank = 0;
|
||||
*DestinationLinesForPrefetch = 0;
|
||||
LinesToRequestPrefetchPixelData = 0;
|
||||
*VRatioPrefetchY = 0;
|
||||
*VRatioPrefetchC = 0;
|
||||
*RequiredPrefetchPixDataBWLuma = 0;
|
||||
|
||||
@@ -4097,12 +4097,9 @@ bool dml32_CalculatePrefetchSchedule(
|
||||
|
||||
if (MyError) {
|
||||
*PrefetchBandwidth = 0;
|
||||
TimeForFetchingMetaPTE = 0;
|
||||
TimeForFetchingRowInVBlank = 0;
|
||||
*DestinationLinesToRequestVMInVBlank = 0;
|
||||
*DestinationLinesToRequestRowInVBlank = 0;
|
||||
*DestinationLinesForPrefetch = 0;
|
||||
LinesToRequestPrefetchPixelData = 0;
|
||||
*VRatioPrefetchY = 0;
|
||||
*VRatioPrefetchC = 0;
|
||||
*RequiredPrefetchPixDataBWLuma = 0;
|
||||
|
||||
Reference in New Issue
Block a user