[PATCH v2 3/4] dt-bindings: PCI: Add UltraRISC DP1000 PCIe controller
Jia Wang
wangjia at ultrarisc.com
Tue Apr 7 23:43:19 PDT 2026
On 2026-04-08 08:28 +0200, Krzysztof Kozlowski wrote:
> On 08/04/2026 05:34, Jia Wang wrote:
> >>> + max-link-speed:
> >>> + $ref: /schemas/types.yaml#/definitions/uint32
> >>> + const: 4
> >>
> >> If const then deducible from the compatible. Drop the property.
> >>
> >
> > Will replace `const: 4` with `maximum: 4` in v3.
>
> Why? Wasn't maximum link speed fixed to 4?
>
Just to make sure I fully understand: since the maximum link speed is a
fixed hardware property and is implied by the compatible, we should drop
the `max-link-speed` property from the binding.
In that case, should I set `pci->max_link_speed = 4` in the driver during
probe? I want to make sure this is the correct way to handle it.
>
> Best regards,
> Krzysztof
>
Best regards,
Jia Wang
More information about the linux-riscv
mailing list