Received: by 2002:a25:824b:0:0:0:0:0 with SMTP id d11csp1763791ybn; Thu, 26 Sep 2019 01:43:27 -0700 (PDT) X-Google-Smtp-Source: APXvYqxUwzS5q5nk05MCskaWgOOCKvPTtBzOUNYw6cFWejy/adfxgXCY1Z0PzmTx97xxcyQfkVKO X-Received: by 2002:a05:6402:1f4:: with SMTP id i20mr2283916edy.137.1569487407313; Thu, 26 Sep 2019 01:43:27 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1569487407; cv=none; d=google.com; s=arc-20160816; b=ZSHWJVfDI0wWHRHY7wyZcL6Dfx7LtBRGb25hQMIaMdKg0A8MUbWGi7a77CLWqsIAlp DBuVsXHMdwoQ1v3HHGksOhbkIRE3UEdpaMUZAcv1ITVARVPGa1ypYcfRcdtXleaqrwiQ KgLSujKCHLwrebz6RSqqqlmntrzNmJS80ZFyhryyaLMt6C9Vy0XfwgJVJsduZ+8QMX0/ OsbtXWQoBHh/NJNP/G5NwYVwtuOIQBo1m547ugmcT3dEXjVFpu34YX9XFzquB75G5Gvb fI7kiLyiqr4XT4A1xipHFn0tJd0VteMCjBhTsjzscp442wXddE4ymMoFgfu35biOy4X8 ZjOw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:mime-version:content-transfer-encoding :dlp-reaction:dlp-version:dlp-product:content-language :accept-language:in-reply-to:references:message-id:date:thread-index :thread-topic:subject:cc:to:from; bh=XbUMgmyQquh1XkyUykHoJ7d0u8B/FOQvUhFHGc+G5X0=; b=Q4XsP3IS4uRbEGBbyOtemCoskiPqx9r3bP63xt8TQJY5WnhZEiWP/kBwcvsMg6zMm4 2Hb2cwWmK9Ug/EYFHdn62UqNEXHy3efFl/KaVmmPYgsC+PV3OjZx8dR1vq0SlNZwlrUJ QJM+QyUJjli8VM0mEN6Fik1QLcbkdmArVW8NZWyNSKOx577mlXyoNBCtuHjd1WFGacgY DO0+kBDbXZ9Tclk/ezGzT+z8HQBHkuCblH7kBPdX+e32XekwrNqTGrfcRp8cl4tgLC/I vQevro+YkoKv8Madjlnj6I7dp+gFwjL7Yf9MwAB6XL5F8chnzBwqY3GVK4OKKYDLSLhZ ozYw== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=intel.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id v2si641429eju.3.2019.09.26.01.43.03; Thu, 26 Sep 2019 01:43:27 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=intel.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2441891AbfIXTX2 convert rfc822-to-8bit (ORCPT + 99 others); Tue, 24 Sep 2019 15:23:28 -0400 Received: from mga12.intel.com ([192.55.52.136]:11584 "EHLO mga12.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S2407325AbfIXTX1 (ORCPT ); Tue, 24 Sep 2019 15:23:27 -0400 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga002.jf.intel.com ([10.7.209.21]) by fmsmga106.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 24 Sep 2019 12:23:27 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.64,545,1559545200"; d="scan'208";a="201006599" Received: from orsmsx102.amr.corp.intel.com ([10.22.225.129]) by orsmga002.jf.intel.com with ESMTP; 24 Sep 2019 12:23:26 -0700 Received: from orsmsx121.amr.corp.intel.com ([169.254.10.190]) by ORSMSX102.amr.corp.intel.com ([169.254.3.63]) with mapi id 14.03.0439.000; Tue, 24 Sep 2019 12:23:26 -0700 From: "Keller, Jacob E" To: Felipe Balbi , Richard Cochran CC: "Hall, Christopher S" , "netdev@vger.kernel.org" , "linux-kernel@vger.kernel.org" Subject: RE: [PATCH v4 2/2] PTP: add support for one-shot output Thread-Topic: [PATCH v4 2/2] PTP: add support for one-shot output Thread-Index: AQHVaGiJVAygg13ZgEaRBcEK9JTjfKc7SaLQ Date: Tue, 24 Sep 2019 19:23:26 +0000 Message-ID: <02874ECE860811409154E81DA85FBB58968D24E2@ORSMSX121.amr.corp.intel.com> References: <20190911061622.774006-1-felipe.balbi@linux.intel.com> <20190911061622.774006-2-felipe.balbi@linux.intel.com> In-Reply-To: <20190911061622.774006-2-felipe.balbi@linux.intel.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiYzBiMDY1MDEtNGJhZi00NDg3LThmN2MtOTMzZGViYTZhYmZhIiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX05UIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE3LjEwLjE4MDQuNDkiLCJUcnVzdGVkTGFiZWxIYXNoIjoiUGJZdkpiMHJIU3lXTmx5dTZSRlByRHVsMzRja1RLWU1mNXhvQnROcnd0V2hvajBcL1pxN0NTQnlTeENoYkt2UGQifQ== x-ctpclassification: CTP_NT dlp-product: dlpe-windows dlp-version: 11.2.0.6 dlp-reaction: no-action x-originating-ip: [10.22.254.140] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org > -----Original Message----- > From: netdev-owner@vger.kernel.org [mailto:netdev-owner@vger.kernel.org] On > Behalf Of Felipe Balbi > Sent: Tuesday, September 10, 2019 11:16 PM > To: Richard Cochran > Cc: Hall, Christopher S ; netdev@vger.kernel.org; > linux-kernel@vger.kernel.org; Felipe Balbi > Subject: [PATCH v4 2/2] PTP: add support for one-shot output > > Some controllers allow for a one-shot output pulse, in contrast to > periodic output. Now that we have extensible versions of our IOCTLs, we > can finally make use of the 'flags' field to pass a bit telling driver > that if we want one-shot pulse output. > > Signed-off-by: Felipe Balbi > --- > > Changes since v3: > - Remove bogus bitwise negation > > Changes since v2: > - Add _PEROUT_ to bit macro > > Changes since v1: > - remove comment from .flags field > > include/uapi/linux/ptp_clock.h | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/include/uapi/linux/ptp_clock.h b/include/uapi/linux/ptp_clock.h > index 9a0af3511b68..f16301015949 100644 > --- a/include/uapi/linux/ptp_clock.h > +++ b/include/uapi/linux/ptp_clock.h > @@ -38,8 +38,8 @@ > /* > * Bits of the ptp_perout_request.flags field: > */ > -#define PTP_PEROUT_VALID_FLAGS (0) > - > +#define PTP_PEROUT_ONE_SHOT (1<<0) > +#define PTP_PEROUT_VALID_FLAGS (PTP_PEROUT_ONE_SHOT) > /* > * struct ptp_clock_time - represents a time value > * > @@ -77,7 +77,7 @@ struct ptp_perout_request { > struct ptp_clock_time start; /* Absolute start time. */ > struct ptp_clock_time period; /* Desired period, zero means disable. */ > unsigned int index; /* Which channel to configure. */ > - unsigned int flags; /* Reserved for future use. */ > + unsigned int flags; > unsigned int rsv[4]; /* Reserved for future use. */ > }; > > -- > 2.23.0 Hi Felipe, Do you have any examples for how you envision using this? I don't see any drivers or other code on the list for doing so. Additionally, it seems weird because we do not have support for specifying the pulse width. I guess you leave that up to driver choice? Thanks, Jake