Re: [PATCH 1/2] dt-bindings: socfpga: Add the Agilex7 series SoC's

[Krzysztof Kozlowski <krzk@kernel.org>]

On 14/04/2026 14:53, Dinh Nguyen wrote:
> 
> 
> On 4/14/26 02:17, Krzysztof Kozlowski wrote:
>> On Mon, Apr 13, 2026 at 09:45:52AM -0500, Dinh Nguyen wrote:
>>> The Agilex7 is a series of devices from Altera that are derived from
>>> the Agilex family.
>>>
>>> The Agilex7F device supports PCIE 4.0 and DDR4. The Agilex7I device supports
>>> PCIE 5.0 and DDR4, while the Agilex7M device supports DDR4, DDR5, LPDDR5
>>> and PCIE 5.0.
>>>
>>> All other peripherals from these devices are the same as the Agilex
>>> device.
>>>
>>> Signed-off-by: Dinh Nguyen <dinguyen@kernel.org>
>>> ---
>>>   Documentation/devicetree/bindings/arm/altera.yaml | 10 ++++++++++
>>>   1 file changed, 10 insertions(+)
>>>
>>> diff --git a/Documentation/devicetree/bindings/arm/altera.yaml b/Documentation/devicetree/bindings/arm/altera.yaml
>>> index 206686f3eebc..5ee09f8d4698 100644
>>> --- a/Documentation/devicetree/bindings/arm/altera.yaml
>>> +++ b/Documentation/devicetree/bindings/arm/altera.yaml
>>> @@ -115,6 +115,16 @@ properties:
>>>                 - intel,socfpga-agilex5-socdk-nand
>>>             - const: intel,socfpga-agilex5
>>>   
>>> +      - description: Agilex7 series F, I and M boards
>>> +        items:
>>> +          - enum:
>>> +              - intel,socfpga-agilex7m-socdk
>>> +          - enum:
>>> +              - intel,socfpga-agilex7f
>>> +              - intel,socfpga-agilex7i
>>> +              - intel,socfpga-agilex7m
>>> +          - const: intel,socfpga-agilex
>>
>> And separate question - why previous soc "agilex" is used as fallback?
>> Even more confusing.
>>
> 
> You're right. Sorry for the confusion. The Agilex7M, I, F devices are 
> basically "agilex" devices with some few additions (PCIE, DDR5). Maybe I 
> should place the Agilex7M/I/F devices into the "agilex" boards area?

Compatibles should be specific and not based on families, thus what is
"intel,socfpga-agilex"? SoC, right?

Then "intel,socfpga-agilex7f" is a new SoC, no?

Best regards,
Krzysztof