Hi Quentin,

On 10/9/2026 12:08 AM, Quentin Schulz wrote:
> Hi Jonas,
> 
> On 8/4/26 12:57 PM, Jonas Karlman wrote:
>> Modern Rockchip device drivers use syscon_regmap_lookup_by_phandle()
>> to get a regmap or use a constant to get the fixed base address.
>>
>> Remove the unneeded RK3506 syscon driver to discourage new uses of
>> syscon_get_regmap_by_driver_data() or syscon_get_first_range().
>>
>> The generic_syscon driver will instead be used for the compatible
>> previously matched by the now removed driver.
>>
> 
> Same remark as for the patch for RK3528 except that here it's worse, 
> there isn't even an upstream DTS(I) for RK3506. Just slightly reword to 
> avoid the misleading claim.

Good catch, and yes there is no upstream DT for RK3506.

The DT at [1] is likely ready to be upstreamed, just not enough time ;-)

[1] 
https://git.u-boot-project.org/u-boot/contributors/kwiboo/u-boot/-/blob/rk3562/arch/arm/dts/rk3506.dtsi

Regards,
Jonas

> 
> Reviewed-by: Quentin Schulz <[email protected]>
> 
> Thanks!
> Quentin

Reply via email to