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 gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (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 83D4CECAAD3 for ; Mon, 19 Sep 2022 21:11:01 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 081D310E052; Mon, 19 Sep 2022 21:11:00 +0000 (UTC) Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by gabe.freedesktop.org (Postfix) with ESMTPS id 0476910E052 for ; Mon, 19 Sep 2022 21:10:55 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1663621856; x=1695157856; h=date:from:to:cc:subject:message-id:references: content-transfer-encoding:in-reply-to:mime-version; bh=7/VK3Iij77QucYQRGUG6FofaAqgoZhl67ggSi1axJ/A=; b=Igj906K+zv00Xlbev5Ty5RJlbQx4TYYxMmFzdt809vJVLpqNjZTdYVIp Karh2wJZ/Qv4NYzlgDpg84JrG7QbFA5+iJzoeQHO2SlcxK5xWdvhnv98n Qx4BDauToITEMzQpDfvIX4HSdLdcS406pXYcpxyVAt03na/q5/VtgeEuV JiXhRugtRL9Sj4LJ+q9+1PbfFGNbYQHBN6uAG6AwZf4Dq9I8PbqZESnei SijlEtSMMbwBo3JArOt/J29gNKewVuesEi8J2rbcvdEYO1GtZU/sZ+qfD jfyPYRDBFwD4kTNShTfbwwc1ZnuF/MAmNGhqK3KrbmXklr/5PfEcrYMkf w==; X-IronPort-AV: E=McAfee;i="6500,9779,10475"; a="286569936" X-IronPort-AV: E=Sophos;i="5.93,328,1654585200"; d="scan'208";a="286569936" Received: from orsmga007.jf.intel.com ([10.7.209.58]) by orsmga101.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 19 Sep 2022 14:10:55 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.93,328,1654585200"; d="scan'208";a="614143476" Received: from orsmsx601.amr.corp.intel.com ([10.22.229.14]) by orsmga007.jf.intel.com with ESMTP; 19 Sep 2022 14:10:55 -0700 Received: from orsmsx612.amr.corp.intel.com (10.22.229.25) by ORSMSX601.amr.corp.intel.com (10.22.229.14) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2375.31; Mon, 19 Sep 2022 14:10:55 -0700 Received: from orsmsx610.amr.corp.intel.com (10.22.229.23) by ORSMSX612.amr.corp.intel.com (10.22.229.25) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2375.31; Mon, 19 Sep 2022 14:10:54 -0700 Received: from ORSEDG602.ED.cps.intel.com (10.7.248.7) by orsmsx610.amr.corp.intel.com (10.22.229.23) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2375.31 via Frontend Transport; Mon, 19 Sep 2022 14:10:54 -0700 Received: from NAM02-SN1-obe.outbound.protection.outlook.com (104.47.57.44) by edgegateway.intel.com (134.134.137.103) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2375.31; Mon, 19 Sep 2022 14:10:54 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=VYqTi5HVV/PNXQFLtt6L29G03FjJT2aqTBf39ppB7PY6qkBaVEfop+EZ672hjHbyCvkLJGcorKxF04LVWXlmAWWHdamNcQflVVdlptKQRcsA2mfmXvjm37pMTC+DD79jY+EXgcaDQVOIrpSpeeT1zhafogMF2qnRnCJUNDCsc43EYRP/0vrSFiyyZdOTD9tXgJ48iHE/QLQLZtpvvDe675yZIjVjsaP35FA5Vak/j/dvhJNF1Lknh3TaDYHbp13adKAp1VTofqYKwihmy7PXZDJB2dV71TxKDPwarNRaQIpLpJpwX9sd+tspL77TH6m8fac+uV0GU5R5RrCfzEubNQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=44zPDb6+/dWDMrT62BcYZwOE9vRf8983hYAbFhKlX8A=; b=jPgd3ODdfSxd7J21BI4B5dCPTFnwKJ2RtgPNKjF45imvPuDkhwEvMaFry796LaZVZ8614J0JuXsxM8RWIm5qj4vLJOXdZrGa5BiiCXBAYzjfsfqefFCDJGSURZfS1AvjfzgAS3qjV71uGpcpeArd5oNcRh7W/SlrBPjHwNDwYWnYXXXcixmx7D5zLL1RUgmZuk2h/uC2U6QXkDYsj9dAmJMYOjRQUilyRnv/8Yem7QW/P8CVFeJfpOB80AJXUlrkGdfb/FmXpP6TmC/qClyYJ0tW7t7MoaOA/7avxbEByrIQ15chWUxWGV7oFq7dK2MPZviGKI6HxpwumnL96QgBtg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=intel.com; dmarc=pass action=none header.from=intel.com; dkim=pass header.d=intel.com; arc=none Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=intel.com; Received: from MN0PR11MB6059.namprd11.prod.outlook.com (2603:10b6:208:377::9) by DS0PR11MB6376.namprd11.prod.outlook.com (2603:10b6:8:c8::16) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5632.21; Mon, 19 Sep 2022 21:10:52 +0000 Received: from MN0PR11MB6059.namprd11.prod.outlook.com ([fe80::7ea6:6f6c:f2dc:cec7]) by MN0PR11MB6059.namprd11.prod.outlook.com ([fe80::7ea6:6f6c:f2dc:cec7%3]) with mapi id 15.20.5632.017; Mon, 19 Sep 2022 21:10:52 +0000 Date: Mon, 19 Sep 2022 17:10:47 -0400 From: Rodrigo Vivi To: "Navare, Manasi" Message-ID: References: <20220917004404.414981-1-anusha.srivatsa@intel.com> <20220917004404.414981-7-anusha.srivatsa@intel.com> <20220919194645.GA1061693@mdnavare-mobl1.jf.intel.com> Content-Type: text/plain; charset="utf-8" Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20220919194645.GA1061693@mdnavare-mobl1.jf.intel.com> X-ClientProxiedBy: BY5PR17CA0011.namprd17.prod.outlook.com (2603:10b6:a03:1b8::24) To MN0PR11MB6059.namprd11.prod.outlook.com (2603:10b6:208:377::9) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: MN0PR11MB6059:EE_|DS0PR11MB6376:EE_ X-MS-Office365-Filtering-Correlation-Id: d9e6511c-4fff-43be-af0c-08da9a836eb8 X-LD-Processed: 46c98d88-e344-4ed4-8496-4ed7712e255d,ExtAddr X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: +/fxx5l8pOxpAi3nYezL7J8AqtW6tDjOjMrlZ2ZnpgsvaEJj6ShLgGizmTWPpSHv2mcVywcjkj7waXjRRLHUjsUE4enJsJ8/+4M8a5vSroOr0zum40ci2RP9WcgOREdXZh0dcftdLXvLO2Bu0dKKWfACEVaa9ZkzUmIXnnDhgn1AokzVWc7Gn6f3KXsSCdaiqL742HCcscQ2O9/zI1aW+6kuTJrhLNs8SUQP5vfCDDgOoE5Brin2w+Z0Q9lKg6khaP/0kx3np8sisyxEDrwkKTvl/gyKIA4o62BrvmrHu3VNUa+2boe5pj0PnU0dWj8XkxvGgzUKWPtavomdRvbMP+wFDynjdgf5larvmxwn+7p2ZURI0Q4bc/NnFHy4kB0VqQgMCsKFiGachyHsG8a+UR5gmbUqteVQGdeALhLiBUX5kANxmTlHq1f17GaMRreAtDyY5roKZEb9V7ni+Vae2USuu7JmkIv1b9iYC36to97Kg6jn+aLse8G6BkOwSFiOBa1aBeIM9v0uZ7w5MSxEDAn72yzJ6oheV2ymRXcep4YNn7TsrFjf38VqZ8o5qhheq58LBoPuMzDQKp3/PmQpf6lX1nPVKqsj+IRzR6Fysdf7mjdyv4J3HlDhzRajZfqL6EIeGA6B1iOHuqI1i51bsAiNiKIcvtGkPGjkuQjMiony7xkPQwyka7u4gr64+jI2sdjZFsi3pQL0NZv+ZJbhc/6NsNdH9UJi6wudVDaevTkUlh9CI3cI7a9wCoIQRm/Y X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:MN0PR11MB6059.namprd11.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230022)(39860400002)(376002)(396003)(346002)(136003)(366004)(451199015)(6512007)(6506007)(26005)(2616005)(82960400001)(38100700002)(44832011)(478600001)(6486002)(966005)(2906002)(5660300002)(66476007)(86362001)(36756003)(186003)(6666004)(83380400001)(41300700001)(6636002)(4326008)(66556008)(66946007)(316002)(8676002)(37006003)(8936002)(6862004); DIR:OUT; SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?aERqaWZET3Nlb3MxNkNBZHZ6NWhraUlLU3lQUXhWYkhQUmlGRmVnRHpUQ3F4?= =?utf-8?B?d2RiMnV6clVZRVFpYVoyVCt0SWlQUWFROHRoSXgwRUJxb0JFa01OOXFyOG8x?= =?utf-8?B?QUJ0VWQvZXdOV2hXbm1kVDdHVk0vMEZsUTlNcnBUUWg2YnRnbFJxeWo0c3Ju?= =?utf-8?B?YnA0NkhvL0JtcUtlN2NyVXZZSUdqek04aTExK1A4TEVnc1c2eXl1bEJnNkkv?= =?utf-8?B?Mmo1bFdBYWo1cGVHc0NMZTVieDNuZ044ZVdJZkV1NWxYRkV4WExUaC9kNXVN?= =?utf-8?B?RVJGdk5Id21Ca2Ywdi9NQzBycTZ5ak9oTDcyVzRwVGJoRDkvUjcwRzE0aHFY?= =?utf-8?B?VXZjNUxpdUg1MzhiS0x4b1kwRk1HN0hEUWdqZlJZK1gvWDkrbHJOY3NoYnZI?= =?utf-8?B?c25uWVlZcnFDcHRISzI1TDhxazJ5UnVNNTRCQTRFcWNxb285Rzl2R0dKYlhN?= =?utf-8?B?dEZjeEl4K1YxWTZRVkdnWWluV3BLVVMveExWRDVRQXJkdTRockpLQ0FzRkFC?= =?utf-8?B?QkNwanFjeWVoZU1Jb3FNdmgwc3NOb1I5Q2JXNXRFKzJ0Vmg1RXMwRkR4aWwy?= =?utf-8?B?eEUxYnVqYmQwWDhjMEJBVUpwamc5YmZjZW00c2ZlYTltSTY1bUtac0w5VnhI?= =?utf-8?B?djVWbGczNnVhQVF1NW51QkZVY0hGeVRSY29OR01ubjQxVnN1cVAzZjRGdVBE?= =?utf-8?B?aVZMMW5Oa2xYZGsrNXRPVUFVTFhnSVo2dDRwUXBkRTFFRzZlQlhQUmdhcDZ3?= =?utf-8?B?MlZoY2NYZXp3clVva2ZIdk5iOEdXUzlzSXRWaFBDaFFnSFpTT2ZEYWd3UXZS?= =?utf-8?B?VGZtaTJYSHBkU0t5MldCekNmMHdPK2s2VDk2dXhuYnZvYndSZ2VOS1dUUlY1?= =?utf-8?B?SGw0ZEFvek9ZNUNBZ2d4cVZmYld0RytSUGwzR2xIU3ZYZ3NQWERmbXloVWhH?= =?utf-8?B?NTVuQW1NVmFObFJzYWhqWVgyd3IxclJGeGZ4NVZCRkhnTldlODl4VzNtTFRU?= =?utf-8?B?Uzdkb2pHVU5SU2pxWHVlT3gwemhRcDI1UGdTQXp4N3BPQXgxb3dFU1hSQWlQ?= =?utf-8?B?NXZHeUpicjc5MlpBY1ZDdFJYeW1tamtPNWdnbEdJajZUODkyVmlOWjZaMXl3?= =?utf-8?B?U3ViQjdvNzIzYVorOHA1WllrdU1nNkZmVGM3cHNGMm8xc05uczBzQkVjeGZq?= =?utf-8?B?NHpIQmRxRWxyYjhDNWRRMkRJRkY1TEExUUtFZ0FsYmtzN2NpR2prNWNLSklo?= =?utf-8?B?dzh4Q1NpWTFhWVpJNlAzcm9Ba1h4V24wVnhTQ3Z5ZVNod2xrOHMranJNWXZU?= =?utf-8?B?N2pZTkxYQU1qNk4wYjVDZW9mOTVLR3VwQS9IbE8rU3EwUi95RWdpYkN2VXZk?= =?utf-8?B?SGFXOS9maUk0UFROcUNMWkxmMXRKTDhDb3hEbkl5RWxnall1L2NyaTMrMnFV?= =?utf-8?B?SjBkbVVNTEhQQjVBTEN4T0VVSldtWC94U3pKR3BoU3B5WUkyMUt1b0YvRmhx?= =?utf-8?B?WkxTZzBrMGF0dGxobGIya3ZTOEQxWTkxbGsrNE9tdS9PV2lFR0JvTG8yTnZo?= =?utf-8?B?VkFqQ2xEMjBYb3VNODc5eGxRZVpRTUpjQUlKM2pEalNQc0dzK3IycnRlaVBz?= =?utf-8?B?MGhESEZtNkpUcHZQQlVQQUo3cTlmNEZQMEdCWTNiRjdCQ1BTVDJpSnpkSWsr?= =?utf-8?B?dDFDWk5pdnR1a2JrTzNaR0VwTGNKSkcxU0JaTGZMTzN6UWRwN3hOa1htdjBq?= =?utf-8?B?Y3hKWXZoZHE5TmhXVVVOSnZYR3VuS2wxaFpyQXQ3dUcvUVg3R2t3Z1BiVWR6?= =?utf-8?B?WnozUHJrNlZzM05ySTBITWdqN2w2TVZWYURoSjJOS0Z6b1JKN1h1T1N4b2hz?= =?utf-8?B?dmNUNjd3NFE2OWhIUlh0d28yeGliVTM4NUhDeDJLckt3ZitHSlg4cldFNEdi?= =?utf-8?B?b1VHOGpha0RJNXNDQU41T0hVNjlpRGFXemsxS2dZeENNU3Q2U1RFbzVvVEVH?= =?utf-8?B?T0ZqVEZRVk5lVDlVQ2N4RU5xaFFiYzAzT1ZWaEtUcUxvSzJFdU9aSjVqaldZ?= =?utf-8?B?ZU5QSDA4WUtpeE9jUVB2cGxYU0VodTV2U1BBajNvL2pETlBiK1U3RkJoYWF5?= =?utf-8?B?UHZlYUEzdW43blBGNFduMHpkVWNpcHRYd2hXTDBWVi9pYzRFb292bkNnOEJX?= =?utf-8?B?QkE9PQ==?= X-MS-Exchange-CrossTenant-Network-Message-Id: d9e6511c-4fff-43be-af0c-08da9a836eb8 X-MS-Exchange-CrossTenant-AuthSource: MN0PR11MB6059.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 19 Sep 2022 21:10:51.9878 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: NyyyA3izc/6QVPrnaM4xmcxRgN/MzATZflHnPI6wItSmUFObFIEJUYfBy1KjjrAp1VDEuwVkma7q5IJZ8Za1Aw== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS0PR11MB6376 X-OriginatorOrg: intel.com Subject: Re: [Intel-gfx] [PATCH 6/6] drm/i915/display: Dump the new cdclk config values X-BeenThere: intel-gfx@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Intel graphics driver community testing & development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: intel-gfx@lists.freedesktop.org Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" On Mon, Sep 19, 2022 at 12:46:45PM -0700, Navare, Manasi wrote: > On Fri, Sep 16, 2022 at 05:44:04PM -0700, Anusha Srivatsa wrote: > > Add a helper function to get stringify values of the > > desired cdclk action and dump it with rest of the > > cdclk config values > > > > Signed-off-by: Anusha Srivatsa > > Please add Suggested-by: field to give proper credits as per our > discussion. This applies to other patches as well to add proper credits > to other folks who suggested design changes/ fixes. > This needs to be followed as per the upstream patch review methodology. Having the suggested by is a good way to give the proper credits when the whole big idea and design was co authored, or started by the other developer. We really need to mind about it. However it is not a very good thing for all of the patches we have. Many ideas come out of conversation with many folks and we cannot simply start adding the suggested by list with all the names involved. There's always a threshold there that we should mind. And the official rule as reference is this one: "A Suggested-by: tag indicates that the patch idea is suggested by the person named and ensures credit to the person for the idea. Please note that this tag should not be added without the reporter’s permission, especially if the idea was not posted in a public forum." [1] As a reference we don't keep adding official suggested-by tags for any and all of the suggestions we receive during reviews. The simple name version in the commit message is what we traditionally uses for small things. Looking to the history of this series here I see that this patch looks like an evolution of the previous series with small increment and suggestions from multiple folks. Not sure if it is really worthwhile to have to add the official tag and start to ping everyone to get the ack if it is okay or not to add it. [1] - https://www.kernel.org/doc/html/latest/process/submitting-patches.html#using-reported-by-tested-by-reviewed-by-suggested-by-and-fixes > > > --- > > drivers/gpu/drm/i915/display/intel_cdclk.c | 18 ++++++++++++++++-- > > 1 file changed, 16 insertions(+), 2 deletions(-) > > > > diff --git a/drivers/gpu/drm/i915/display/intel_cdclk.c b/drivers/gpu/drm/i915/display/intel_cdclk.c > > index bc627daade3e..12f5e4d23245 100644 > > --- a/drivers/gpu/drm/i915/display/intel_cdclk.c > > +++ b/drivers/gpu/drm/i915/display/intel_cdclk.c > > @@ -1688,6 +1688,19 @@ static u32 cdclk_squash_waveform(struct drm_i915_private *dev_priv, > > > > return 0xffff; > > } > > Missing newline causing checkpatch error > > Manasi > > > +static const char *cdclk_sequence_to_string(enum cdclk_sequence cdclk_sequence) > > +{ > > + switch (cdclk_sequence) { > > + case CDCLK_SQUASH_ONLY: > > + return "Squash only"; > > + case CDCLK_CRAWL_ONLY: > > + return "Crawl only"; > > + case CDCLK_LEGACY: > > + return "Legacy method"; > > + default: > > + return "Not a valid cdclk sequence"; > > + } > > +} > > > > static void dg2_prog_squash_ctl(struct drm_i915_private *i915, u16 waveform) > > { > > @@ -2083,10 +2096,11 @@ void intel_cdclk_dump_config(struct drm_i915_private *i915, > > const struct intel_cdclk_config *cdclk_config, > > const char *context) > > { > > - drm_dbg_kms(&i915->drm, "%s %d kHz, VCO %d kHz, ref %d kHz, bypass %d kHz, voltage level %d\n", > > + drm_dbg_kms(&i915->drm, "%s %d kHz, VCO %d kHz, ref %d kHz, bypass %d kHz, voltage level %d, %s action\n", > > context, cdclk_config->cdclk, cdclk_config->vco, > > cdclk_config->ref, cdclk_config->bypass, > > - cdclk_config->voltage_level); > > + cdclk_config->voltage_level, > > + cdclk_sequence_to_string(cdclk_config->steps->action)); > > } > > > > /** > > -- > > 2.25.1 > >