From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-lf1-f46.google.com (mail-lf1-f46.google.com [209.85.167.46]) (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 A3B2B443313 for ; Wed, 19 Aug 2026 11:10:56 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.167.46 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1787137860; cv=none; b=pZuj2nNW+XmFyo9GBX2k/5FcquB4Up3xgGNY4hh/+JASuV1A5P17DqpjNW+WUnqdsLIcrghl0utO1NN6wUe67tbksHZvYYQVgcNTIDhT8+UgZk8WbJVRVUpQvnNS83/8MjbruwPRuf08ZQNCq2Wu91VOIgvhbmKk/yZ1tVZsgLY= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1787137860; c=relaxed/simple; bh=T8G0ZcWK/K84AczPHDRHSzzAJA9uLSt710Kf5KRx7gc=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=Bp5l2CnOjexnGbXSGOsi5GaD1rdHFZFODbqcqkoyKpcx6j4yScCp5ZkfaEjLXlJ4gVcPj3YqcN/DOpuv7enDTvRLfvYassg9OtqZTc2NKL5sjXcXU+LS7VqYgnevV/KsTYP4a/33PodvQUl8V5pqq/kkCcvX9bnaibdZ+OZKjFk= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=Jrda/MTn; arc=none smtp.client-ip=209.85.167.46 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="Jrda/MTn" Received: by mail-lf1-f46.google.com with SMTP id 2adb3069b0e04-5aebe49b227so184959e87.2 for ; Wed, 19 Aug 2026 04:10:56 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1787137854; x=1787742654; darn=vger.kernel.org; h=content-transfer-encoding:content-type:in-reply-to:from:references :cc:to:subject:user-agent:mime-version:date:message-id:from:to:cc :subject:date:message-id:reply-to:content-type; bh=MHM+2qgBqc21q4hkz2G6UjCV2/ZpXTwtrV94xxu9kWA=; b=Jrda/MTnlQALPEJSlfv8bjyXT0hAyNcBr2EIVHbFG1G907aMrSl+BCIvo4nhAu/6US SaULGUGaFI0DwNk71wkl96dB3XOIylvx4ORl2D1G1qquBebUPkrKRIjfUqhZgtma+vl+ YH/IC5yXorE6YbsYxe7fr1U3PVgtdtJZhByxaxlsjt+PQkmuKSs5ZKHZIDn0xzySTIaf wQ0n92dViXPpfM4afSOfxwkvlYtI3yYKPUezpeQgnnd9u5n/XBxTiSY0GY1GjgE8U/u4 RL2IN2aGznaypEtAx05iR0lDlKm2KZgHDFns27GgD1+chxuS4YvUVZ77w0gxvm1/xvDQ VQuA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1787137854; x=1787742654; h=content-transfer-encoding:content-type:in-reply-to:from:references :cc:to:subject:user-agent:mime-version:date:message-id:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to :content-type; bh=MHM+2qgBqc21q4hkz2G6UjCV2/ZpXTwtrV94xxu9kWA=; b=bmX6rUemHUGq/hNOFcAPgAKsMJlk1Ixx1qLd7Sk6Jrsk7CZXS/qQWLEDBi4O9jOL77 U/y0nXLOw8+R8V9bE4tgqga4zSzhBCnUoDBIZLbIocC17RrD6pFnG0uLvWJ6UQ5HrwwL 7jwbPbwv+X2aHG0vC5hdIagZNPmNnPhQxoY4Ij23YlOleUOvvrsXZSsKjQKghIz7wXnn dnypl6bD+ig/GlTjunSI2KD7hJ7iZisBaK1eF8gWGlihKVqCQebxZX8kj7ZXyB2XepHm 0Jnwk5GrfsMkcmTatbwAcSHNhOTGwT2lsPTlJAtgikgRkIXPIeHkRlsBc3aYx7UXkigx GV2A== X-Forwarded-Encrypted: i=1; AHgh+RrjiLhCb6BoUxAE/Qa/nPl9iWouXZ15rYznGxpkQodeYfVQl3jrBwOg2eIuZx2J2OinBp8PICv6YKxp59E=@vger.kernel.org X-Gm-Message-State: AOJu0Yy+ThWTXmQDfyA1AvEsVOm1Ll7c0FvDUzk59MVXODU648OjBAHG PtTqq4Wj2TINbrTkeDuC/QK/E68tGaw82DMw2176IM0nvMld4BrCoN3R+wjrMukWo9U= X-Gm-Gg: AR+sD11HCP9qYBIcDh/cYf13kSDsj8cqnHDLoQJRFVKJbMFux3qcHljv5/W7v0u0DnV JYYYhr/kknxxionMtWhehZjB0u7kS9N9Plw7c0/c/SpcNluIqZ8OjQwxduTKIYcSAvpcNWlumFW 1D4vkIW9yc0rbtg/maGR2wM8ZqliA2joDlgh/7DQbI62MC4OSU2RgT6cn/rTNnszrt7fJtdYba7 RcwXqsg+Is8ve/qSgtJO3Rg/OOycL6dwHLR86QUvDrh63XlGQCkfNwiQLMh0PeIOrYabUg2S1pw c/eLG1TIhdzlqP0MP/kooEokzVhMbfqNvmhm20RbrAeLh1qzHxqseohudcxJQCXAh0HHlz3aVv2 j2edsVvlXx7O9olJ5E9Hg7jlP0eQyjmRNmG/6XyTP0SXChPXCBG1dy9hE7S3GwMkb1dLEzSXqjN RFg+y+qM3qq+1asNbbZZsBd8cb2tcUrkOBQfKFgzDPDdU8RNNyR7HlfwpXhR5Ugy0LTSO+U+ib7 ZfIpg8yr+3fmXkL3UeWpvG0hJJ/LAIM3bxxhkSY1A== X-Received: by 2002:a05:6512:68f:b0:5b4:59a3:c67 with SMTP id 2adb3069b0e04-5b478d15f12mr694543e87.3.1787137854067; Wed, 19 Aug 2026 04:10:54 -0700 (PDT) Received: from [192.168.1.100] (91-159-24-186.elisa-laajakaista.fi. [91.159.24.186]) by smtp.gmail.com with ESMTPSA id 2adb3069b0e04-5b4788fa30bsm433544e87.82.2026.08.19.04.10.53 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 19 Aug 2026 04:10:53 -0700 (PDT) Message-ID: <62bf3468-a485-4255-80e5-76997398431f@linaro.org> Date: Wed, 19 Aug 2026 14:10:52 +0300 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v4 1/5] i2c: qcom-cci: Switch msm8953 to the CCI v2 timing/rate config To: Loic Poulain , Robert Foss , Andi Shyti , Wolfram Sang , Dmitry Baryshkov , Luca Weiss Cc: linux-i2c@vger.kernel.org, linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org, konradybcio@kernel.org, stephan.gerhold@linaro.org References: <20260801-cci-clk-fix-v4-0-e1d80da54e01@oss.qualcomm.com> <20260801-cci-clk-fix-v4-1-e1d80da54e01@oss.qualcomm.com> From: Vladimir Zapolskiy In-Reply-To: <20260801-cci-clk-fix-v4-1-e1d80da54e01@oss.qualcomm.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit On 8/1/26 23:10, Loic Poulain wrote: > The msm8953 CCI timing table is internally inconsistent. Its Standard > and Fast timings match v1/v1.5, which are calibrated for a 19.2 MHz CCI > clock, but its Fast+ timings are essentially the v2 values, which are > calibrated for 37.5 MHz. Since all masters share a single CCI clock, > no single rate can satisfy all three modes with the current table, and > the DT assigns 19.2 MHz, so Fast+ timings are wrong. > > The msm8953 CCI is the same hardware version as msm8996/sdm630, which > already use the cci_v2_data config (37.5 MHz). 37.5 MHz is supported by > the msm8953 CCI RCG, so reuse cci_v2_data for msm8953 as well and drop > the redundant, inconsistent standalone table. This makes all three I2C > modes self-consistent under a single clock rate. > > Note this requires the CCI clock to run at 37.5 MHz, the driver selects > and enforces the proper rate in following patches. > > Signed-off-by: Loic Poulain Reviewed-by: Vladimir Zapolskiy -- Best wishes, Vladimir