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 8AD22C001DC for ; Fri, 21 Jul 2023 11:15:07 +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=HjKzIHVDuIhUvgL3A9eApDaorgfH5bUMVT5f1Zem2Ck=; b=YfOYrVd8SMY+66 Q7BDLyY3jIUSjSFuyFXZ28M3jj2i/UfpQhX0HhKigtsPXOFZrxaudTtB99MP3tdjn1nWoiskJNW45 caFWY7GcAllM94bLdOITvKzxxccnNJy9xBznCzl+jVtVn+JK664Y61/1WgFnJmUo0f17j9L90glJY eMnG/EmnHqngt+GX/OG7k3DiHacF2CbIjnjpfJvbN6MbHT2xD3vtbIZaJQLmslz34hqISlozEotYx lFnt1fpMHeGOuQ8Q7dincLAKJyTDITTujOHNAYMJhWhuMOqYq4eaCUH4dKLPrR24KCUC8xdep6Z5/ mJmnhuGD5QWLjFUlyOtQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qMo5u-00DuDC-1W; Fri, 21 Jul 2023 11:14:46 +0000 Received: from mail-wm1-x32d.google.com ([2a00:1450:4864:20::32d]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qMo5q-00DuCD-1V for linux-arm-kernel@lists.infradead.org; Fri, 21 Jul 2023 11:14:45 +0000 Received: by mail-wm1-x32d.google.com with SMTP id 5b1f17b1804b1-3fbfa811667so18144695e9.1 for ; Fri, 21 Jul 2023 04:14:38 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=resnulli-us.20221208.gappssmtp.com; s=20221208; t=1689938077; x=1690542877; 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=KsKYTxerwELik8pjR4/+BgcM5ijG8hNe+3EkcfX1ez0=; b=nwgrQtb4QMEpkERwBnAAEXwK4HunSVUwxvHmAvE+X1Ms3OgrBejoCzkC5kreACyzOp gzHmFDIw8MensyPWbiQn/89a7+8PkpX6GdY58mPU3/WnBpntIVAZYYsX0N08acW3St5s 2PApigSYe3bV3OUtLu5PRP0so+DqsBwt6KaKX8S1gQvFLA9UwvxmQn8u/a5lyJ8EId19 m/DjCtmji9+sEvne6YsinOUjwwJtvXqb2qYDowbV6guEWEbGLhGo/Xf/YigNS/4dRLAy iV0b9ZWI/yLJ167YZA1WUC9do6vW7zuyk8cE+Ap3zl0giRESNsGxdjQJ1t7d3lOkefHc MBCw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1689938077; x=1690542877; 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=KsKYTxerwELik8pjR4/+BgcM5ijG8hNe+3EkcfX1ez0=; b=HF5bM8mbkYALxOTF2enG9McUE+Qs7jTcXAFBIEPqt71u3vyZ1QCweL3BW72lrCRDok xWbeZjIFzIt5WqfYAd5r06CvzrMGE8lxI5h88YqB76r1eQjb9OJ875573V1VmuiFySre SjnpmDATxPCLimXH24GGosov9kSOAjIWuM8oSjtq4h5EuDY6CY7PEI8bPDqPm5tsHBZL KvdM0jyLSVH36YuTywFJmtzClmBTmALEGVegHYXC7IHApBBsfabPDSO5aYKQf2l5Bh8R tb8B2Al5+XLzQyYvBQSLSe5cgNUd/cwGQvONFq5ZxSsZ0CMs0PFtJE2IMMVpzFwxSOgU uWrQ== X-Gm-Message-State: ABy/qLZu2EffrZQzXlif7Yzy4reN2WT4/pUnWWdBNO/HUkhwGJ8ap2cf j9okfjToAegJbw62E1osn77PBg== X-Google-Smtp-Source: APBJJlEWjQYHfhTevL2bgZ8/xfts+EcqB9AKykH+DAxcFu3KOQ5dhN2mwoZR93J97JYQKPD7VC+HoQ== X-Received: by 2002:a05:600c:1d1b:b0:3f9:68f:9c1a with SMTP id l27-20020a05600c1d1b00b003f9068f9c1amr7350230wms.15.1689938077196; Fri, 21 Jul 2023 04:14:37 -0700 (PDT) Received: from localhost (host-213-179-129-39.customer.m-online.net. [213.179.129.39]) by smtp.gmail.com with ESMTPSA id f14-20020a7bcd0e000000b003fc01f7b415sm5937987wmj.39.2023.07.21.04.14.36 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 21 Jul 2023 04:14:36 -0700 (PDT) Date: Fri, 21 Jul 2023 13:14:35 +0200 From: Jiri Pirko To: Vadim Fedorenko Cc: Jakub Kicinski , Arkadiusz Kubalewski , Jonathan Lemon , Paolo Abeni , Milena Olech , Michal Michalik , linux-arm-kernel@lists.infradead.org, poros@redhat.com, mschmidt@redhat.com, netdev@vger.kernel.org, linux-clk@vger.kernel.org, Bart Van Assche Subject: Re: [PATCH net-next 00/11] Create common DPLL configuration API Message-ID: References: <20230720091903.297066-1-vadim.fedorenko@linux.dev> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20230720091903.297066-1-vadim.fedorenko@linux.dev> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230721_041442_715066_B4DF307B X-CRM114-Status: UNSURE ( 9.37 ) X-CRM114-Notice: Please train this message. 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 There are couple of issues that came up during our internal ci run: 10:16:04 error: drivers/dpll/dpll_netlink.c:452:5: error: no previous prototype for '__dpll_device_change_ntf' [-Werror=missing-prototypes] 10:16:04 error: drivers/dpll/dpll_netlink.c:1283:13: error: no previous prototype for 'dpll_netlink_fini' [-Werror=missing-prototypes] 10:16:04 error: drivers/dpll/dpll_core.c:221:1: error: no previous prototype for 'dpll_xa_ref_dpll_find' [-Werror=missing-prototypes] 10:27:31 error: drivers/dpll/dpll_core.c:220:21: warning: symbol 'dpll_xa_ref_dpll_find' was not declared. Should it be static? 10:27:31 error: drivers/dpll/dpll_netlink.c:452:5: warning: symbol '__dpll_device_change_ntf' was not declared. Should it be static? 10:27:31 error: drivers/dpll/dpll_netlink.c:1283:13: warning: symbol 'dpll_netlink_fini' was not declared. Should it be static? 10:27:41 error: drivers/net/ethernet/intel/ice/ice_dpll.c:461:3: error: a label can only be part of a statement and a declaration is not a statement I believe that you didn't run make with C=2, otherwise you would hit these. Checkpatch issue: 10:29:30 CHECK: struct mutex definition without comment 10:29:30 #6581: FILE: drivers/net/ethernet/intel/ice/ice_dpll.h:85: 10:29:30 + struct mutex lock; Spelling errors: 10:45:08 error: Documentation/netlink/specs/dpll.yaml:165: prority ==> priority 10:45:08 error: include/uapi/linux/dpll.h:128: prority ==> priority 10:45:08 error: drivers/net/ethernet/intel/ice/ice_dpll.c:2008: userpsace ==> userspace 10:45:08 error: drivers/net/ethernet/intel/ice/ice_dpll.h:20: properities ==> properties Thu, Jul 20, 2023 at 11:18:52AM CEST, vadim.fedorenko@linux.dev wrote: >Implement common API for clock/DPLL configuration and status reporting. >The API utilises netlink interface as transport for commands and event >notifications. This API aim to extend current pin configuration and s/API aim/API aims/ >make it flexible and easy to cover special configurations. I don't follow. How this could "aim to extend current pin configuration" ? This is a new thing. Could you re-phrase? What's "special configuration"? Sounds odd. > >Netlink interface is based on ynl spec, it allows use of in-kernel >tools/net/ynl/cli.py application to control the interface with properly >formated command and json attribute strings. Here are few command >examples of how it works with `ice` driver on supported NIC: > >- dump dpll devices >$# ./tools/net/ynl/cli.py --spec Documentation/netlink/specs/dpll.yaml \ "$#" looks a bit odd. Just "$" with "sudo" when you want to emphasize root is needed to perform the command. >--dump device-get >[{'clock-id': 282574471561216, > 'id': 0, > 'lock-status': 'unlocked', > 'mode': 'automatic', > 'module-name': 'ice', > 'type': 'eec'}, > {'clock-id': 282574471561216, > 'id': 1, > 'lock-status': 'unlocked', > 'mode': 'automatic', > 'module-name': 'ice', > 'type': 'pps'}] > >- get single pin info: >$# ./tools/net/ynl/cli.py --spec Documentation/netlink/specs/dpll.yaml \ >--do pin-get --json '{"pin-id":2}' >{'clock-id': 282574471561216, > 'module-name': 'ice', > 'pin-board-label': 'C827_0-RCLKA', > 'pin-dpll-caps': 6, > 'pin-frequency': 1953125, > 'pin-id': 2, > 'pin-parenti-device': [{'id': 0, This looks like manual edit went wrong :) s/parenti/parent/ > 'pin-direction': 'input', > 'pin-prio': 11, > 'pin-state': 'selectable'}, > {'id': 1, > 'pin-direction': 'input', > 'pin-prio': 9, > 'pin-state': 'selectable'}], > 'pin-type': 'mux'} > >- set pin's state on dpll: >$# ./tools/net/ynl/cli.py --spec Documentation/netlink/specs/dpll.yaml \ >--do pin-set --json '{"pin-id":2, "pin-parent-device":{"id":1, "pin-state":2}}' > >- set pin's prio on dpll: >$# ./tools/net/ynl/cli.py --spec Documentation/netlink/specs/dpll.yaml \ >--do pin-set --json '{"pin-id":2, "pin-parent-device":{"id":1, "pin-prio":4}}' > >- set pin's state on parent pin: >$# ./tools/net/ynl/cli.py --spec Documentation/netlink/specs/dpll.yaml \ >--do pin-set --json '{"pin-id":13, \ > "pin-parent-pin":{"pin-id":2, "pin-state":1}}' > [...] _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel