drivers: dma: intel-adsp-hda: add a missing "break"

lyakh · carlescufi · commit db1a718341a3 · 2023-07-01T12:38:02.000+02:00
A "switch" statement in intel_adsp_hda_dma_status() seems to be
missing a "break". The second "break" is unneeded but seems to be a
part of the coding style.

Signed-off-by: Guennadi Liakhovetski &lt;guennadi.liakhovetski@linux.intel.com&gt;
diff --git a/drivers/dma/dma_intel_adsp_hda.c b/drivers/dma/dma_intel_adsp_hda.c
@@ -212,13 +212,15 @@ int intel_adsp_hda_dma_status(const struct device *dev, uint32_t channel,
 			intel_adsp_hda_underrun_clear(cfg->base, cfg->regblock_size, channel);
 			return -EPIPE;
 		}
+		break;
 	case PERIPHERAL_TO_MEMORY:
 		xrun_det = intel_adsp_hda_is_buffer_overrun(cfg->base, cfg->regblock_size,
 							    channel);
 		if (xrun_det) {
 			intel_adsp_hda_overrun_clear(cfg->base, cfg->regblock_size, channel);
 			return -EPIPE;
 		}
+		break;
 	default:
 		break;
 	}

Original file line number	Diff line number	Diff line change
`@@ -212,13 +212,15 @@ int intel_adsp_hda_dma_status(const struct device *dev, uint32_t channel,`
`212`	`212`	`intel_adsp_hda_underrun_clear(cfg->base, cfg->regblock_size, channel);`
`213`	`213`	`return -EPIPE;`
`214`	`214`	`}`
	`215`	`+ break;`
`215`	`216`	`case PERIPHERAL_TO_MEMORY:`
`216`	`217`	`xrun_det = intel_adsp_hda_is_buffer_overrun(cfg->base, cfg->regblock_size,`
`217`	`218`	`channel);`
`218`	`219`	`if (xrun_det) {`
`219`	`220`	`intel_adsp_hda_overrun_clear(cfg->base, cfg->regblock_size, channel);`
`220`	`221`	`return -EPIPE;`
`221`	`222`	`}`
	`223`	`+ break;`
`222`	`224`	`default:`
`223`	`225`	`break;`
`224`	`226`	`}`