On 12/07/2025 15:39, Magnus Damm wrote:
> From: Magnus Damm <damm@xxxxxxxxxxxxx>
>
> Add RZ/A1 and RZ/A2 compat strings to the rpc-if driver. Also make the
> reset controller optional. This is because RZ/A1 does not have any reset
> bits assigned to the device so there is no reset controller available.
That's different ABI and if you tested bindings, you would see warnings.
My previous instruction was really important so I am disappointed that
you replied that you are not going to test bindings or your DTS patches.
Fix the bindings.
> +
> static const struct of_device_id rpcif_of_match[] = {
> + { .compatible = "renesas,r7s72100-rpc-if", .data = &rpcif_info_rz_a },
> + { .compatible = "renesas,r7s9210-rpc-if", .data = &rpcif_info_rz_a },
So devices are compatible. Express it with proper fallbacks and don't
create redundant entries.
Or explain in the binding WHY they are not compatible.
Best regards,
Krzysztof
