Re: [PATCH v2 1/3] dt-bindings: usb: usb251xb: make reg optional for no I2C ctrl use case

(追記) (追記ここまで)

• To: Jisheng Zhang <jszhang@xxxxxxxxxx>
• Subject: Re: [PATCH v2 1/3] dt-bindings: usb: usb251xb: make reg optional for no I2C ctrl use case
• From: Krzysztof Kozlowski <krzk@xxxxxxxxxx>
• Date: 2025年8月19日 11:22:51 +0200
• Cc: Richard Leitner <richard.leitner@xxxxxxxxx>, Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>, Rob Herring <robh@xxxxxxxxxx>, Krzysztof Kozlowski <krzk+dt@xxxxxxxxxx>, Conor Dooley <conor+dt@xxxxxxxxxx>, linux-usb@xxxxxxxxxxxxxxx, devicetree@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx
• In-reply-to: <20250819002217.30249-2-jszhang@kernel.org>
• References: <20250819002217.30249-1-jszhang@kernel.org> <20250819002217.30249-2-jszhang@kernel.org>

On Tue, Aug 19, 2025 at 08:22:15AM +0800, Jisheng Zhang wrote:
> Make "reg" optional to allow the driver to be used to manage GPIO
> based control of the device.
I don't understand how optional reg allows or disallows anything for the
driver. It's really not relevant to GPIO at all.
Also feels wrong, but maybe you just lack proper, reasonable
justification. Just explain carefully the hardware, not drivers.
Best regards,
Krzysztof

• Follow-Ups:
  • Re: [PATCH v2 1/3] dt-bindings: usb: usb251xb: make reg optional for no I2C ctrl use case
    • From: Jisheng Zhang

• References:
  • [PATCH v2 0/2] usb: usb251xb: support usage case without I2C control
    • From: Jisheng Zhang
  • [PATCH v2 1/3] dt-bindings: usb: usb251xb: make reg optional for no I2C ctrl use case
    • From: Jisheng Zhang

• Prev by Date: Re: [PATCH v2 3/3] usb: usb251xb: support usage case without I2C control
• Next by Date: Re: [REGRESSION 6.16] xHCI host not responding to stop endpoint command after suspend and resume
• Previous by thread: [PATCH v2 1/3] dt-bindings: usb: usb251xb: make reg optional for no I2C ctrl use case
• Next by thread: Re: [PATCH v2 1/3] dt-bindings: usb: usb251xb: make reg optional for no I2C ctrl use case
• Index(es):
  • Date
  • Thread