On Feb 11, 2024, at 10:39 AM, Krzysztof Kozlowski krzysztof.kozlowski@xxxxxxxxxx wrote: > On 07/02/2024 19:01, Charles Perry wrote: >> By convention, gpio consumer names should not contain underscores >> (prog_b here) and shouldn't contain active low suffixes (-b here). >> >> Signed-off-by: Charles Perry <charles.perry@xxxxxxxxxxxxxxxxxxxx> >> --- >> .../bindings/fpga/xlnx,fpga-slave-serial.yaml | 12 ++++++------ >> 1 file changed, 6 insertions(+), 6 deletions(-) >> >> diff --git a/Documentation/devicetree/bindings/fpga/xlnx,fpga-slave-serial.yaml >> b/Documentation/devicetree/bindings/fpga/xlnx,fpga-slave-serial.yaml >> index 614d86ad825f3..650a4d8792b64 100644 >> --- a/Documentation/devicetree/bindings/fpga/xlnx,fpga-slave-serial.yaml >> +++ b/Documentation/devicetree/bindings/fpga/xlnx,fpga-slave-serial.yaml >> @@ -36,7 +36,7 @@ properties: >> reg: >> maxItems: 1 >> >> - prog_b-gpios: >> + prog-gpios: > > Please deprecate old property and add allOf excluding the usage of both. > Driver still parses old property, so we should have it documented. > > https://lore.kernel.org/all/20230118163208.GA117919-robh@xxxxxxxxxx/ > > Best regards, > Krzysztof Ok. Thank you for the example. Regards, Charles