[coreboot-gerrit] Change in coreboot[master]: soc/rockchip/rk3399/clock: Add rkclk_ddr_reset() function

[Angel Pons (Code Review)]

Attention is currently required from: Julius Werner, Moritz Fischer.
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/50889 )

Change subject: soc/rockchip/rk3399/clock: Add rkclk_ddr_reset() function
......................................................................

Patch Set 2: Code-Review+1

(1 comment)

File src/soc/rockchip/rk3399/clock.c:

https://review.coreboot.org/c/coreboot/+/50889/comment/20d42396_843bea53 
PS2, Line 656: 					((n) << (9 + (ch) * 4)))
Hmm, am I missing something, or could this be simpler?

#define CRU_SFTRST_DDR_CTRL(ch, n)	((1 << 16 | (n)) << (8 + (ch) * 4))
 #define CRU_SFTRST_DDR_PHY(ch, n)	((1 << 16 | (n)) << (9 + (ch) * 4))