Received: by 2002:a05:6a11:4021:0:0:0:0 with SMTP id ky33csp2319483pxb; Mon, 20 Sep 2021 18:42:37 -0700 (PDT) X-Google-Smtp-Source: ABdhPJyZvSNFEdAXFsdBpUcE69cdei7aHX4pEv8SvHHrXqGZGtqVkZJgAa2MSejMh4A6ARWhT2ha X-Received: by 2002:a5d:9601:: with SMTP id w1mr3306189iol.209.1632188557489; Mon, 20 Sep 2021 18:42:37 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1632188557; cv=none; d=google.com; s=arc-20160816; b=IfuOmvRLinMl7mD627kBtgpx5HggWZzk5FZZ4gjDtEryrp+lEhDEoFiiFbM1XA9t7I pY/B0dCF0dCWG79kUa6C4LqGNjwPX4mm/4kKuacasCD6OWNbnJhRWFMo7Tichs6+cZBv QAGe5vcYtdtuBzwdFNX4BD6VHfmqoU7Ylxe1k268LkMnzZ6dwd78QsnVLKVWYdzZuj9x q510r9rvm3AgW9sU5lVAWt3Gy0TCfftgqGqE3Re9M4WJB6Pa44ZM8BNW9mrs+7AJpOcY zY7TqijSWcYKaIsutJ1SEtcYvlzfkaLZBsSlPSRiTxiDYmEuLuADpDl83xEqMf56vCGd CpEA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-disposition:mime-version :references:message-id:subject:cc:to:from:date:dkim-signature; bh=PdjbhjDKUH4VNwSZHwS+PmBefZ+LAhElGG9ogBWEbR8=; b=mAwKtyOS9jIryMf3Rzw+tye1Zrt7GS5ZLzkXJJrd0OLlr9/SVQOF+OOjU0sZhqrln2 8iD1lGdmmT/anuyEuHEMZ7FQ8t8Ih3ZD49oHnQfkUU+8aVVY3tNQhdePJjuU0TE6MEov KMlBC6AHQsYtiLWkue0W1hGR1RUV5ixa8WLBK9J6gUpLd+6Dx1EQ4b0+YU+0dwt+nM8M CtZMKvsPT6OcY99y0aht9ETz4aQk1bL3FTvy/A+hG3zcYBIYv9SVEpIy3ZDkrmH+55aS rEwUyiggwk1RtlsdTfTbrAWFfGHX87s85dcgO16CBUEb5ygYtwoLWnUfj6tD1shBRMNh hUAw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@lunn.ch header.s=20171124 header.b=4YYUBo3L; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id g19si9225095jam.54.2021.09.20.18.42.26; Mon, 20 Sep 2021 18:42:37 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@lunn.ch header.s=20171124 header.b=4YYUBo3L; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S241701AbhITQzE (ORCPT + 99 others); Mon, 20 Sep 2021 12:55:04 -0400 Received: from vps0.lunn.ch ([185.16.172.187]:50612 "EHLO vps0.lunn.ch" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S240476AbhITQuw (ORCPT ); Mon, 20 Sep 2021 12:50:52 -0400 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lunn.ch; s=20171124; h=In-Reply-To:Content-Disposition:Content-Type:MIME-Version: References:Message-ID:Subject:Cc:To:From:Date:From:Sender:Reply-To:Subject: Date:Message-ID:To:Cc:MIME-Version:Content-Type:Content-Transfer-Encoding: Content-ID:Content-Description:Content-Disposition:In-Reply-To:References; bh=PdjbhjDKUH4VNwSZHwS+PmBefZ+LAhElGG9ogBWEbR8=; b=4YYUBo3LWOkun1WSOq+O7nvmwk 2xEPNxsfzRcoMXscHpRtTV7sLQRvzMq6P4No3bUtCQqO+qc4YYBcTMcGOWuVEfXDBAdb6vOOlvi7M BlgMG+VonLvGr+bx9L0M9NJ8zroiBb+oPurmvlMj3/ix99ajjzZCyWBSv3ZQm+ro7qXs=; Received: from andrew by vps0.lunn.ch with local (Exim 4.94.2) (envelope-from ) id 1mSMTo-007WD4-4F; Mon, 20 Sep 2021 18:49:20 +0200 Date: Mon, 20 Sep 2021 18:49:20 +0200 From: Andrew Lunn To: Min Li Cc: Jakub Kicinski , "richardcochran@gmail.com" , "netdev@vger.kernel.org" , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH net v2 2/2] ptp: idt82p33: implement double dco time correction Message-ID: References: <1631889589-26941-1-git-send-email-min.li.xe@renesas.com> <1631889589-26941-2-git-send-email-min.li.xe@renesas.com> <20210917125401.6e22ae13@kicinski-fedora-pc1c0hjn.dhcp.thefacebook.com> <20210917140631.696aadc9@kicinski-fedora-pc1c0hjn.dhcp.thefacebook.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Sep 20, 2021 at 03:13:14PM +0000, Min Li wrote: > > > > > On Fri, Jun 25, 2021 at 02:24:24PM +0000, Min Li wrote: > > > > How would you suggest to implement the change that make the new > > driver behavior optional? > > > I would say, module parameter or debugfs knob. > > > > Aright, in which case devlink or debugfs, please. > > > Hi Jakub > > The target platform BSP is little old and doesn't support devlink_params_register yet. > > And our design doesn't allow debugfs to be used in released software. That does not matter. You are not submitting code to your BSP, you are submitting to mainline. Please use the features mainline provides. Andrew