Re: [PATCH 6/6] clk: renesas: rza1: Remove struct rz_cpg

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

 



On Wed, Jun 08, 2022 at 03:41:15PM +0200, Geert Uytterhoeven wrote:
> The register block base pointer as stored in the reg member of the
> rz_cpg structure is only used during initialization.  Hence move
> it to a local variable, and pass it as a parameter to
> rz_cpg_register_clock().
> 
> After this, the data member is the only remaining member of the rz_cpg
> structure, so the whole structure can be replaced by the data member.
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>

I totally forgot I modified this driver in 2014. Nice memories, though.

Reviewed-by: Wolfram Sang <wsa+renesas@xxxxxxxxxxxxxxxxxxxx>

Disclaimer: My reviews for this series were visual. I don't have the HW
but it all LGTM.

>  	 * let them run at fixed current speed and implement the details later.

Seems that was never needed :)

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [Linux Samsung SOC]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux