From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wr1-f42.google.com (mail-wr1-f42.google.com [209.85.221.42]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id DE44031062E for ; Wed, 10 Dec 2025 18:00:33 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.42 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1765389635; cv=none; b=Dr43plHx1Sfe620gVmwJrrG1SCo0kzhu19iVl9nZdQmNgdRfzRYkIsLAGXx8xXsntjxa1FNh7OQqxqDHwhxo1/JYNFt5lc9xiz91iVi9nOgf2azhH/7cAuM3mUJmRzHoUU63ANBJNaL0w1t4yB5/qmuG3ESq1JFM0SeeG7+x14s= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1765389635; c=relaxed/simple; bh=a+WgLV8Me8he8vsaz9WjfiuVCZvCj6zKFhsOIiTYBLs=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=YmgW+bq1+JqLFBj31q8xz88a08bGkgRMWJhHbglXrjyCKyTaLy4QSWfrQa4YCD45NcPIg4agYVAqrG9j63HB15OZ8RuatJmVQGoEK5HOAHmwJAQpImEz5/UVoJj0UQ3QbrksEZzJcZtxI3RJ46Bi69GmGx+tozvZ4FbGM1sGeoY= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=chromium.org; spf=pass smtp.mailfrom=chromium.org; dkim=pass (1024-bit key) header.d=chromium.org header.i=@chromium.org header.b=JInx26C1; arc=none smtp.client-ip=209.85.221.42 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=chromium.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=chromium.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=chromium.org header.i=@chromium.org header.b="JInx26C1" Received: by mail-wr1-f42.google.com with SMTP id ffacd0b85a97d-42e2e628f8aso13538f8f.1 for ; Wed, 10 Dec 2025 10:00:33 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; t=1765389632; x=1765994432; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=vNo6uQ8AO0yOaAEJTUC3/zvI8C5HZVj+hKvmdCETuYM=; b=JInx26C1T9J4bIK8XSvC0fuP1epZ1mBZq02rYH0f/BaN3cbM6qusuYxq4pKQ3dllxp xQsWKTr4XHbPLFgWXWcTOptPq26oOiG1D69YNKH3V/I3A30ohO5ByoJaLTAuPQtqKJnW X+prjUObi70W7zv4fMdlW04mPK63PaT0bgeJA= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1765389632; x=1765994432; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-gg:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=vNo6uQ8AO0yOaAEJTUC3/zvI8C5HZVj+hKvmdCETuYM=; b=feu3ewiveRTSzuHOz9x1X2iwPBzcfAOjag5WrTtp9GqpiE46YtdjPx4tjkjBZzLZX4 o59HzLftONWiVZqJZu05A5NpZZ4RDIch6PjKa3Bghz2YqOOrd4bD3mxRW96ZxSjByNiM 1GO3uRBUqBOF7Telr1qegbCkMV8f3Jighov7gWJwimPPQKE3DSeBml2zB8mJOXXVNXF4 8hQaK6XHmvmYHVrzz5jjKDWVmkCoQNMdy4NuYvkCU19WWixopALkpak8Ha7cGlBLI84P RaBleWWlSDmPZMoeQNneMWRB7P6urTWuOfbUoLvRoMjNKjvS9X0vm1xsQV1w0sfdiWAC vCow== X-Forwarded-Encrypted: i=1; AJvYcCXu85JS+m344QndmlseU0kbH9T94cOHGkOHo1lajpHBNp+0X0WDG1DES8EGHTSbVrnWI69zNYs1Z1YhFc0=@vger.kernel.org X-Gm-Message-State: AOJu0YzAeCD1geNbQPoSwTlYFNOd+9js3UoVmFTuiB+WQMTVmgddedSX TxlafDAivWJKXb0FIuyADTtdUp5wyTZvoKyvp3oo+nEw4gzCdI8KSWmD7PRZC6L0WQ== X-Gm-Gg: AY/fxX43Vi7wV/2NAgQ4I/BVL2KHhkIBPUUfFqBLGzt9dteyjmlJ6qE//4EMWu4Q5kr 9JP2oN0404JwmLop+yJnZp3ir1ybZbGajq+ffNBWwyUMnFCx7ziLjIqDasUUvm2frF7sw+PCVwW GygepcrXIAzGX4wUgtOn6WU5EvWO/JWv6SHxk0dw9XmLTdpcDEPDrf4gh814FM5zckIy8i3V62J XVFa2bScwlbYgsltPSxxiBPrfSuzFP8ulhyXa3StszujrwtZQHeFDe7Wj1y8qcwwHMEYDorZXF9 rtiO5my+HMV1sMIKSPhmzNhBs5WlUFINXbB8ubUspSyo/cVKzYBQl4eSqKkhyrUystwZ451pmhJ vs+l0PZtmm2jN6IbdrkBC4SZ2qr/rpjkuEKPWmhL/y4yEggrGPvbO39gJ943++AzmzjTiJ8tWVU GUN+Tc+kL/HmlqSqhQ/A== X-Google-Smtp-Source: AGHT+IGPESWqtdwT/XRTR5/RmcuAb3YXC3b7FMypahW+beW9pJZ9LM1+rqVviz4PyBnlnycCgdJNOQ== X-Received: by 2002:a05:6000:608:b0:429:b9bc:e826 with SMTP id ffacd0b85a97d-42fa3b079c1mr3929886f8f.53.1765389632184; Wed, 10 Dec 2025 10:00:32 -0800 (PST) Received: from google.com ([37.228.206.31]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-42fa8b8601csm291118f8f.22.2025.12.10.10.00.30 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 10 Dec 2025 10:00:31 -0800 (PST) Date: Wed, 10 Dec 2025 18:00:29 +0000 From: Fabio Baltieri To: Rob Herring Cc: Dmitry Torokhov , Krzysztof Kozlowski , Conor Dooley , Benson Leung , Guenter Roeck , Tzung-Bi Shih , Simon Glass , linux-input@vger.kernel.org, devicetree@vger.kernel.org, chrome-platform@lists.linux.dev, linux-kernel@vger.kernel.org Subject: Re: [PATCH v1 3/3] dt-bindings: google,cros-ec-keyb: add fn-key and f-keymap props Message-ID: References: <20251209154706.529784-1-fabiobaltieri@chromium.org> <20251209154706.529784-4-fabiobaltieri@chromium.org> <20251209192243.GA963693-robh@kernel.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20251209192243.GA963693-robh@kernel.org> Hey Rob, thanks for the review. On Tue, Dec 09, 2025 at 01:22:43PM -0600, Rob Herring wrote: > On Tue, Dec 09, 2025 at 03:47:06PM +0000, Fabio Baltieri wrote: > > + fn-key: > > + $ref: /schemas/types.yaml#/definitions/uint32 > > + description: | > > + An u32 containing the coordinate of the Fn key, use the MATRIX_KEY(row, > > + col, code) macro, code is ignored. > > + > > + fn-keymap: > > If keymap is linux,keymap, then this should perhaps be linux,fn-keymap. > Depends if we still think linux,keymap is Linux specific? I'm open for suggestions, trying to understand the pattern, these are specific to this binding I think if anything they should be google,fn-key and google,fn-keymap, similarly to the existing google,needs-ghost-filter -- no idea why function-row-physmap was not prefixed but I guess it slipped in and now it's not worth changing it. Would it make sense? Thanks, Fabio -- Fabio Baltieri