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 C7EE8C4708E for ; Wed, 7 Dec 2022 13:19:29 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229691AbiLGNT2 (ORCPT ); Wed, 7 Dec 2022 08:19:28 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:51804 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229554AbiLGNT0 (ORCPT ); Wed, 7 Dec 2022 08:19:26 -0500 Received: from mail-ej1-x636.google.com (mail-ej1-x636.google.com [IPv6:2a00:1450:4864:20::636]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id A71311277C for ; Wed, 7 Dec 2022 05:19:25 -0800 (PST) Received: by mail-ej1-x636.google.com with SMTP id kw15so10479341ejc.10 for ; Wed, 07 Dec 2022 05:19:25 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=resnulli-us.20210112.gappssmtp.com; s=20210112; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=8IsR/7BBSr99fNGUQc2zD3gScYgcXe04DZhFoUN8RzE=; b=NqOLNQ99Mm5MnH16nxDcXzdPyAK+lhiZV5mfD4s1VWTntvNtRtx6hGLwU/igrx1dJT QLBws3FchORmSVWVupm5ZsdVsRV/FjcPw676LKTLXR1Xg9HrQvLGvlpujYqKhoyEHc+A vHECdlvEba2G1cJR7lcPpkUjIhB71bdo+ZsNdQkS/bv1DPO++HT3yj9iaxGXFUJ0QdcV p4+7Te52kuwvF6GM8phr+op8Qkik+NUhl9GWR6PlP/bfU/vyg+iRk5nvEO8Hgdh3f+Fg uWDYK01EOlPzwbaN7u5OIC4T74FMZZgoxLA4tpgdAumTMk/TGTMU6ZUXCIsL4c80pFgq +ETQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=8IsR/7BBSr99fNGUQc2zD3gScYgcXe04DZhFoUN8RzE=; b=N8MSX9Lv4OEumbbOzKZsPz2k5+H79HfiEgcdhlfISvss0hYOt6iDpB8JjTxzhraS4m cWkdxjXpNz8S7IbRVxbTYyWAnHJ5L+eN0qESRuN3CAV5jpPwPedAXmHZUDYK1oazCleA y1SrWli7uZtw9IRiL6JAH9hpr1CrnfUeBi9RobXPds19/soGdOaGzypk+KF8PaDhFqga kjRLiJBdpfYo6JWsrJvvjfne6YtR3wnFQ99Y5h09soFjjO+8rU5p2QQ+/uMNTNdGlxbM XQFj2Jpsbh/FdchjbZX20gCljHYAYam8AnY+1/CqF+yq6kauG1z1azZcEWzk9sSzWwtC AQ4g== X-Gm-Message-State: ANoB5pnPkMTN0p2UaxZXRWmBMQAve3urrXMw+uhfbLfBzWcVFLlUkdjr /eoVd8BEUfOiWDITdILQubv9SA== X-Google-Smtp-Source: AA0mqf65DnAvWOUpig9182b+xplZFrf2ffwn/LssCXLx6RHMw6LFoQVfWvmx3u5SXkByC4FDNxAMbw== X-Received: by 2002:a17:906:6c7:b0:78d:4061:5e1b with SMTP id v7-20020a17090606c700b0078d40615e1bmr67809617ejb.47.1670419164119; Wed, 07 Dec 2022 05:19:24 -0800 (PST) Received: from localhost (host-213-179-129-39.customer.m-online.net. [213.179.129.39]) by smtp.gmail.com with ESMTPSA id i13-20020a17090639cd00b0073022b796a7sm8646766eje.93.2022.12.07.05.19.23 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 07 Dec 2022 05:19:23 -0800 (PST) Date: Wed, 7 Dec 2022 14:19:22 +0100 From: Jiri Pirko To: Jakub Kicinski Cc: "Kubalewski, Arkadiusz" , Vadim Fedorenko , Jonathan Lemon , Paolo Abeni , "netdev@vger.kernel.org" , Vadim Fedorenko , "linux-arm-kernel@lists.infradead.org" , "linux-clk@vger.kernel.org" Subject: Re: [RFC PATCH v4 4/4] ptp_ocp: implement DPLL ops Message-ID: References: <20221129213724.10119-1-vfedorenko@novek.ru> <20221129213724.10119-5-vfedorenko@novek.ru> <20221206183313.713656f8@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20221206183313.713656f8@kernel.org> Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org Wed, Dec 07, 2022 at 03:33:13AM CET, kuba@kernel.org wrote: >On Fri, 2 Dec 2022 14:39:17 +0000 Kubalewski, Arkadiusz wrote: >> >>>Btw, did you consider having dpll instance here as and auxdev? It >> >>>would be suitable I believe. It is quite simple to do it. See >> >>>following patch as an example: >> >> >> >>I haven't think about it, definetly gonna take a look to see if there >> >>any benefits in ice. >> > >> >Please do. The proper separation and bus/device modelling is at least one >> >of the benefits. The other one is that all dpll drivers would happily live >> >in drivers/dpll/ side by side. >> >> Well, makes sense, but still need to take a closer look on that. >> I could do that on ice-driver part, don't feel strong enough yet to introduce >> Changes here in ptp_ocp. > >FWIW auxdev makes absolutely no sense to me for DPLL :/ >So Jiri, please say why. Why not? It's a subdev of a device. In mlx5, we have separate auxdevs for eth, rdma, vnet, representors. DPLL is also a separate entity which could be instantiated independently (as it is not really dependent on eth/rdma/etc)). Auxdev looks like an awesome fit. Why do you think it is not? Also, what's good about auxdev is that you can maintain them quite independetly. So there is going to be driver/dpll/ directory which would contain all dpll drivers. 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 7813BC352A1 for ; Wed, 7 Dec 2022 13:20:29 +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-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=SjuxEXauEvgHYQoLQLGPiTLbOZOg2vvzVdfOrGTc1XY=; b=DZvk9Y/2qKJJXD Ero+QiZtcq0ofI19jiMYKi6kGh2caPs4/DcAkF/8vn8JfMIzkimoXeYgZGF7m1huroHc0ooma0ZqA lwo6h0i8mtqTuTvS45G8qAiOXyViW22YUpAwhdhWnURMebAFHlkUnRolf5m21gWbaqRaWISek3Wq6 EXmK/4E2H+uuvSrhbWTTD45anCIQXNsE9L0NIXvUKOsVioav//NXPVwsFtQcQ50alP3xQGCatqFIA SLZ13p8+qm/e5++e/zc4vbSzbfSj3Qt8VKCeiwMvZIVjKKEVpth51SdvX4n/QtlUz6s/B7Udne5HZ E7eZhCaDDRZoTO1hUckw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1p2uKe-003JDG-Tk; Wed, 07 Dec 2022 13:19:29 +0000 Received: from mail-ej1-x62d.google.com ([2a00:1450:4864:20::62d]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1p2uKb-003J5m-Mp for linux-arm-kernel@lists.infradead.org; Wed, 07 Dec 2022 13:19:27 +0000 Received: by mail-ej1-x62d.google.com with SMTP id m18so12371468eji.5 for ; Wed, 07 Dec 2022 05:19:25 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=resnulli-us.20210112.gappssmtp.com; s=20210112; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=8IsR/7BBSr99fNGUQc2zD3gScYgcXe04DZhFoUN8RzE=; b=NqOLNQ99Mm5MnH16nxDcXzdPyAK+lhiZV5mfD4s1VWTntvNtRtx6hGLwU/igrx1dJT QLBws3FchORmSVWVupm5ZsdVsRV/FjcPw676LKTLXR1Xg9HrQvLGvlpujYqKhoyEHc+A vHECdlvEba2G1cJR7lcPpkUjIhB71bdo+ZsNdQkS/bv1DPO++HT3yj9iaxGXFUJ0QdcV p4+7Te52kuwvF6GM8phr+op8Qkik+NUhl9GWR6PlP/bfU/vyg+iRk5nvEO8Hgdh3f+Fg uWDYK01EOlPzwbaN7u5OIC4T74FMZZgoxLA4tpgdAumTMk/TGTMU6ZUXCIsL4c80pFgq +ETQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=8IsR/7BBSr99fNGUQc2zD3gScYgcXe04DZhFoUN8RzE=; b=q8l54Oobu6bJ57Tes7DAbWMNZYIrIfmSdAsi7FDL6sbC5Mchpq9D2sQGZ6Xq/yBsl/ Tdb+KtuI8QAVPRNIELSYvhAG5OYJ+eZe3feE7g8Xmca+h8IjczUXqHlfvgRR98wA2217 4jnMgItkzwXbm2xv9cRCj74im9LGtfDIy7JM7S5fWs6XGthVYLYlPEpbeQkZGQDT90PX gYFhPewWA5sB6VmQ2p87/8u+6ihirDc0SKnRVu413wvajzz7HuJSiSx6oW0LfmlfZlNa FXIhq8lz6TUU9OFeBMtRnem6J9NJDjIu/6AG2x+BZkq+9SMy5kQOAFI0iejFlzJL988i NJiQ== X-Gm-Message-State: ANoB5pk1B0i4oUMBFJV1SppAGGEcU74htahB4CkK02PC5SyURsYSmh4s UGmdyIVjOZScaJ26Wg8sQMGAzA== X-Google-Smtp-Source: AA0mqf65DnAvWOUpig9182b+xplZFrf2ffwn/LssCXLx6RHMw6LFoQVfWvmx3u5SXkByC4FDNxAMbw== X-Received: by 2002:a17:906:6c7:b0:78d:4061:5e1b with SMTP id v7-20020a17090606c700b0078d40615e1bmr67809617ejb.47.1670419164119; Wed, 07 Dec 2022 05:19:24 -0800 (PST) Received: from localhost (host-213-179-129-39.customer.m-online.net. [213.179.129.39]) by smtp.gmail.com with ESMTPSA id i13-20020a17090639cd00b0073022b796a7sm8646766eje.93.2022.12.07.05.19.23 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 07 Dec 2022 05:19:23 -0800 (PST) Date: Wed, 7 Dec 2022 14:19:22 +0100 From: Jiri Pirko To: Jakub Kicinski Cc: "Kubalewski, Arkadiusz" , Vadim Fedorenko , Jonathan Lemon , Paolo Abeni , "netdev@vger.kernel.org" , Vadim Fedorenko , "linux-arm-kernel@lists.infradead.org" , "linux-clk@vger.kernel.org" Subject: Re: [RFC PATCH v4 4/4] ptp_ocp: implement DPLL ops Message-ID: References: <20221129213724.10119-1-vfedorenko@novek.ru> <20221129213724.10119-5-vfedorenko@novek.ru> <20221206183313.713656f8@kernel.org> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20221206183313.713656f8@kernel.org> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20221207_051925_766807_6B73483D X-CRM114-Status: GOOD ( 16.02 ) 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-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Wed, Dec 07, 2022 at 03:33:13AM CET, kuba@kernel.org wrote: >On Fri, 2 Dec 2022 14:39:17 +0000 Kubalewski, Arkadiusz wrote: >> >>>Btw, did you consider having dpll instance here as and auxdev? It >> >>>would be suitable I believe. It is quite simple to do it. See >> >>>following patch as an example: >> >> >> >>I haven't think about it, definetly gonna take a look to see if there >> >>any benefits in ice. >> > >> >Please do. The proper separation and bus/device modelling is at least one >> >of the benefits. The other one is that all dpll drivers would happily live >> >in drivers/dpll/ side by side. >> >> Well, makes sense, but still need to take a closer look on that. >> I could do that on ice-driver part, don't feel strong enough yet to introduce >> Changes here in ptp_ocp. > >FWIW auxdev makes absolutely no sense to me for DPLL :/ >So Jiri, please say why. Why not? It's a subdev of a device. In mlx5, we have separate auxdevs for eth, rdma, vnet, representors. DPLL is also a separate entity which could be instantiated independently (as it is not really dependent on eth/rdma/etc)). Auxdev looks like an awesome fit. Why do you think it is not? Also, what's good about auxdev is that you can maintain them quite independetly. So there is going to be driver/dpll/ directory which would contain all dpll drivers. _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel