Browse Source

ASoC: omap: mcbsp: Remove redundant checks for the st_data pointer

The parent functions of omap_st_start/stop also checks the validity of the
st_data pointer so we do not need to do it again inside of omap_st_start/stop

Signed-off-by: Peter Ujfalusi <peter.ujfalusi@ti.com>
Acked-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
Acked-by: Jarkko Nikula <jarkko.nikula@bitmer.com>
Signed-off-by: Liam Girdwood <lrg@ti.com>
Peter Ujfalusi 13 years ago
parent
commit
58db1dcde0
1 changed files with 2 additions and 2 deletions
  1. 2 2
      sound/soc/omap/mcbsp.c

+ 2 - 2
sound/soc/omap/mcbsp.c

@@ -318,7 +318,7 @@ static int omap_st_start(struct omap_mcbsp *mcbsp)
 {
 {
 	struct omap_mcbsp_st_data *st_data = mcbsp->st_data;
 	struct omap_mcbsp_st_data *st_data = mcbsp->st_data;
 
 
-	if (st_data && st_data->enabled && !st_data->running) {
+	if (st_data->enabled && !st_data->running) {
 		omap_st_fir_write(mcbsp, st_data->taps);
 		omap_st_fir_write(mcbsp, st_data->taps);
 		omap_st_chgain(mcbsp);
 		omap_st_chgain(mcbsp);
 
 
@@ -350,7 +350,7 @@ static int omap_st_stop(struct omap_mcbsp *mcbsp)
 {
 {
 	struct omap_mcbsp_st_data *st_data = mcbsp->st_data;
 	struct omap_mcbsp_st_data *st_data = mcbsp->st_data;
 
 
-	if (st_data && st_data->running) {
+	if (st_data->running) {
 		if (!mcbsp->free) {
 		if (!mcbsp->free) {
 			omap_st_off(mcbsp);
 			omap_st_off(mcbsp);
 			st_data->running = 0;
 			st_data->running = 0;