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 4105FC4332F for ; Tue, 13 Dec 2022 15:44:34 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S235733AbiLMPod (ORCPT ); Tue, 13 Dec 2022 10:44:33 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:41894 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S236240AbiLMPoU (ORCPT ); Tue, 13 Dec 2022 10:44:20 -0500 Received: from mail-lf1-x12c.google.com (mail-lf1-x12c.google.com [IPv6:2a00:1450:4864:20::12c]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id C9DD4D5A for ; Tue, 13 Dec 2022 07:44:18 -0800 (PST) Received: by mail-lf1-x12c.google.com with SMTP id x28so5554815lfn.6 for ; Tue, 13 Dec 2022 07:44:18 -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:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=EGstHtifQPW5GO3v7hxlfERswPjhZXfr0wahKT87Wbg=; b=cdQmXOl+woEiaypum/Th1omypryAPzX//HQ/hCldqepnH8fUd7OI4/+pAsXuQKfgcR OhWVHeeAqCzRuYcp0xNRJ2A9bAB/QkkGQRfV3/uBqMDOQInUS3MaUdL1r8G+htDpL48E HUHqY4E9T3VIyRjmf1z8VxCjxZ59ngpflZKhpjLKvvSmJ1i8TaNbMZgQMb/xoRycEjOX V/sAAnni9mcHle0mlarWzd8w159pDGIrydPsokHSBiCraVtx0/1+33iCy+Cxt0KWlGuh 4OC+dM8kwPHmYQpbllKuc9V7vFid/Ep6Wb/oF6PsMT8KrrMN2KQaA52v5XXZrYXYg0GT epew== 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:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=EGstHtifQPW5GO3v7hxlfERswPjhZXfr0wahKT87Wbg=; b=53hg/56L4gmnIoY4yeQGx1qiUwUj6pFU73PP8qu/7gWPyWjucqaX+Pi9ZaYe7jyAsh ZhgHdRsPgx2XZMv5TxCKB/32jnhoUkpwTvEADeWyX/Qi0aFY7ZG17qWTWqttH0t9Npwl et5dxzLEPjQqTxhBikFq44cuDuVW38QJj/ofhAoHXUfElpOk2WvPnrCN12+aS5cmDgv2 R3DG3l6liLH19tuzt5RlkU/HAj7VVTVoLz2/xD2I8jn45rJjGHZ++mRtJfOmYU5JcFTx XEHe6CuHcXlWW9AU4QWRY8zQ+OeTRgQfVVrcKElYagvDr8Z+aS+TnI+s1I2vEijHxE6+ bCzA== X-Gm-Message-State: ANoB5pkr6+R/b5pStarcIHDiMuaODlO89XMsNZoOHjoYFl3eE96ZlmyV +bHh3fRcYpRNIjPlnslDFAgenw== X-Google-Smtp-Source: AA0mqf5/eioldK0Ihu8+1CQheeMf4HYHScxc0xfuWrWmdsCn7jqwdvf58iRG6fQe109pDxdiJo79lg== X-Received: by 2002:a05:6512:298c:b0:4b5:b2b1:69a9 with SMTP id du12-20020a056512298c00b004b5b2b169a9mr4642149lfb.20.1670946257217; Tue, 13 Dec 2022 07:44:17 -0800 (PST) Received: from [192.168.1.101] (abxh44.neoplus.adsl.tpnet.pl. [83.9.1.44]) by smtp.gmail.com with ESMTPSA id e13-20020a05651236cd00b00498fc3d4d15sm411410lfs.190.2022.12.13.07.44.15 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 13 Dec 2022 07:44:16 -0800 (PST) Message-ID: Date: Tue, 13 Dec 2022 16:44:15 +0100 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.5.1 Subject: Re: [PATCH 1/4] arm64: dts: qcom: sc8280xp: rename i2c5 to i2c21 Content-Language: en-US To: Johan Hovold , Shazad Hussain Cc: Brian Masney , andersson@kernel.org, krzysztof.kozlowski+dt@linaro.org, robh+dt@kernel.org, johan+linaro@kernel.org, linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, ahalaney@redhat.com, echanude@redhat.com References: <20221212182314.1902632-1-bmasney@redhat.com> <20221212182314.1902632-2-bmasney@redhat.com> <03252ccd-e200-a112-b4dd-ba9fbf25c480@quicinc.com> From: Konrad Dybcio In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org On 13.12.2022 16:42, Johan Hovold wrote: > On Tue, Dec 13, 2022 at 04:39:54PM +0100, Johan Hovold wrote: >> On Tue, Dec 13, 2022 at 09:04:39PM +0530, Shazad Hussain wrote: >>> On 12/13/2022 8:58 PM, Johan Hovold wrote: > >>>>> So qup2_i2c18 becomes qup2_i2c2. Would I use the flat naming scheme for >>>>> the alias like so? >>>>> >>>>> aliases { >>>>> i2c18 = &qup2_i2c2; >>>>> } >>>> >>>> Or perhaps the i2c controllers should use a zero-based index instead of >>>> being named after the serial engines (e.g. as we do for the console >>>> uart). >>>> >>>> How are they named in the schematics? >>> >>> We should use from 0 to N. >> >> With N being 23 after the number of serial engines, or the number of >> available i2c buses on a particular board minus one? > > Looks like the more recent Qualcomm platforms use aliases that reflect > the engine number (i.e. 0 to 23) for i2c and spi. IMO it makes the most sense, as it tells the userspace "hello, this device is connected to the physical I2Cn on the SoC" as opposed to "hello, this device is connected to the nth enabled bus on this particular board". Konrad > > Johan