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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id A9B20C77B78 for ; Wed, 3 May 2023 07:41:31 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:From:References:Cc:To: Subject:MIME-Version:Date:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=hJO9HLmggDaMLp7KJltRo/G1Ro1rP5WSFCRWnjluBFg=; b=vSFcbS3QBcd3Pf cun9T5LybzCRaKiP8hmulDXaA5Fuj+M8SL566VoaR1HtEaLGTfIAniV2Slp1XX5XVYQOKLCJjOYJy TBHbNS/OrCAnL09+OK0PgoJeh7F/CpCCN5iOzVCen4ND7P76i0iMR0RQGQcb8LERGMqk/urolRj9C 2JWdoARytbiXBpG3vS4/FpNUBwfWT9AwKstZeOAIxIjWiHXM3FIfUQ0AbtGdOy/dLThP0hGlKkcrO ki2WKNyC1xzbrSUoEhe/6LvScNv1wjIvuGEooNRAr1v3rIZXZTTpyMfqTjG3gy4S9DSwcOWqUEDoU rYJHsI8qfLYrrM2s99xw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1pu76G-003gyw-12; Wed, 03 May 2023 07:40:32 +0000 Received: from mail-ej1-x62e.google.com ([2a00:1450:4864:20::62e]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1pu76D-003gxJ-3B for linux-arm-kernel@lists.infradead.org; Wed, 03 May 2023 07:40:31 +0000 Received: by mail-ej1-x62e.google.com with SMTP id a640c23a62f3a-9505214c47fso950125466b.1 for ; Wed, 03 May 2023 00:40:28 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1683099626; x=1685691626; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=+B6qBFwYGEdx2I+AJuVx8f7CeJtBz3CE79fX6H+gKfI=; b=xShLfHY0es50xrc63DRoABgtmTjJwmiqZjM4+7x7OtkViNpZcTx55XN34AVsNvudlG 85uyRNZEMXzOXYEvUIGRWfIrNm7E/lJvXjYbiZs0RgY407M6AZOiUe/iXvwYeTUj9dZt C8+JfbMBBzPMV+O2avRg1YKOmU0R5WnlZqF1h8zoTXbG/O6DnuwA1Vqu6oYT17aL54z1 QD+c6MSI4NbStxlhDxssJvOOicMaW0jMSSD2off5ALklBIUMYnwRlr8qOIdu5Se8ZK7p qVDdK73lAr3klZ7BiQlfkcIv+yZGk15BLY/+A4ulmB56yFStfVLFUkWO/ojd77aksjTF tN+w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1683099626; x=1685691626; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=+B6qBFwYGEdx2I+AJuVx8f7CeJtBz3CE79fX6H+gKfI=; b=BlzPdmhykVisDeL+/bY+fkgejFb2cvHi4NXA73pMG/WioWWJSUi/PcarBBdzH4MUUc n0wnPTdSr42srmKNj0Jol1E5uOUyDH41tvW2s+8k8L1HLLFBx4SYY/qjNGoHssxTjJLu gCP0d5uLT9SYo4QPn1XEtOsTlhMACkuhkqMIOEhTgidtXJzFazdA56Z0MCcJoBkrHC2c hiZdlgejfOHPW5upyC4G0LMxnR/SrWzP6On0JrDUrBJzR+M7/2/bF8IlQ9eeYP+TvN/b sBDPT73/j7tqEkBc1DkIpwwujzufVFkZQJb06JWVAZNEMR8Mnkky4ZGH1UXJKAWQ1qEm Izvg== X-Gm-Message-State: AC+VfDw2JvjDJ0v6/069hFaVhw1wzYKcmBp88zDUIOQNI9ZCSAgOCwVr xbTffJdz0KGq63Awcoy/pIh8/w== X-Google-Smtp-Source: ACHHUZ7GYFsin08QNS3ECdYX6FCdM1vCNW99xuijP7899juVk+jO7Vx0Oh3Isj80fqCZbvKbFmhNeQ== X-Received: by 2002:a17:907:9722:b0:889:5686:486a with SMTP id jg34-20020a170907972200b008895686486amr2268005ejc.30.1683099625870; Wed, 03 May 2023 00:40:25 -0700 (PDT) Received: from ?IPV6:2a02:810d:15c0:828:a40b:9d8:1fa0:ecc5? ([2a02:810d:15c0:828:a40b:9d8:1fa0:ecc5]) by smtp.gmail.com with ESMTPSA id h13-20020a170906718d00b0095386574c81sm17032945ejk.51.2023.05.03.00.40.24 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 03 May 2023 00:40:25 -0700 (PDT) Message-ID: <89276de5-ded8-2f38-bcea-708fdd35ca6d@linaro.org> Date: Wed, 3 May 2023 09:40:23 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.10.1 Subject: Re: [PATCH] ARM: zynq: dts: Setting default i2c clock frequency to 400kHz To: Michal Simek , linux-kernel@vger.kernel.org, monstr@monstr.eu, michal.simek@xilinx.com, git@xilinx.com Cc: Varalaxmi Bingi , Krzysztof Kozlowski , Rob Herring , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org References: <4dde5d1eb8e4572dae4295a19a4c83002a58e5da.1683035611.git.michal.simek@amd.com> <958f4336-8c38-4cc1-4313-06a54e397d42@amd.com> Content-Language: en-US From: Krzysztof Kozlowski In-Reply-To: <958f4336-8c38-4cc1-4313-06a54e397d42@amd.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230503_004030_038609_F8AA5543 X-CRM114-Status: GOOD ( 23.58 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 03/05/2023 09:27, Michal Simek wrote: > > > On 5/2/23 22:13, Krzysztof Kozlowski wrote: >> CAUTION: This message has originated from an External Source. Please use proper judgment and caution when opening attachments, clicking links, or responding to this email. >> >> >> On 02/05/2023 15:53, Michal Simek wrote: >>> From: Varalaxmi Bingi >>> >>> Setting default i2c clock frequency for Zynq to maximum rate of 400kHz. >>> Current default value is 100kHz. >>> >>> Signed-off-by: Varalaxmi Bingi >>> Signed-off-by: Michal Simek >>> --- >>> >>> arch/arm/boot/dts/zynq-7000.dtsi | 2 ++ >>> 1 file changed, 2 insertions(+) >>> >>> diff --git a/arch/arm/boot/dts/zynq-7000.dtsi b/arch/arm/boot/dts/zynq-7000.dtsi >>> index cd9931f6bcbd..a7db3f3009f2 100644 >>> --- a/arch/arm/boot/dts/zynq-7000.dtsi >>> +++ b/arch/arm/boot/dts/zynq-7000.dtsi >>> @@ -149,6 +149,7 @@ i2c0: i2c@e0004000 { >>> clocks = <&clkc 38>; >>> interrupt-parent = <&intc>; >>> interrupts = <0 25 4>; >>> + clock-frequency = <400000>; >> >> I think this should be rather property of the board, because it depends >> on I2C devices, right? > > We have 15+ boards with i2c connected and from this count I am aware about only > one configuration which is not working on 400k. > It is a question if make sense to update all boards (except for 1 and not all in > kernel yet) to 400k or just say 400k is default and if there is any issue use > lower frequency. > Obviously this patch is setting up 400k as default and let "problematic" boards > to setup lower freq. > The same approach was used for modepin description. > > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/arch/arm64/boot/dts/xilinx/zynqmp.dtsi?h=v6.3&id=53ba1b2bdaf7f481fdd878e9c18cd0e54081fac9 > > The next point is that 100k is default in Linux but doesn't mean it is default > in other projects. > > But if you think that it must be board property I think we should setup 100k as > default in this file too to make it clear. No, if it is conscious choice, it's fine. Best regards, Krzysztof _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel