From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f47.google.com (mail-wm1-f47.google.com [209.85.128.47]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id CFB7A2FDC30 for ; Thu, 25 Jun 2026 08:45:50 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.47 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1782377155; cv=none; b=uEVMedM89QqLxg2E/+4gG7qBTzhvex42+g1g4X692rVi8JYs4tgfr7EdbT0sF/9Cy9y1mFMsqlfEFgeHuPOcEWB3ZoBh1+q3WncRcyPZBbrEG3ppFIBtHfgmQ/OeBsM0VcPYTMRtFba6/aIbcgMmZ0/e4lMNQqM7Ft4DKQAfll8= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1782377155; c=relaxed/simple; bh=qR4A+m766w/c6ewO98NEv5b2pIqO2z9SA5O/ySjWWVE=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=bfNaPkCN+PnMYr1Ne78DniGqeUn10uzcr2YySdOiZBFPtaEZRgz6Vun4qHOzl6seNIfBn10P5n+Qbe+ofAuEkoPw/GEl3efwIvL+Btnr/T9KwC0Sa+REt9z2aR+1DH5LJyvetubu2kMwj+Yjn8CXaxQTUyXs33FNNNjv7GKxof0= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=resnulli.us; spf=none smtp.mailfrom=resnulli.us; dkim=pass (2048-bit key) header.d=resnulli-us.20251104.gappssmtp.com header.i=@resnulli-us.20251104.gappssmtp.com header.b=w+UnEQ9N; arc=none smtp.client-ip=209.85.128.47 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=resnulli.us Authentication-Results: smtp.subspace.kernel.org; spf=none smtp.mailfrom=resnulli.us Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=resnulli-us.20251104.gappssmtp.com header.i=@resnulli-us.20251104.gappssmtp.com header.b="w+UnEQ9N" Received: by mail-wm1-f47.google.com with SMTP id 5b1f17b1804b1-4924593f45dso22916375e9.1 for ; Thu, 25 Jun 2026 01:45:50 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=resnulli-us.20251104.gappssmtp.com; s=20251104; t=1782377149; x=1782981949; darn=vger.kernel.org; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:message-id:subject:cc:to:from:date:from:to :cc:subject:date:message-id:reply-to; bh=jjzeGJ6UjAoFVnlOFZBR/ApcCgu66rueAWjixl+vBgI=; b=w+UnEQ9NlC0qEf7tb6N72Gt8WO6G5RAGY8JeanAoL4pUDfQhC91KXhVdwciXQWNN60 Gc843fVAawoW735rfWba+1AGAhwlFuVOXJVqWB5Ua28XMud6s/ZIy4+odHked2DSjc6l PFeMyI4X9asgJ8IEsPxyPvrDeS7rvOAcU4GiTp/kzpuMNag54VOIahBb8RVYG6UHSz07 15jeiBVuX5qf7C1uPWs71mLr6H/MMZo99cdqlm1ZJnMRhLKIE160INMWBP1WCi0+5ZlB fFWdw3D8GaWTqVTL7qOuHCIzAp0osmr+ltF5WuDEIOmfq3MBH5AdrPspY5oHTcbBEmFD +0eg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1782377149; x=1782981949; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:message-id:subject:cc:to:from:date:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=jjzeGJ6UjAoFVnlOFZBR/ApcCgu66rueAWjixl+vBgI=; b=mI9E1SZz5Rw12OssS23wghNxl7+oVkhojl9VoPDCCye7h57vbzxpEZaCwjtGmxG2mm W+d7S1RbC5bn70Ef4LchAhQqZJcwIdjlD6U0BgEbvZK//G9Gaxy1CwqmV9xxqXhFzKB6 rhChQFPaGKfy4NP6aS0s+WRV8Or0MyFk156NmvtBiMP5uAWSRy4QjQfFdKeq8Kx7uD0X nTqJGgTB5xP1Pg+NDBo+1aMd0FXe9JQoiUUAO4ON6hsXTmzO/aJiupwD8ltRdQnrCkbm NBsQKrLY8Ak2saV7Tklz9HWi8sg0bjY+QLTf5fU0/LPqVBUAM/v83Z/YAz0SJmpmfA9N fUcA== X-Forwarded-Encrypted: i=1; AFNElJ+4Uuqu9FS07WOyThlbnsM4Xiu7xSCH5H0mZ4SiZJTz9g412IzPnO+H1fBuPxWElc57eSqtLjN2dZ4CzOQ=@vger.kernel.org X-Gm-Message-State: AOJu0YyS29RTEC+cT+A9+N392SpQDuqp/CE12X1+keo3St5nichObPlk 19/rX7asYDu1i2hdZdsSDE+2Xj6aTGyqvFvmdZ/XouYv0gcGilK8UEq8Mjv2DmA81kU= X-Gm-Gg: AfdE7clLcJN9RntTGI8ISVu5yBgiIDdXNOWZsQyI4w8ot36LjW1/QdDMEh2k+bkg7yt fMuJox9IxnOsCK5NU6kx7T5AyBuOkRz8pqyh/1G7PIDCe2ysCgUjQ5s4MUxnGwq206Rzwa7bPX/ JwBts11i4DVyUu0Tfe32sf6syhIfkltbltRxjFdPt1qOeVJMIbYuMSDe3efeo9VzuEg6rBShlC7 xhvHdkRjcgw9/LCikqkAe2b38NWs48EEkbN3myv2NvDBzC3WOxxQsoE36lhpnjQmyZToRTPMTzQ EsU1iM7VKh1LPabI4qA1iweWtNj6MQAnebIuOhhEzrzIvnKe2LZOs2EAJsdn4+uboiWWm6CSlTa qeyEOe47NRlJ5xie5lTcVQb9lzEC8HrfXL5Ml4rhUy3V6itaMMd36twFrNcG4pml8gO3L8kV22I jYpT8D/TrgRFZLbvCONwcGYKP1msy0N60= X-Received: by 2002:a05:600c:a0d:b0:492:6487:a87 with SMTP id 5b1f17b1804b1-49266893009mr20193815e9.32.1782377148642; Thu, 25 Jun 2026 01:45:48 -0700 (PDT) Received: from localhost ([208.127.45.13]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-4926544dfaesm32135595e9.2.2026.06.25.01.45.47 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 25 Jun 2026 01:45:48 -0700 (PDT) Date: Thu, 25 Jun 2026 10:45:44 +0200 From: Jiri Pirko To: Vadim Fedorenko Cc: Ivan Vecera , "Kubalewski, Arkadiusz" , Jakub Kicinski , "netdev@vger.kernel.org" , Jiri Pirko , "David S. Miller" , Donald Hunter , Eric Dumazet , "Schmidt, Michal" , Paolo Abeni , "Vaananen, Pasi" , "Oros, Petr" , Prathosh Satish , Simon Horman , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH net-next v5 1/4] dpll: add DPLL_PIN_TYPE_INT_NCO pin type Message-ID: References: <99cebea4-156a-4379-922e-07c50f766fbe@redhat.com> <23e47140-f69f-451d-9154-29071130c11c@redhat.com> <0f8fe4e0-72d8-48a6-96ad-d1650919d2df@linux.dev> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <0f8fe4e0-72d8-48a6-96ad-d1650919d2df@linux.dev> Wed, Jun 24, 2026 at 05:57:35PM +0200, vadim.fedorenko@linux.dev wrote: >On 19/06/2026 18:07, Ivan Vecera wrote: [...] >> >> Proposal: >> 1) new pin capability >>    - name: state-connected-override >>    - doc: pin state can be changed to connected in any DPLL mode >> >> 2) new NCO pin type to switch the DPLL to NCO mode when connected >> >> 3) automatic-only DPLL >>    - should expose NCO pin with state-connected-override capability >> >> 4) manual-only DPLL >>   - does not need to expose NCO pin with state-connected-override cap >> >> 5) dual-mode DPLL (supporting mode switching) >>   - if it exposes NCO pin with the override cap then it has to support >>     switching to NCO mode directly from AUTO mode >>   - if does not expose NCO pin with the override cap then a user MUST >>     switch the DPLL mode from AUTO to MANUAL to be able to make NCO >>     pin connected to the DPLL > >I still don't see good reasoning for the pin. Even this sentence says >"DPLL mode" which keeps me thinking whether we have to move it to a >special DPLL mode. All these items look like overcomplication of a >simple function of the device itself. DPLL can be either in the closed >loop when one of the pins provides a signal to align to, or in the open >loop meaning that software can control adjustments to phase/frequency. >But it's definitely a property of the device, and it's not a pin in any >kind... Vadim, did you see this: https://lore.kernel.org/all/aiftnkuT9IP31qUm@FV6GYCPJ69/ ? I very thoroughly described what you are questioning. There is 0 reply to that email so perhaps you missed it? IDK.