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 35CDDC77B7A for ; Mon, 17 Apr 2023 15:54:12 +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-Type: Content-Transfer-Encoding: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=XYXoEV3kDU7/l2X1m0uV/yrFMiETEUhq2miLRegvgKI=; b=gM5EPk4srQYfDa bAG+9WJbv+XlINbz3mCu1gilRDjRF048IFxT2fywq/+tsvXXcBQzvf8/MeR5NZSXJW/IVeIP12yev q+D8s/5KKLOc0AjLRTN2QX6S1LwDpVFehlvIAoD/2zZxQ0pD/yZRl2ZmedS+OEtDhaWR7heZqPsWr 4NSRbGGaIaQZsl6x8gf5aR3tri0pBh5SK6yskSZakIwhDB7JJ6zP+ytxiq6gb0kTkVKYWq+uhkyDy A9GeMdhMeUEZUomKLIGlIN4Xw/IAZOo0RFg9KDDWwt6ZdZFjFc5HybFQ6rulP7krkoQggQ+Tjuket O5GMvegX8R1fUKwTvR4A==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1poRAP-00GxdL-1q; Mon, 17 Apr 2023 15:53:21 +0000 Received: from out-36.mta1.migadu.com ([2001:41d0:203:375::24]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1poRAL-00GxaU-2v for linux-arm-kernel@lists.infradead.org; Mon, 17 Apr 2023 15:53:20 +0000 Message-ID: DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.dev; s=key1; t=1681746786; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=b3YsTgVO4RylBnjdLN7yJQDgG+k90LtVzdRN7p16/bg=; b=H8DpitgZjbMn5bOEkl1h0l7Gdv4mbNJnD/cWezPSke1ZG9UuUiTqRr5enkiszSqnJKdoUa DLvxS42YJIaYOaNGiavj3qoyCwsjMad+fdE+YVO9BDAU5531xehe3DmlzKfGonNnH7uPcI K9ebqmGgUfFoCMrGaxQUR52fplNcT70= Date: Mon, 17 Apr 2023 16:53:00 +0100 MIME-Version: 1.0 Subject: Re: [PATCH RFC v6 2/6] dpll: Add DPLL framework base functions To: Jiri Pirko , Jakub Kicinski Cc: "Kubalewski, Arkadiusz" , Vadim Fedorenko , Jonathan Lemon , Paolo Abeni , poros , mschmidt , "netdev@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "linux-clk@vger.kernel.org" , "Olech, Milena" , "Michalik, Michal" References: <20230312022807.278528-1-vadfed@meta.com> <20230312022807.278528-3-vadfed@meta.com> <2b749045-021e-d6c8-b265-972cfa892802@linux.dev> <20230403111812.163b7d1d@kernel.org> <20230410153149.602c6bad@kernel.org> Content-Language: en-US X-Report-Abuse: Please report any abuse attempt to abuse@migadu.com and include these headers. From: Vadim Fedorenko In-Reply-To: X-Migadu-Flow: FLOW_OUT X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230417_085318_536875_D87D403F X-CRM114-Status: GOOD ( 12.39 ) 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-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 16/04/2023 17:23, Jiri Pirko wrote: > Tue, Apr 11, 2023 at 12:31:49AM CEST, kuba@kernel.org wrote: >> On Sun, 9 Apr 2023 09:51:48 +0200 Jiri Pirko wrote: >>> Wait, not sure you get the format of the "name". It does not contain any >>> bus address, so the auxdev issue you pointed out is not applicable. >>> It is driver/clock_id/index. >>> All 3 are stable and user can rely on them. Do you see any issue in >>> that? >> >> What is index? I thought you don't want an index and yet there is one, >> just scoped by random attributes :( > > Index internal within a single instance. Like Intel guys, they have 1 > clock wired up with multiple DPLLs. The driver gives every DPLL index. > This is internal, totally up to the driver decision. Similar concept to > devlink port index. It feels like a dead-lock in conversation here. We have to agree on something because for now it's the only blocker to post the next version with all the comments from the previous one addressed in the code. My position here is that I'm ok to have any of the properties being an identifier as well as keep both of them, the code already has all the lines to support any decision. I just to want to go back to this part again in the next iteration. _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel