Re: [PATCH 1/2] dt-bindings: mtd: nuvoton,ma35d1-nand: add new bindings

Dear Krzysztof,

Thank you for your reply.



On 2024/8/12 下午 05:12, Krzysztof Kozlowski wrote:

On 12/08/2024 11:02, Hui-Ping Chen wrote:

quoted

quoted

quoted

+
+      nand-ecc-step-size:
+        enum: [512, 1024]

No defaults? So is this required?

This is required, but I will also add a default.

If this is required and should be in required: list. Default does not
make sense then... it contradicts the point of being required.

I will add it to the required list.

quoted

quoted

quoted

+
+      nand-ecc-strength:
+        enum: [8, 12, 24]

No defaults? So is this required?

This is required, but I will also add a default.

Ditto

I will add it to the required list.

Best regards,
Krzysztof

Best regards,

Hui-Ping Chen