| Message ID | 20241120101911.1142500-1-heiko@sntech.de |
|---|---|
| State | Accepted |
| Delegated to: | Kever Yang |
| Headers | show |
| Series | clk: rockchip: rk3588: fix mask define for aclk_vop_root | expand |
Hi Heiko, On 11/20/24 11:19 AM, Heiko Stuebner wrote: > The mask for aclk_vop_root is 3-bit wide, not 2-bit wide according > to the TRM, so set the mask accordingly. > > Signed-off-by: Heiko Stuebner <heiko@sntech.de> Reviewed-by: Quentin Schulz <quentin.schulz@cherry.de> Was this a drive-by bug fix or did you actually experience some bug? Basically trying to figure out if we should push for this being merged for 2025.01 or if it's fine if it misses it? Thanks! Quentin
Am Mittwoch, 20. November 2024, 12:13:32 CET schrieb Quentin Schulz: > On 11/20/24 11:19 AM, Heiko Stuebner wrote: > > The mask for aclk_vop_root is 3-bit wide, not 2-bit wide according > > to the TRM, so set the mask accordingly. > > > > Signed-off-by: Heiko Stuebner <heiko@sntech.de> > > Reviewed-by: Quentin Schulz <quentin.schulz@cherry.de> > > Was this a drive-by bug fix or did you actually experience some bug? > Basically trying to figure out if we should push for this being merged > for 2025.01 or if it's fine if it misses it? on my side, it was a "drive-by" thing and I actually don't remember anymore, what made me look at that. So I don't experience any bug on my side. Heiko
On 2024/11/20 18:19, Heiko Stuebner wrote: > The mask for aclk_vop_root is 3-bit wide, not 2-bit wide according > to the TRM, so set the mask accordingly. > > Signed-off-by: Heiko Stuebner <heiko@sntech.de> There is a same change/fix to vendor U-Boot at 2022. Reviewed-by: Kever Yang <kever.yang@rock-chips.com> Thanks, - Kever > --- > arch/arm/include/asm/arch-rockchip/cru_rk3588.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arm/include/asm/arch-rockchip/cru_rk3588.h b/arch/arm/include/asm/arch-rockchip/cru_rk3588.h > index dad484813fa..afce8a44af3 100644 > --- a/arch/arm/include/asm/arch-rockchip/cru_rk3588.h > +++ b/arch/arm/include/asm/arch-rockchip/cru_rk3588.h > @@ -346,7 +346,7 @@ enum { > ACLK_VOP_LOW_ROOT_SEL_100M, > ACLK_VOP_LOW_ROOT_SEL_24M, > ACLK_VOP_ROOT_SEL_SHIFT = 5, > - ACLK_VOP_ROOT_SEL_MASK = 3 << ACLK_VOP_ROOT_SEL_SHIFT, > + ACLK_VOP_ROOT_SEL_MASK = 7 << ACLK_VOP_ROOT_SEL_SHIFT, > ACLK_VOP_ROOT_SEL_GPLL = 0, > ACLK_VOP_ROOT_SEL_CPLL, > ACLK_VOP_ROOT_SEL_AUPLL,
diff --git a/arch/arm/include/asm/arch-rockchip/cru_rk3588.h b/arch/arm/include/asm/arch-rockchip/cru_rk3588.h index dad484813fa..afce8a44af3 100644 --- a/arch/arm/include/asm/arch-rockchip/cru_rk3588.h +++ b/arch/arm/include/asm/arch-rockchip/cru_rk3588.h @@ -346,7 +346,7 @@ enum { ACLK_VOP_LOW_ROOT_SEL_100M, ACLK_VOP_LOW_ROOT_SEL_24M, ACLK_VOP_ROOT_SEL_SHIFT = 5, - ACLK_VOP_ROOT_SEL_MASK = 3 << ACLK_VOP_ROOT_SEL_SHIFT, + ACLK_VOP_ROOT_SEL_MASK = 7 << ACLK_VOP_ROOT_SEL_SHIFT, ACLK_VOP_ROOT_SEL_GPLL = 0, ACLK_VOP_ROOT_SEL_CPLL, ACLK_VOP_ROOT_SEL_AUPLL,
The mask for aclk_vop_root is 3-bit wide, not 2-bit wide according to the TRM, so set the mask accordingly. Signed-off-by: Heiko Stuebner <heiko@sntech.de> --- arch/arm/include/asm/arch-rockchip/cru_rk3588.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)