Re: [PATCH v2 1/2] dt-bindings: chrome: Add binding for ChromeOS Pogo pin connector
From: Konrad Dybcio <hidden>
Date: 2025-02-06 11:30:56
Also in:
chrome-platform, linux-arm-msm, linux-patches, lkml
On 6.02.2025 12:30 AM, Stephen Boyd wrote:
quoted hunk ↗ jump to hunk
Describe the set of pins used to connect the detachable keyboard on detachable ChromeOS devices. The set of pins is called the "pogo pins". It's basically USB 2.0 with an extra pin for base detection. We expect to find a keyboard on the other side of this connector with a specific vid/pid, so describe that as a child device at the port of the usb device connected upstream. Cc: Rob Herring <robh@kernel.org> Cc: Krzysztof Kozlowski <krzk+dt@kernel.org> Cc: Conor Dooley <conor+dt@kernel.org> Cc: Benson Leung <bleung@chromium.org> Cc: <redacted> Cc: <redacted> Cc: Pin-yen Lin <redacted> Signed-off-by: Stephen Boyd <redacted> --- .../chrome/google,pogo-pin-connector.yaml | 68 +++++++++++++++++++ 1 file changed, 68 insertions(+) create mode 100644 Documentation/devicetree/bindings/chrome/google,pogo-pin-connector.yamldiff --git a/Documentation/devicetree/bindings/chrome/google,pogo-pin-connector.yaml b/Documentation/devicetree/bindings/chrome/google,pogo-pin-connector.yaml new file mode 100644 index 000000000000..622e171b6b08 --- /dev/null +++ b/Documentation/devicetree/bindings/chrome/google,pogo-pin-connector.yaml@@ -0,0 +1,68 @@ +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/chrome/google,pogo-pin-connector.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: Google Pogo Pin Connector
This looks like a very generic piece of hw.. many boards (esp. convertibles) do the same thing, with 4 or 5 pins on the bottom of the device. But of course hw manufacturers being hw manufacturers, many different kinds of signals go through such connectors - if it's not USB then it's perhaps I2C or some variation thereof IMO, we could perhaps add this to usb-connector.yaml as "usb-custom-connector" or so Konrad