Received-SPF: pass (google.com: domain of linux-kernel+bounces-26521-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
Date: Mon, 15 Jan 2024 16:32:36 -0600
From: Jeff LaBundy <jeff@labundy.com>
To: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Cc: Lee Jones <lee@kernel.org>, Rob Herring <robh+dt@kernel.org>,
	Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
	Conor Dooley <conor+dt@kernel.org>, devicetree@vger.kernel.org,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH] dt-bindings: mfd: iqs62x: Do not override firmware-name
 $ref
Message-ID: <ZaWyhLcS86Xlw0Vt@nixie71>
References: <20240115182042.1610134-1-krzysztof.kozlowski@linaro.org>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20240115182042.1610134-1-krzysztof.kozlowski@linaro.org>
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?us-ascii?Q?jw8eRONuZBzzcMAz4W0IwFb5ZOdJaoQjvi7cCDJv8bvdlOPB0dHEiKLCuh1W?=
 =?us-ascii?Q?T4znowcQURUFNq8ym/T3WlApvX+C80X08m7jNzIan4eQ2EobtYSJ0zkF/4jt?=
 =?us-ascii?Q?3jnztPf1V5bzFl3mis+CQT7SSAIqM8J8pmzuKprremzs2uUh2iKbFuL56+gO?=
 =?us-ascii?Q?42qkiYt6rSKbyVoxE6w4pZHhx6lrdenerZPjXYyMHlxh8qG9HKLTnokMlGX+?=
 =?us-ascii?Q?TFHx240XyhWch7YmQYuB030HsFlpCj8ScsrO8NuleMTgNgSJV2iD/I0RlDIz?=
 =?us-ascii?Q?JwxQotbY/7RG4ZYJ2jLu266QEN8L65kDPisAuiDI9ok2aC396O25dCZsnrHf?=
 =?us-ascii?Q?1zKx4DO/NUzrk0gu717PN7VnqR4cxh/IitW3EvI+R9Wektojd8SHjc0VEefG?=
 =?us-ascii?Q?2qJef7maavQhhWC+lRfzFpVeEEUzSn5MND1IYOzuEWrYX33kB+7HvhyPx7FV?=
 =?us-ascii?Q?6MnZwi3zoGvwpPin8hHTSD99wmwPO17bhPTAk4sZL3S7LqFJg2ThkUpe5u3U?=
 =?us-ascii?Q?nRkVrXFRdpf9YeRq0xbVCVuyA8mwyRidlEqjwqbsbUcMFIo17k8ff9Tgdiio?=
 =?us-ascii?Q?EwbpasguTf9FNdGgrA3NFUdHvkTOpgXbcJ2WT+RjgMM70q+IRp+5SGWNQR0c?=
 =?us-ascii?Q?SNr6ezBCQpSBS0jZJEGB4X4prf+qHBFVl28msCVpHhJVfvHSC5Wv6BI7mBOz?=
 =?us-ascii?Q?qCBPwZIFY5zxH4Vvchx8NK+F0iVHSs1LOZR4C+x89FyBYNRqsv4e375hy+jb?=
 =?us-ascii?Q?Hn8Iyg/oz6UVe2JgaQTZq5mG2Mr/j4iEiE40goVI5EZviWt1iaFZkn0nLvHm?=
 =?us-ascii?Q?N+FTGoa73QbqZgbLZ2KO5Yhs7zaGGe7eDhm2fgXvVTFUsX2N9TxtBiktOeyB?=
 =?us-ascii?Q?GIEGRO+SOkEm2R6WoJPhqQYpvw+0WMQMBvLIoMbmzlaG0ZviYCjM0P7lz9QV?=
 =?us-ascii?Q?a4Hrcg5wIwX23FDsDFYP/TE0Yz73Ssr78FB0u+Ajg4qksCw7I20U7oHsqEyL?=
 =?us-ascii?Q?/GFvx35t+S8cV/FTwG87Y4maP0Oj7VGh/n5psyLAVfmx6vfA5he0juCEQuX0?=
 =?us-ascii?Q?UOuEXToIy6Q4UGgSN0bsE/VWlFCrk631rlym/AxNJCtWINV/+cZACpbm8bSu?=
 =?us-ascii?Q?J4eqaWryrfr0TAn2uBvzN30Z9HJqr3jOyBQltfLSnMQTgEq0Fe/5xfxL7ykc?=
 =?us-ascii?Q?MpCtyrYn+1srPIV754YhM48BVTT3QQx0hTEv1sATIv6Yt3Gm2gY6CpeUi2XX?=
 =?us-ascii?Q?WL/c2Eb5zHJTddGNQCapVRtsaSfYucLSJPlJ/rEU8JTdq6F88BxwA2pdHAwg?=
 =?us-ascii?Q?CJVVNAWmiK6xmudGE+7Rmtzyv6vIS51HBFCAQykraDV7DCgfw8WlxYcNypWa?=
 =?us-ascii?Q?ieYtjS25+BIwm6qI76NaNC23gQ+raxeYm2tDSGnL16unCofzBlJVaHLJVjRK?=
 =?us-ascii?Q?C4poPqRF3FOXf+GirTMpKdonCXPebwolZy/yc+xgxh+TfHOhDSoC9IjBZrZp?=
 =?us-ascii?Q?L3auuqUa2PLQhuqKAycyYz8rLLBfTxfZ1c5Zf7eBlTscMTbGr2u+m5U5sO6k?=
 =?us-ascii?Q?iMljqdcVhnmWllsA4nweb5zckeYJ3sTvxlATl1wt?=
X-OriginatorOrg: labundy.com
X-MS-Exchange-CrossTenant-Network-Message-Id: fac17534-c64c-4ee2-e8b5-08dc1619e1ab
X-MS-Exchange-CrossTenant-AuthSource: SN4PR0801MB3774.namprd08.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 15 Jan 2024 22:32:40.4703
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 00b69d09-acab-4585-aca7-8fb7c6323e6f
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: Yb6mkjOUJ5I6FH14l1xj+ufL0RrY+0vKHKdKe4BnXGWAPMqdDW0PpploFUT2yqiOShHp/WznCpWp+zBN5uyaxA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MW5PR08MB8192

On Mon, Jan 15, 2024 at 07:20:42PM +0100, Krzysztof Kozlowski wrote:
> dtschema package defines firmware-name as string-array, so individual
> bindings should not make it a string but instead just narrow the number
> of expected firmware file names.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Acked-by: Jeff LaBundy <jeff@labundy.com>

> ---
>  Documentation/devicetree/bindings/mfd/iqs62x.yaml | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/mfd/iqs62x.yaml b/Documentation/devicetree/bindings/mfd/iqs62x.yaml
> index 044cd7542c2b..f438c2374966 100644
> --- a/Documentation/devicetree/bindings/mfd/iqs62x.yaml
> +++ b/Documentation/devicetree/bindings/mfd/iqs62x.yaml
> @@ -31,7 +31,7 @@ properties:
>      maxItems: 1
>  
>    firmware-name:
> -    $ref: /schemas/types.yaml#/definitions/string
> +    maxItems: 1
>      description:
>        Specifies the name of the calibration and configuration file selected by
>        the driver. If this property is omitted, the name is chosen based on the
> -- 
> 2.34.1
>