On 07/04/2022 10:36, Gabriel Moyano wrote:
--- spec/build/cpukit/cpuopts.yml | 2 ++ spec/build/cpukit/optppssync.yml | 16 ++++++++++++++++ 2 files changed, 18 insertions(+) create mode 100644 spec/build/cpukit/optppssync.ymldiff --git a/spec/build/cpukit/cpuopts.yml b/spec/build/cpukit/cpuopts.yml index 301d49ccea..db0e05395f 100644 --- a/spec/build/cpukit/cpuopts.yml +++ b/spec/build/cpukit/cpuopts.yml @@ -49,6 +49,8 @@ links: uid: optparavirt - role: build-dependency uid: optposix +- role: build-dependency + uid: optppssync - role: build-dependency uid: optprofiling - role: build-dependency diff --git a/spec/build/cpukit/optppssync.yml b/spec/build/cpukit/optppssync.yml new file mode 100644 index 0000000000..6766cf0d40 --- /dev/null +++ b/spec/build/cpukit/optppssync.yml @@ -0,0 +1,16 @@ +SPDX-License-Identifier: CC-BY-SA-4.0 OR BSD-2-Clause +actions: +- get-boolean: null +- env-enable: null +- define-condition: null +build-type: option +copyrights: +- Copyright (C) 2022 German Aerospace Center (DLR) +default: false +default-by-variant: [] +description: | + Enable time synchronization using a Pulse Per Second signal +enabled-by: true +links: [] +name: PPS_SYNC +type: build
If this should really be a CPU option, then it needs a proper RTEMS_* prefix. Why do we need the option. What is the overhead if it is always enabled.
In general, the patch set lacks test cases. -- embedded brains GmbH Herr Sebastian HUBER Dornierstr. 4 82178 Puchheim Germany email: sebastian.hu...@embedded-brains.de phone: +49-89-18 94 741 - 16 fax: +49-89-18 94 741 - 08 Registergericht: Amtsgericht München Registernummer: HRB 157899 Vertretungsberechtigte Geschäftsführer: Peter Rasmussen, Thomas Dörfler Unsere Datenschutzerklärung finden Sie hier: https://embedded-brains.de/datenschutzerklaerung/ _______________________________________________ devel mailing list devel@rtems.org http://lists.rtems.org/mailman/listinfo/devel
