Re: [PATCH] [RFC] ASoC: rsnd: Add missing initialization of ADG req_rate

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

 



Hi Geert

> If the "clock-frequency" DT property is not found, req_rate is used
> uninitialized, and the "audio_clkout" clock will be created with an
> arbitrary clock rate.
> 
> This uninitialized kernel stack data may leak to userspace through
> /sys/kernel/debug/clk/clk_summary, cfr. the value in the "rate" column:
> 
>        clock     enable_cnt  prepare_cnt        rate   accuracy   phase
>     --------------------------------------------------------------------
>      audio_clkout         0            0  4001836240          0 0
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>
> ---

Acked-by: Kuninori Morimoto <kuninori.morimoto.gx@xxxxxxxxxxx>

> Should the absence of this property be considered an error?
> 
> Documentation/devicetree/bindings/sound/renesas,rsnd.txt states that
> both "#clock-cells" and "clock-frequency" are required properties, but
> none of the R-Car Gen1 and Gen2 DTSes seem to have them?

This clkout is option for each board
will fix
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel



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

  Powered by Linux