From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 20B9CC64EC4 for ; Sat, 18 Feb 2023 10:10:33 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229803AbjBRKKb (ORCPT ); Sat, 18 Feb 2023 05:10:31 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:53702 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229506AbjBRKKa (ORCPT ); Sat, 18 Feb 2023 05:10:30 -0500 Received: from mail-ed1-x52f.google.com (mail-ed1-x52f.google.com [IPv6:2a00:1450:4864:20::52f]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 820153668F for ; Sat, 18 Feb 2023 02:10:29 -0800 (PST) Received: by mail-ed1-x52f.google.com with SMTP id k5so2741810edo.3 for ; Sat, 18 Feb 2023 02:10:29 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=content-transfer-encoding:in-reply-to:from:content-language :references:to:subject:user-agent:mime-version:date:message-id:from :to:cc:subject:date:message-id:reply-to; bh=HF0DcS0ZS+01AvWrN70KNxaOHsB2n9jllNRuloPze0Y=; b=kl5doSpWXi3Kei5ZgZG95krT8GMZx2JqflbaScqDaqQydetrFulbPqz0JWomcjDEjX JcxfQ5ztullNQFpgR/zCaOwWMOXQO+O3m+oXhfsay0TaNbCDDkSXU+8y0W31n2WDMXu4 9aD0Hj4ihvgwTzGkACrNT1lvy7EagPt4ReyW6QOF4ArzUJsP1BAS/qv0y8qc2J+8ZSSI B0iG+37MK6cUqvk1JAA076ynwqYpGsMcSenViIZquQct0DUW7r2yInSgu4X0Ez2Lefa4 mpEw91OB44Gv8jn/E0mjeGbuZBjAqKLVvyIpd5e/QRCRWW0UGrQjarYAvzOUxMLZpDs3 kN4g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:in-reply-to:from:content-language :references:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=HF0DcS0ZS+01AvWrN70KNxaOHsB2n9jllNRuloPze0Y=; b=F9Sz9nE9cPsBFkIUJ6mqnL7BEUEDBpR/ISs6V6VnfSwNS5CDxUaDpWyGMacM/FEawO U8Dd0zagOnCP7NRzvFspz54fcZHlCQRYNO5vhku65r4IV4N7YV14WjItAR5ummevuxM/ JbBzmEMz1FyJ1bFigs4YQryHwvARzMa2G+g7tNSvcj/95nl76GHh+SJPyHgqYPg1eT+T uZatDxdaxNgFzOjl2PCI0NgYjvU75DGqi4LGYyqWHnAaozhzr8hOFfRKmD3UaSX2bOwD SZY5yeykzOKshAfG/mzThTlHVr4ggCr/Fero/15lgY9MY8VsHyb5vTXMStLJe4hheYNg 9w0A== X-Gm-Message-State: AO0yUKWUKy5E2tBeHPnOuJQYJvZ/kAL3IoZ0POpY7YHzZrVHsCcyjpgh Ry4taGimwBTg1pNaSw5HRjfh4g== X-Google-Smtp-Source: AK7set8Oz5135GMjU3IWKlGah+gmHtwLX1ikikSRnXZqIU/R0XwcVJQ4kDyRpW7TI+dkkdbXVFovNQ== X-Received: by 2002:aa7:c051:0:b0:4ac:d30d:c3a3 with SMTP id k17-20020aa7c051000000b004acd30dc3a3mr2891721edo.32.1676715028023; Sat, 18 Feb 2023 02:10:28 -0800 (PST) Received: from [192.168.1.109] ([178.197.216.144]) by smtp.gmail.com with ESMTPSA id e9-20020a50a689000000b004accf3a63cbsm3318177edc.68.2023.02.18.02.10.26 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Sat, 18 Feb 2023 02:10:27 -0800 (PST) Message-ID: Date: Sat, 18 Feb 2023 11:10:25 +0100 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.7.1 Subject: Re: [PATCH v5 3/5] ARM: dts: hpe: Add I2C Topology To: nick.hawkins@hpe.com, verdun@hpe.com, robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, linux@armlinux.org.uk, linux-i2c@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, wsa@kernel.org References: <20230217155054.99757-1-nick.hawkins@hpe.com> <20230217155054.99757-4-nick.hawkins@hpe.com> Content-Language: en-US From: Krzysztof Kozlowski In-Reply-To: <20230217155054.99757-4-nick.hawkins@hpe.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-i2c@vger.kernel.org On 17/02/2023 16:50, nick.hawkins@hpe.com wrote: > diff --git a/arch/arm/boot/dts/hpe-gxp.dtsi b/arch/arm/boot/dts/hpe-gxp.dtsi > index cf735b3c4f35..3bc071149bae 100644 > --- a/arch/arm/boot/dts/hpe-gxp.dtsi > +++ b/arch/arm/boot/dts/hpe-gxp.dtsi > @@ -122,6 +122,131 @@ > interrupts = <6>; > interrupt-parent = <&vic0>; > }; > + > + sysreg_system_controller: syscon@f8 { > + compatible = "hpe,gxp-sysreg", "syscon"; > + reg = <0xf8 0x8>; > + }; > + > + i2c0: i2c@2000 { > + compatible = "hpe,gxp-i2c"; > + reg = <0x2000 0x70>; > + interrupts = <9>; > + interrupt-parent = <&vic0>; > + #address-cells = <1>; > + #size-cells = <0>; > + status = "disabled"; > + hpe,sysreg = <&sysreg_system_controller>; > + clock-frequency = <100000>; clock-frequency is a property of the board. If it is fixed in the SoC, then make it fixed in the driver and there is no point for this property in DT. Best regards, Krzysztof