drm/nouveau: Fix fall-through warnings for Clang

GustavoARSilva · GustavoARSilva · commit f49efb108aa4 · 2021-06-03T23:21:51.000-05:00
In preparation to enable -Wimplicit-fallthrough for Clang, fix a couple of warnings by explicitly adding a couple of break statements instead of letting the code fall through to the next case. Link: KSPP#115 Reviewed-by: Karol Herbst <kherbst@redhat.com> Signed-off-by: Gustavo A. R. Silva <gustavoars@kernel.org>
diff --git a/drivers/gpu/drm/nouveau/nouveau_bo.c b/drivers/gpu/drm/nouveau/nouveau_bo.c
@@ -440,6 +440,7 @@ nouveau_bo_pin(struct nouveau_bo *nvbo, uint32_t domain, bool contig)
 			break;
 		case TTM_PL_TT:
 			error |= !(domain & NOUVEAU_GEM_DOMAIN_GART);
+			break;
 		default:
 			break;
 		}
diff --git a/drivers/gpu/drm/nouveau/nouveau_connector.c b/drivers/gpu/drm/nouveau/nouveau_connector.c
@@ -157,6 +157,7 @@ nouveau_conn_atomic_set_property(struct drm_connector *connector,
 			default:
 				break;
 			}
+			break;
 		case DRM_MODE_SCALE_FULLSCREEN:
 		case DRM_MODE_SCALE_CENTER:
 		case DRM_MODE_SCALE_ASPECT:

Original file line number	Diff line number	Diff line change
`@@ -440,6 +440,7 @@ nouveau_bo_pin(struct nouveau_bo *nvbo, uint32_t domain, bool contig)`
`440`	`440`	`break;`
`441`	`441`	`case TTM_PL_TT:`
`442`	`442`	`error \|= !(domain & NOUVEAU_GEM_DOMAIN_GART);`
	`443`	`+ break;`
`443`	`444`	`default:`
`444`	`445`	`break;`
`445`	`446`	`}`
Original file line number	Diff line number	Diff line change
`@@ -157,6 +157,7 @@ nouveau_conn_atomic_set_property(struct drm_connector *connector,`
`157`	`157`	`default:`
`158`	`158`	`break;`
`159`	`159`	`}`
	`160`	`+ break;`
`160`	`161`	`case DRM_MODE_SCALE_FULLSCREEN:`
`161`	`162`	`case DRM_MODE_SCALE_CENTER:`
`162`	`163`	`case DRM_MODE_SCALE_ASPECT:`