Unverified Commit c0ea089d authored by Kuninori Morimoto's avatar Kuninori Morimoto Committed by Mark Brown
Browse files

ASoC: rsnd: rsnd_mod_name() handles both name and ID 



Current rsnd driver is using "%s[%d]" for mod name and ID,
but, this ID portion might confusable.
For example currently, CTU ID is 0 to 7, but using 00 to 13
(= 00, 01, 02, 03, 10, 11, 12, 13) is very best matching to datasheet.

In the future, we will support BUSIFn, but it will be more complicated
numbering. To avoid future confusable code, this patch modify
rsnd_mod_name() to return understandable name.

To avoid using pointless memory, it uses static char and snprintf,
thus, rsnd_mod_name() user should use it immediately, and shouldn't keep
its pointer.

Signed-off-by: default avatarKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Signed-off-by: default avatarMark Brown <broonie@kernel.org>
parent c16015f3

sound/soc/sh/rcar/core.c

+32 −9
Original line number Diff line number Diff line
@@ -123,8 +123,8 @@ void rsnd_mod_make_sure(struct rsnd_mod *mod, enum rsnd_mod_type type) 
		struct rsnd_priv *priv = rsnd_mod_to_priv(mod);

		struct device *dev = rsnd_priv_to_dev(priv);



		dev_warn(dev, "%s[%d] is not your expected module\n",

			 rsnd_mod_name(mod), rsnd_mod_id(mod));

		dev_warn(dev, "%s is not your expected module\n",

			 rsnd_mod_name(mod));

	}

}
@@ -137,6 +137,30 @@ struct dma_chan *rsnd_mod_dma_req(struct rsnd_dai_stream *io, 
	return mod->ops->dma_req(io, mod);

}



#define MOD_NAME_SIZE 16

char *rsnd_mod_name(struct rsnd_mod *mod)

{

	static char name[MOD_NAME_SIZE];



	/*

	 * Let's use same char to avoid pointlessness memory

	 * Thus, rsnd_mod_name() should be used immediately

	 * Don't keep pointer

	 */

	if ((mod)->ops->id_sub) {

		snprintf(name, MOD_NAME_SIZE, "%s[%d%d]",

			 mod->ops->name,

			 rsnd_mod_id(mod),

			 rsnd_mod_id_sub(mod));

	} else {

		snprintf(name, MOD_NAME_SIZE, "%s[%d]",

			 mod->ops->name,

			 rsnd_mod_id(mod));

	}



	return name;

}



u32 *rsnd_mod_get_status(struct rsnd_mod *mod,

			 struct rsnd_dai_stream *io,

			 enum rsnd_mod_type type)
@@ -494,15 +518,14 @@ static int rsnd_status_update(u32 *status, 
						__rsnd_mod_shift_##fn,	\

						__rsnd_mod_add_##fn,	\

						__rsnd_mod_call_##fn);	\

		rsnd_dbg_dai_call(dev, "%s[%d]\t0x%08x %s\n",		\

			rsnd_mod_name(mod), rsnd_mod_id(mod), *status,	\

		rsnd_dbg_dai_call(dev, "%s\t0x%08x %s\n",		\

			rsnd_mod_name(mod), *status,	\

			(func_call && (mod)->ops->fn) ? #fn : "");	\

		if (func_call && (mod)->ops->fn)			\

			tmp = (mod)->ops->fn(mod, io, param);		\

		if (tmp && (tmp != -EPROBE_DEFER))			\

			dev_err(dev, "%s[%d] : %s error %d\n",		\

				rsnd_mod_name(mod), rsnd_mod_id(mod),	\

						     #fn, tmp);		\

			dev_err(dev, "%s : %s error %d\n",		\

				rsnd_mod_name(mod), #fn, tmp);		\

		ret |= tmp;						\

	}								\

	ret;								\
@@ -529,8 +552,8 @@ int rsnd_dai_connect(struct rsnd_mod *mod, 


	io->mod[type] = mod;



	dev_dbg(dev, "%s[%d] is connected to io (%s)\n",

		rsnd_mod_name(mod), rsnd_mod_id(mod),

	dev_dbg(dev, "%s is connected to io (%s)\n",

		rsnd_mod_name(mod),

		rsnd_io_is_play(io) ? "Playback" : "Capture");



	return 0;

sound/soc/sh/rcar/dma.c

+8 −13
Original line number Diff line number Diff line
@@ -174,8 +174,8 @@ static int rsnd_dmaen_start(struct rsnd_mod *mod, 
	cfg.src_addr_width = DMA_SLAVE_BUSWIDTH_4_BYTES;

	cfg.dst_addr_width = DMA_SLAVE_BUSWIDTH_4_BYTES;



	dev_dbg(dev, "%s[%d] %pad -> %pad\n",

		rsnd_mod_name(mod), rsnd_mod_id(mod),

	dev_dbg(dev, "%s %pad -> %pad\n",

		rsnd_mod_name(mod),

		&cfg.src_addr, &cfg.dst_addr);



	ret = dmaengine_slave_config(dmaen->chan, &cfg);
@@ -369,8 +369,7 @@ static u32 rsnd_dmapp_get_id(struct rsnd_dai_stream *io, 
	if ((!entry) || (size <= id)) {

		struct device *dev = rsnd_priv_to_dev(rsnd_io_to_priv(io));



		dev_err(dev, "unknown connection (%s[%d])\n",

			rsnd_mod_name(mod), rsnd_mod_id(mod));

		dev_err(dev, "unknown connection (%s)\n", rsnd_mod_name(mod));



		/* use non-prohibited SRS number as error */

		return 0x00; /* SSI00 */
@@ -692,12 +691,10 @@ static void rsnd_dma_of_path(struct rsnd_mod *this, 
		*mod_to		= mod[1];

	}



	dev_dbg(dev, "module connection (this is %s[%d])\n",

		rsnd_mod_name(this), rsnd_mod_id(this));

	dev_dbg(dev, "module connection (this is %s)\n", rsnd_mod_name(this));

	for (i = 0; i <= idx; i++) {

		dev_dbg(dev, "  %s[%d]%s\n",

		dev_dbg(dev, "  %s%s\n",

			rsnd_mod_name(mod[i] ? mod[i] : &mem),

			rsnd_mod_id  (mod[i] ? mod[i] : &mem),

			(mod[i] == *mod_from) ? " from" :

			(mod[i] == *mod_to)   ? " to" : "");

	}
@@ -762,12 +759,10 @@ static int rsnd_dma_alloc(struct rsnd_dai_stream *io, struct rsnd_mod *mod, 
	if (ret < 0)

		return ret;



	dev_dbg(dev, "%s[%d] %s[%d] -> %s[%d]\n",

		rsnd_mod_name(*dma_mod), rsnd_mod_id(*dma_mod),

	dev_dbg(dev, "%s %s -> %s\n",

		rsnd_mod_name(*dma_mod),

		rsnd_mod_name(mod_from ? mod_from : &mem),

		rsnd_mod_id  (mod_from ? mod_from : &mem),

		rsnd_mod_name(mod_to   ? mod_to   : &mem),

		rsnd_mod_id  (mod_to   ? mod_to   : &mem));

		rsnd_mod_name(mod_to   ? mod_to   : &mem));



	ret = attach(io, dma, mod_from, mod_to);

	if (ret < 0)

sound/soc/sh/rcar/gen.c

+6 −6
Original line number Diff line number Diff line
@@ -83,8 +83,8 @@ u32 rsnd_read(struct rsnd_priv *priv, 


	regmap_fields_read(gen->regs[reg], rsnd_mod_id(mod), &val);



	dev_dbg(dev, "r %s[%d] - %-18s (%4d) : %08x\n",

		rsnd_mod_name(mod), rsnd_mod_id(mod),

	dev_dbg(dev, "r %s - %-18s (%4d) : %08x\n",

		rsnd_mod_name(mod),

		rsnd_reg_name(gen, reg), reg, val);



	return val;
@@ -102,8 +102,8 @@ void rsnd_write(struct rsnd_priv *priv, 


	regmap_fields_force_write(gen->regs[reg], rsnd_mod_id(mod), data);



	dev_dbg(dev, "w %s[%d] - %-18s (%4d) : %08x\n",

		rsnd_mod_name(mod), rsnd_mod_id(mod),

	dev_dbg(dev, "w %s - %-18s (%4d) : %08x\n",

		rsnd_mod_name(mod),

		rsnd_reg_name(gen, reg), reg, data);

}
@@ -119,8 +119,8 @@ void rsnd_bset(struct rsnd_priv *priv, struct rsnd_mod *mod, 
	regmap_fields_force_update_bits(gen->regs[reg],

					rsnd_mod_id(mod), mask, data);



	dev_dbg(dev, "b %s[%d] - %-18s (%4d) : %08x/%08x\n",

		rsnd_mod_name(mod), rsnd_mod_id(mod),

	dev_dbg(dev, "b %s - %-18s (%4d) : %08x/%08x\n",

		rsnd_mod_name(mod),

		rsnd_reg_name(gen, reg), reg, data, mask);



}

sound/soc/sh/rcar/rsnd.h

+1 −1
Original line number Diff line number Diff line
@@ -377,7 +377,6 @@ struct rsnd_mod { 
#define __rsnd_mod_call_pointer		0



#define rsnd_mod_to_priv(mod)	((mod)->priv)

#define rsnd_mod_name(mod)	((mod)->ops->name)

#define rsnd_mod_power_on(mod)	clk_enable((mod)->clk)

#define rsnd_mod_power_off(mod)	clk_disable((mod)->clk)

#define rsnd_mod_get(ip)	(&(ip)->mod)
@@ -400,6 +399,7 @@ u32 *rsnd_mod_get_status(struct rsnd_mod *mod, 
int rsnd_mod_id(struct rsnd_mod *mod);

int rsnd_mod_id_raw(struct rsnd_mod *mod);

int rsnd_mod_id_sub(struct rsnd_mod *mod);

char *rsnd_mod_name(struct rsnd_mod *mod);

struct rsnd_mod *rsnd_mod_next(int *iterator,

			       struct rsnd_dai_stream *io,

			       enum rsnd_mod_type *array,

sound/soc/sh/rcar/src.c

+2 −3
Original line number Diff line number Diff line
@@ -349,9 +349,8 @@ static bool rsnd_src_error_occurred(struct rsnd_mod *mod) 
	status0 = rsnd_mod_read(mod, SCU_SYS_STATUS0);

	status1 = rsnd_mod_read(mod, SCU_SYS_STATUS1);

	if ((status0 & val0) || (status1 & val1)) {

		rsnd_dbg_irq_status(dev, "%s[%d] err status : 0x%08x, 0x%08x\n",

			rsnd_mod_name(mod), rsnd_mod_id(mod),

			status0, status1);

		rsnd_dbg_irq_status(dev, "%s err status : 0x%08x, 0x%08x\n",

			rsnd_mod_name(mod), status0, status1);



		ret = true;

	}

CRA Git | Maintained and supported by SUSTech CRA and CCSE