Re: [bug report] ASoC: rsnd: adg: clearly handle clock error / NULL case

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Dan

Thank you for your feedback

> 	sound/soc/sh/rcar/adg.c:396 rsnd_adg_create_null_clk()
> 	warn: passing zero to 'ERR_CAST'
(snip)
> The clk_register_fixed_rate() function doesn't have any documentation,
> but generally when functions return a mix of error pointers and NULL
> then the NULL case means the feature is deliberately disabled.  So it
> should not print an error message.  Perhaps an info message at most.
> 
> --> 396 		return ERR_CAST(clk);
> 
> No need to cast this.  The types are the same.

Thanks. Will fix these in next week.

Thank you for your help !!

Best regards
---
Kuninori Morimoto



[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Pulse Audio]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux