Received: by 2002:ab2:710b:0:b0:1ef:a325:1205 with SMTP id z11csp1938497lql; Wed, 13 Mar 2024 12:25:14 -0700 (PDT) X-Forwarded-Encrypted: i=3; AJvYcCULgwF7DaIQTaB0ufKGW9i/lifxQCZDSh9fQws+KX/zE4fMLIY78jE81xWd4nEOilJeFQd0YSHuX+jG7vK0lKe3l8zTHdkTCADrRShGaQ== X-Google-Smtp-Source: AGHT+IHRkkeq0xyw8VtUqzY7pJNtNmaIP5oKFhYD1BgoPWL8gWCVEJa9Zlu6Eo0uXM/KN5ij0cgY X-Received: by 2002:a17:907:d301:b0:a3f:33b2:5ce2 with SMTP id vg1-20020a170907d30100b00a3f33b25ce2mr8533663ejc.35.1710357913926; Wed, 13 Mar 2024 12:25:13 -0700 (PDT) ARC-Seal: i=2; a=rsa-sha256; t=1710357913; cv=pass; d=google.com; s=arc-20160816; b=rLbZsXUp5iYGsC/jgANHsyrTeJbv7EFde4h1PM4E3jBFxK66jyQJsSthbAhKW5g/tt KItjk45g6+G5V9mZwju3kZRW22FJ7GP4GBb9r6mXsHFlor4+TzRreU7fbGgTSEZXlT6f Xhj7Mz2VG2FVthLWVWGiOxvVk0wyviu9rGrmXkPG9a+mlaXQs5Uk9ONyGYyxGnXWVHQb jwICh22zbyVTsSuyV+nzDRfsuAeU80XiI7JFXkXlNkUfpQvkWtKFyhNSpf/XS+10PI9G g/RtqRmn7fMXq6xcIc+hZJe7nLjjUBq9HyYzeQnV8/l1Dmrjtlw9imMf9/wjDIniHrSh shoQ== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:list-unsubscribe:list-subscribe:list-id:precedence :dkim-signature; bh=NWiDzUL96PYX6KoSc2593RZofaceS0QVcGsXtgoFgXA=; fh=5VlX47cl2eUjTxh5WXkXuzKH/BrVR/2P+6NcIVNYYmA=; b=GXlri4fVV3hNR2aThTxrdyFd0VMWLjPrxOIv9EZdZW/VtL/l9k2R/DVdKHy4djRhph v5DcrTAqVzMC10XLmH81pWE1tQWOLuuwj3j1J9rS7vf2qc1TkDgou9RzgO1Tp2lUp1ls J60wqoF584JraVD9711fveiesfZZqzBXXlFLFn+YTF8Pu7O6aIJkQ6RnwbJssoOZ2Fk1 rKAmtabgKjt2e/MQ9dMNGM+bRHR7OPsn0/Vkzi3ffvDklnSKfYPyY8jTiDvqMUUuQtym Kov+L6FUAZ1eYYDMmi/adHLZAZVXEEFjswh4j9be8serLmWlT+V8uiKawuKeyIhLnJsn 44Ew==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=qHUsbeIX; arc=pass (i=1 spf=pass spfdomain=linaro.org dkim=pass dkdomain=linaro.org dmarc=pass fromdomain=linaro.org); spf=pass (google.com: domain of linux-kernel+bounces-102414-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) smtp.mailfrom="linux-kernel+bounces-102414-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from am.mirrors.kernel.org (am.mirrors.kernel.org. [147.75.80.249]) by mx.google.com with ESMTPS id c8-20020a17090654c800b00a466bb2e992si114962ejp.552.2024.03.13.12.25.13 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 13 Mar 2024 12:25:13 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel+bounces-102414-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=qHUsbeIX; arc=pass (i=1 spf=pass spfdomain=linaro.org dkim=pass dkdomain=linaro.org dmarc=pass fromdomain=linaro.org); spf=pass (google.com: domain of linux-kernel+bounces-102414-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) smtp.mailfrom="linux-kernel+bounces-102414-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by am.mirrors.kernel.org (Postfix) with ESMTPS id AB1CB1F250CE for ; Wed, 13 Mar 2024 19:24:54 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 5E71F4CE13; Wed, 13 Mar 2024 19:19:05 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="qHUsbeIX" Received: from mail-yb1-f179.google.com (mail-yb1-f179.google.com [209.85.219.179]) (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 D84EF22611 for ; Wed, 13 Mar 2024 19:19:02 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.219.179 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1710357544; cv=none; b=kmE1VqXLQGDCwYSAQUKbWMmja3i7Rwgz7F8drJ45un4E31mezBkGZjtwYbRb8T728cEAhMof0zGBbUS87Tk9zhuEwPxtXJxg+sWIWbIz3j1NXuNaqt9xoBy4zEWTbTWOcPkVviZJD67W5v8FuxUiHnrGR1QuJ5RFkaiDgwDVSSg= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1710357544; c=relaxed/simple; bh=bIUZn3aP0gfZBr2TfmWOdy8wk4pGXFbADHlpjXkL4XM=; h=MIME-Version:References:In-Reply-To:From:Date:Message-ID:Subject: To:Cc:Content-Type; b=C3jISgLmsyylPAKFmUWl5YOJjMBPPqw3vdbgwVWLl5dfnBNkxzds8759yQse04RN0g0rMRPYwl8PWluyiCSiBWXDjP7N/IEC5/EWPjzHzrUwZEpI1eA9sCOO8qT+d3VRBQ0zAjJELO/enkM0I+fXI8YxARwWul9RSSigANMGcM8= 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=qHUsbeIX; arc=none smtp.client-ip=209.85.219.179 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 Received: by mail-yb1-f179.google.com with SMTP id 3f1490d57ef6-dcc80d6004bso115537276.0 for ; Wed, 13 Mar 2024 12:19:02 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1710357542; x=1710962342; darn=vger.kernel.org; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:from:to:cc:subject:date:message-id:reply-to; bh=NWiDzUL96PYX6KoSc2593RZofaceS0QVcGsXtgoFgXA=; b=qHUsbeIX9fjZeadf6qPIg9HrtDL97qVRQb1DvOflxSxujLfuv6Bscn4iWz5ksGfnff /2WmJN8zpCbaYfbJVpVByBt4qkoZinv9l/1VBD9rXMK2BLHFpN3vsbiyJHqsZpyegkk2 aWwfmANWOWb7b2nUloSRo7ozK8C/XuvvReirQH401Wr2zMYYs/H3edtaXUr0ndTpzXXA AfYfAUfd4VTIGXpPUl3GIXW/MqCC+SYlVcienglIllweILeJ0YfPUHoE3prOvqN9I/NS 3gOlwYjxb8EaNsdUhrZurghLRS5eGY7/XvczdttRuZc2DQXudt8zOgkdlJoRy3FvyQu6 XHxg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1710357542; x=1710962342; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=NWiDzUL96PYX6KoSc2593RZofaceS0QVcGsXtgoFgXA=; b=Q3d1Q15kRWKwpL/SRNf8YvG+d56VgCqg9F4hh1TPaPHUFoJwHrGOfbZ/Tfv1al8zZb 5hg/wXT8KowcrL9QHpLEaUWvFM6q+hgfVlv8jQ5xEyBmJOrS4WljcM1MnyrOfvCMZl22 3okl/6iS4qUm5GrJkGsjpqim2MaY200qcENykiSnz2TztFcf6B4qf3ygI4YOys2ycpxE 5rn4M69FRGxMIMQYyc9Q7LotDQWVzI4c9rIK4rAiopaE1BMReJDJoI3OfPWZE8QZZFUz TOcNU8PTaIWl2ZIXOg2dL3VcEG4JANnZAkGZU+6WjjT+f8nrFzyn5Cc8q8+squOKtv86 cHww== X-Forwarded-Encrypted: i=1; AJvYcCWasQKIGO/DQ+VoRPYT2e1sR+AkUMlz/DBguNu3rzbQPj+B6qLLX6vq7lIMJBj5b3th/wR9ZAUB+CqoCj13P/4xRp4W2d8l828QXO3u X-Gm-Message-State: AOJu0YyVQxLV3NMf3u6/b4EZE3d+k0yBGW7dffz600Hzimn8mD+ndrz4 ktuJSKi4992x3Z5JhB5NeawCWIRj1Gh1CGGp3vCGhJwx3WWW0KT8XJBqo7s+9ZWwx6jAhXZlCUK gvBRpuqHsTsSVQo26p86bUQHkww0U7tfp3TpPEQ== X-Received: by 2002:a25:ac9a:0:b0:dc7:5a73:184e with SMTP id x26-20020a25ac9a000000b00dc75a73184emr3449403ybi.14.1710357541837; Wed, 13 Mar 2024 12:19:01 -0700 (PDT) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 References: <20240313-videocc-sm8150-dt-node-v1-0-ae8ec3c822c2@quicinc.com> <20240313-videocc-sm8150-dt-node-v1-2-ae8ec3c822c2@quicinc.com> In-Reply-To: <20240313-videocc-sm8150-dt-node-v1-2-ae8ec3c822c2@quicinc.com> From: Dmitry Baryshkov Date: Wed, 13 Mar 2024 21:18:54 +0200 Message-ID: Subject: Re: [PATCH 2/3] clk: qcom: videocc-sm8150: Add index based clk lookup To: Satya Priya Kakitapalli Cc: Bjorn Andersson , Konrad Dybcio , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Michael Turquette , Stephen Boyd , Ajit Pandey , Imran Shaik , Taniya Das , Jagadeesh Kona , linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-clk@vger.kernel.org Content-Type: text/plain; charset="UTF-8" On Wed, 13 Mar 2024 at 13:11, Satya Priya Kakitapalli wrote: > > Add support to look up for clocks using index instead of fw_name. Why? You are breaking compatibility with existing bindings. Also the commit message is incorrect. You are not _adding_ support. You are changing name-based lookup to index-based one. > > Signed-off-by: Satya Priya Kakitapalli > --- > drivers/clk/qcom/videocc-sm8150.c | 8 ++++++-- > 1 file changed, 6 insertions(+), 2 deletions(-) > > diff --git a/drivers/clk/qcom/videocc-sm8150.c b/drivers/clk/qcom/videocc-sm8150.c > index a0329260157a..2b788a03c5ed 100644 > --- a/drivers/clk/qcom/videocc-sm8150.c > +++ b/drivers/clk/qcom/videocc-sm8150.c > @@ -19,6 +19,10 @@ > #include "reset.h" > #include "gdsc.h" > > +enum { > + DT_BI_TCXO, > +}; > + > enum { > P_BI_TCXO, > P_VIDEO_PLL0_OUT_MAIN, > @@ -49,7 +53,7 @@ static struct clk_alpha_pll video_pll0 = { > .hw.init = &(struct clk_init_data){ > .name = "video_pll0", > .parent_data = &(const struct clk_parent_data){ > - .fw_name = "bi_tcxo", > + .index = DT_BI_TCXO, > }, > .num_parents = 1, > .ops = &clk_alpha_pll_trion_ops, > @@ -63,7 +67,7 @@ static const struct parent_map video_cc_parent_map_0[] = { > }; > > static const struct clk_parent_data video_cc_parent_data_0[] = { > - { .fw_name = "bi_tcxo" }, > + { .index = DT_BI_TCXO }, > { .hw = &video_pll0.clkr.hw }, > }; > > > -- > 2.25.1 > > -- With best wishes Dmitry