Re: [PATCH v1 1/7] pps: clients: gpio: Bail out on error when requesting GPIO echo line

2021-03-09 Thread Rodolfo Giometti
On 09/03/21 12:28, Andy Shevchenko wrote: > On Tue, Mar 09, 2021 at 11:51:58AM +0100, Rodolfo Giometti wrote: >> On 09/03/21 11:47, Andy Shevchenko wrote: >>> +Cc: Greg >>> >>> On Fri, Feb 26, 2021 at 07:03:32PM +0200, Andy Shevchenko wrote: >>>>

Re: [PATCH resend v1 7/7] pps: clients: gpio: Rearrange optional stuff in pps_gpio_setup()

2021-03-09 Thread Rodolfo Giometti
erty_read_bool(dev, "assert-falling-edge"); > + /* sanity check on echo_active_ms */ > + if (!value || value > 999) { > + dev_err(dev, "echo-active-ms: %u - bad value from FW\n", value); > + return -EINVAL; > +

Re: [PATCH resend v1 6/7] pps: clients: gpio: Use struct device pointer directly

2021-03-09 Thread Rodolfo Giometti
to register IRQ %d as PPS source\n", > + dev_err(dev, "failed to register IRQ %d as PPS source\n", > data->irq); > return PTR_ERR(data->pps); > } > > /* register IRQ interrupt handler */ > - ret =

Re: [PATCH resend v1 5/7] pps: clients: gpio: Make use of device properties

2021-03-09 Thread Rodolfo Giometti
active-ms: %u - bad value from FW\n", > data->echo_active_ms); > return -EINVAL; > } > } > > - if (of_property_read_bool(np, "assert-falling-edge")) > - data->assert_falling_edge =

Re: [PATCH resend v1 4/7] pps: clients: gpio: Get rid of legacy platform data

2021-03-09 Thread Rodolfo Giometti
> - * pps-gpio.h -- PPS client for GPIOs > - * > - * Copyright (C) 2011 James Nuss > - */ > - > -#ifndef _PPS_GPIO_H > -#define _PPS_GPIO_H > - > -struct pps_gpio_platform_data { > - struct gpio_desc *gpio_pin; > - struct gpio_desc *echo_p

Re: [PATCH resend v1 3/7] pps: clients: gpio: Remove redundant condition in ->remove()

2021-03-09 Thread Rodolfo Giometti
o pin in any case */ > - gpiod_set_value(data->echo_pin, 0); > - } > + del_timer_sync(&data->echo_timer); > + /* reset echo pin in any case */ > + gpiod_set_value(data->echo_pin, 0); > dev_info(&pdev->dev, "removed IRQ %d as PPS source\n", data

Re: [PATCH resend v1 2/7] pps: clients: gpio: Use dev_err_probe() to avoid log noise

2021-03-09 Thread Rodolfo Giometti
if (IS_ERR(data->echo_pin)) { > - dev_err(&pdev->dev, "failed to request ECHO GPIO\n"); > - return PTR_ERR(data->echo_pin); > - } > + if (IS_ERR(data->echo_pin)) > + return dev_err_probe(&pdev->dev, PTR_ERR(d

Re: [PATCH resend v1 1/7] pps: clients: gpio: Bail out on error when requesting GPIO echo line

2021-03-09 Thread Rodolfo Giometti
est ECHO GPIO\n"); > + return PTR_ERR(data->echo_pin); > + } > > + if (data->echo_pin) { > ret = of_property_read_u32(np, > "echo-active-ms", > &value); > Acked-by: Rodolfo Giometti --

Re: [PATCH v1 1/7] pps: clients: gpio: Bail out on error when requesting GPIO echo line

2021-03-09 Thread Rodolfo Giometti
On 09/03/21 11:47, Andy Shevchenko wrote: > +Cc: Greg > > On Fri, Feb 26, 2021 at 07:03:32PM +0200, Andy Shevchenko wrote: >> On Tue, Feb 16, 2021 at 01:31:48PM +0200, Andy Shevchenko wrote: >>> When requesting optional GPIO echo line, bail out on error, >>> so user will know that something wrong

Re: [PATCH v2 3/4] pps: Use subdir-ccflags-* to inherit debug flag

2021-02-09 Thread Rodolfo Giometti
PS)+= kc.o > obj-$(CONFIG_PPS):= pps_core.o > obj-y+= clients/ generators/ > > -ccflags-$(CONFIG_PPS_DEBUG) := -DDEBUG > +subdir-ccflags-$(CONFIG_PPS_DEBUG) := -DDEBUG > Acked-by: Rodolfo Giometti -- GNU/Linux Solutions e-mai

Re: [PATCH 1/2] misc: c2port: core: Make copying name from userspace more secure

2020-11-03 Thread Rodolfo Giometti
On 03/11/2020 09:57, Lee Jones wrote: > On Mon, 02 Nov 2020, Rodolfo Giometti wrote: > >> On 02/11/2020 14:47, Lee Jones wrote: >>> On Mon, 02 Nov 2020, gre...@linuxfoundation.org wrote: >>> >>>> On Mon, Nov 02, 2020 at 12:43:01PM +, Lee J

Re: [PATCH 1/2] misc: c2port: core: Make copying name from userspace more secure

2020-11-02 Thread Rodolfo Giometti
UL terminator, which means that a 1-byte >>>>>>> leak would be possible *if* this was ever copied to userspace. Ensure >>>>>>> the buffer will always be NUL terminated by using the kernel's >>>>>>> strscpy() which a)

Re: [PATCH 01/10] misc: c2port: core: Ensure source size does not equal destination size in strncpy()

2020-06-26 Thread Rodolfo Giometti
derlying_strncpy’ > 307 | return __underlying_strncpy(p, q, size); > | ^~~~ > > Cc: Rodolfo Giometti > Cc: "Eurotech S.p.A" > Signed-off-by: Lee Jones > --- > drivers/misc/c2port/core.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a

Re: [PATCH v15 3/3] pps: pps-gpio pps-echo implementation

2019-02-04 Thread Rodolfo Giometti
inclusion into the linux kernel by Tom Burkart. Signed-off-by: Lukas Senger Signed-off-by: Tom Burkart Acked-by: Rodolfo Giometti -- GNU/Linux Solutions e-mail: giome...@enneenne.com Linux Device Driver giome...@linux.it Embedded Systems

Re: [PATCH v15 1/3] pps: descriptor-based gpio

2019-02-04 Thread Rodolfo Giometti
On 02/02/2019 11:57, Tom Burkart wrote: This patch changes the GPIO access for the pps-gpio driver from the integer based API to the descriptor based API. Reviewed-by: Philipp Zabel Signed-off-by: Tom Burkart Acked-by: Rodolfo Giometti -- GNU/Linux Solutions e-mail: giome

Re: UBSAN: Undefined behaviour in drivers/pps/pps.c

2019-01-09 Thread Rodolfo Giometti
On 08/01/2019 21:24, Kyungtae Kim wrote: We report a bug in linux-4.20: "UBSAN: Undefined behaviour in drivers/pps/pps.c" kernel config: https://kt0755.github.io/etc/config_v4.20_stable repro: https://kt0755.github.io/etc/repro.a6372.c pps_cdev_pps_fetch() lacks the bounds checking for computin

Re: [PATCH v14 3/3] pps: pps-gpio pps-echo implementation

2019-01-04 Thread Rodolfo Giometti
On 30/12/2018 09:33, Tom Burkart wrote: This patch implements the pps echo functionality for pps-gpio, that sysfs claims is available already. Configuration is done via device tree bindings. This patch was originally written by Lukas Senger as part of a masters thesis project and modified for i

Re: [PATCH v14 2/3] dt-bindings: pps: pps-gpio PPS ECHO implementation

2019-01-04 Thread Rodolfo Giometti
On 30/12/2018 09:33, Tom Burkart wrote: This patch implements the device tree binding changes required for the pps echo functionality for pps-gpio, that sysfs claims is available already. This patch was originally written by Lukas Senger as part of a masters thesis project and modified for inclu

Re: [PATCH -next] pps: using ERR_PTR instead of NULL while pps_register_source fails

2018-11-27 Thread Rodolfo Giometti
On 27/11/2018 08:34, Greg KH wrote: On Mon, Nov 26, 2018 at 11:43:06AM +0100, Rodolfo Giometti wrote: On 26/11/2018 11:24, YueHaibing wrote: pps_register_source() has keeps error codes in a local variable, but it does not make use of the code. This patch let it return the errcode in case of

Re: [PATCH 2/2] pps-gpio: Set echo GPIO pin via devicetree

2018-02-15 Thread Rodolfo Giometti
On 15/02/18 16:08, Lukas Senger wrote: @@ -20,18 +21,20 @@ target = <&gpio>; __overlay__ { pps_pins: pps_pins@12 { - brcm,pins = <18>; - brcm,function = <0>;// in -

Re: [PATCH 1/2] pps-gpio: Avoid flooding syslog

2018-02-15 Thread Rodolfo Giometti
On 15/02/18 16:12, Lukas Senger wrote: Why a void function? You should use it to toggle echoing GPIO... =8-o RFC 2783 says to generate the echo pulse "as rapidly as possible" which is why I do the toggling in the irq handler. However this leaves the default echo function installed which just

Re: [PATCH 2/2] pps-gpio: Set echo GPIO pin via devicetree

2018-02-15 Thread Rodolfo Giometti
On 15/02/18 13:59, Lukas Senger wrote: --- arch/arm/boot/dts/overlays/pps-gpio-overlay.dts | 13 - drivers/pps/clients/pps-gpio.c | 26 ++--- include/linux/pps-gpio.h| 1 + 3 files changed, 24 insertions(+), 16 deletio

Re: [PATCH 1/2] pps-gpio: Avoid flooding syslog

2018-02-15 Thread Rodolfo Giometti
On 15/02/18 13:59, Lukas Senger wrote: ^^^ Missing description and signatures... --- drivers/pps/clients/pps-gpio.c | 1 + include/linux/pps-gpio.h | 2 ++ 2 files changed, 3 insertions(+) diff --git a/drivers/pps/clients/pps-gpio.c b/drivers/pps/clients/pps-gpio.c in

Re: [PATCH] pps-gpio: implement echo pulses

2018-02-07 Thread Rodolfo Giometti
On 06/02/18 16:58, Lukas Senger wrote: pps-gpio reports as having echo capability via sysfs, which is not actually the case. This patch implements it. The output pin is hardcoded as 17. This should probably be configurable via the dtoverlay in the same way as the input pin. No hardcoded stuff

Re: [PATCH 2/2] pps: generator: use new parport device model

2018-01-29 Thread Rodolfo Giometti
On 29/01/18 00:22, Sudip Mukherjee wrote: Modify pps generator driver to use the new parallel port device model. Signed-off-by: Sudip Mukherjee Acked-by: Rodolfo Giometti -- HCE Engineering e-mail: giome...@hce-engineering.it GNU/Linux Solutions

Re: [PATCH 1/2] pps: client: use new parport device model

2018-01-29 Thread Rodolfo Giometti
On 29/01/18 00:22, Sudip Mukherjee wrote: Modify pps client driver to use the new parallel port device model. In that process, added an index to mention the device number when we have more than one parallel port. Signed-off-by: Sudip Mukherjee Acked-by: Rodolfo Giometti -- HCE Engineering

Re: [PATCH] pps: parport: use timespec64 instead of timespec

2017-11-28 Thread Rodolfo Giometti
suspend/resume cycle, which seems worse. Signed-off-by: Arnd Bergmann Acked-by: Rodolfo Giometti -- HCE Engineering e-mail: giome...@hce-engineering.it GNU/Linux Solutions giome...@enneenne.com Linux Device Driver gi

Re: [PATCH] pps-gpio: use IRQ edge config when not capturing both edges

2017-09-12 Thread Rodolfo Giometti
On 12/09/2017 17:54, Tim Harvey wrote: On Sun, Sep 10, 2017 at 3:43 AM, Rodolfo Giometti wrote: On 08/09/2017 21:53, Tim Harvey wrote: PPS signals with very short pulse-widths can be missed if their state changes by the time the interrupt handler reads the GPIO pin state. To avoid this in

Re: [PATCH] MAINTAINERS: update linuxpps mailing list

2017-09-10 Thread Rodolfo Giometti
/ppp_* PPS SUPPORT M:Rodolfo Giometti W:http://wiki.enneenne.com/index.php/LinuxPPS_support -L: linux...@ml.enneenne.com (subscribers-only) +L: discussi...@linuxpps.org S:Maintained F:Documentation/pps/ F:drivers/pps/ The list is discussi...@linuxpps.org but

Re: [PATCH] pps-gpio: use IRQ edge config when not capturing both edges

2017-09-10 Thread Rodolfo Giometti
On 08/09/2017 21:53, Tim Harvey wrote: PPS signals with very short pulse-widths can be missed if their state changes by the time the interrupt handler reads the GPIO pin state. To avoid this in the case where we are only looking for one edge we can use the edge configuration for the pin state bu

Re: Support for BQ28Z610 Battery Monitor IC

2017-09-05 Thread Rodolfo Giometti
On 04/09/2017 15:53, rajender.re...@stesalit.in wrote: Dear Rodolfo Giometti, As discussed please find attached patch. Please let me know if you need more details. It looks OK to me... however: - rewrite the patch's title as "power bq27xxx_battery: add support for BQ28Z610 batte

Re: [PATCH] PPS: Use surrounding "if PPS" to remove numerous dependency checks

2017-08-28 Thread Rodolfo Giometti
On 26/08/2017 16:55, Robert P. J. Day wrote: Adding high-level "if PPS" makes lower-level dependency tests superfluous. Signed-off-by: Robert P. J. Day Acked-by: Rodolfo Giometti -- HCE Engineering e-mail: giome...@hce-engineering.it GNU/Linux

Re: [PATCH v2] PPS: Aesthetic tweaks to PPS-related content

2017-08-28 Thread Rodolfo Giometti
| 2 +- include/linux/pps_kernel.h | 16 +++- include/uapi/linux/pps.h | 4 ++-- kernel/time/timekeeping.c | 2 +- 7 files changed, 43 insertions(+), 36 deleti

Re: [PATCH] Make PPS into a menuconfig to ease disabling

2017-05-09 Thread Rodolfo Giometti
On 04/11/17 16:26, Vincent Legoll wrote: So that there's no need to get into the submenu to disable all related config entries. The BROKEN PPS_GENERATOR_PARPORT now also depends on PPS Signed-off-by: Vincent Legoll Acked-by: Rodolfo Giometti -- HCE Engineering e

Re: [PATCH] c2port: checking for NULL instead of IS_ERR()

2017-04-12 Thread Rodolfo Giometti
On 04/12/17 10:33, Dan Carpenter wrote: c2port_device_register() never returns NULL, it uses error pointers. Fixes: 65131cd52b9e ("c2port: add c2port support for Eurotech Duramar 2150") Signed-off-by: Dan Carpenter Acked-by: Rodolfo Giometti diff --git a/drivers/misc/c2p

Re: [PATCH 8/8] LinuxPPS: pps_gen_parport: Add check for bad clocksource.

2017-03-04 Thread Rodolfo Giometti
On 03/04/17 10:19, Alexander Gerasiov wrote: Hello Rodolfo, I should say, that I found this solution is not the best. For example on high speed PCs (I think over 3GHz) 5 iteration is not enough and module stops working after several hours. That makes it not user-friendly, cause user had to calib

Re: [PATCH 2/2] pps: fix padding issue with PPS_FETCH for ioctl_compat

2017-03-04 Thread Rodolfo Giometti
On 02/24/17 21:23, Matt Ranostay wrote: Issue is that x86 32-bit aligns to 4-bytes instead of 8-bytes so this patchset works around the issue and corrects the data returned in pps_fdata_compat. Cc: Rodolfo Giometti Cc: Moritz Fischer Cc: George McCollister Signed-off-by: Matt Ranostay

Re: [PATCH 8/8] LinuxPPS: pps_gen_parport: Add check for bad clocksource.

2017-03-04 Thread Rodolfo Giometti
see this thread on lkml: https://lkml.org/lkml/2011/2/18/310 Signed-off-by: Nikita Edward Baruzdin Signed-off-by: Alexander GQ Gerasiov Acked-by: Rodolfo Giometti

Re: [PATCH 2/8] ntp/pps: ignore pps_valid decreasing if there is no pps signal.

2017-03-04 Thread Rodolfo Giometti
On 02/15/17 15:31, Andrey Drobyshev wrote: From: Alexander GQ Gerasiov In case pps_dec_valid() is called from second_overflow() in the absence of pps signal, there is no need to decrease pps_valid. Signed-off-by: Alexander GQ Gerasiov Acked-by: Rodolfo Giometti

Re: [PATCH 7/8] LinuxPPS: pps_gen_parport: Add polarity parameter for inverted signal.

2017-03-03 Thread Rodolfo Giometti
(polarity, uint, 0); #define SAFETY_INTERVAL3000/* set the hrtimer earlier for safety (ns) */ Acked-by: Rodolfo Giometti

Re: [PATCH 6/8] LinuxPPS: pps_parport: Ignore interrupt invoked less than 0.5sec after previous.

2017-03-03 Thread Rodolfo Giometti
On 02/15/17 15:31, Andrey Drobyshev wrote: From: Alexander GQ Gerasiov On some devices interrupt may be invoked by parasitic assert event produced while switching from high to low. Such interrupt should be ignored. Signed-off-by: Alexander GQ Gerasiov Acked-by: Rodolfo Giometti

Re: [PATCH 1/2] pps: add ioctl_compat function to correct ioctl definitions

2017-03-03 Thread Rodolfo Giometti
size. Cc: Rodolfo Giometti Cc: Moritz Fischer Cc: George McCollister Signed-off-by: Matt Ranostay --- drivers/pps/pps.c | 13 + 1 file changed, 13 insertions(+) Acked-by: Rodolfo Giometti -- HCE Engineering e-mail: giome...@hce-engineering.com GNU/Linux

Re: [PATCH 5/8] LinuxPPS: pps_parport: Do not generate assert in case of lost signal.

2017-03-03 Thread Rodolfo Giometti
On 02/15/17 15:31, Andrey Drobyshev wrote: From: Alexander GQ Gerasiov Since clear timeout is non-zero, some clear event capture is requested. Therefore, if signal is lost we shouldn't generate assert event alone. Signed-off-by: Alexander GQ Gerasiov Acked-by: Rodolfo Giometti

Re: [PATCH 4/8] LinuxPPS: kapi: Unlock before waking up events queue in pps_event().

2017-03-03 Thread Rodolfo Giometti
On 02/15/17 15:31, Andrey Drobyshev wrote: From: Alexander GQ Gerasiov Otherwise we get "scheduling while atomic" problem. Signed-off-by: Alexander GQ Gerasiov Acked-by: Rodolfo Giometti

Re: [PATCH 3/8] hardpps: fix some pps_jitter issues.

2017-03-03 Thread Rodolfo Giometti
On 02/15/17 15:31, Andrey Drobyshev wrote: Handle possible overflow, implementation-defined result of signed right shift and replace unsuitable constant. Signed-off-by: Andrey Drobyshev Signed-off-by: Alexander GQ Gerasiov Acked-by: Rodolfo Giometti

Re: [PATCH 1/8] hardpps: simple fixes replacing clumsy code with abs() macro.

2017-03-03 Thread Rodolfo Giometti
On 02/15/17 15:31, Andrey Drobyshev wrote: From: Alexander GQ Gerasiov Here are some trivial fixes, including: * simplify comparisons by using abs() macro * remove unnecessary variables * remove duplicate headers * fix typos Signed-off-by: Alexander GQ Gerasiov Acked-by: Rodolfo

Re: [RFC] pps: fixing CONFIG_COMPAT issues

2016-12-23 Thread Rodolfo Giometti
On 12/22/16 21:39, Matt Ranostay wrote: What would be the best way to fix the padding issue without breaking userspace applications? Just fixing the alignment with explicit padding is of course the clean easy way, but bashing the data in compat_ioctl would avoid breakage. Hi Matt, I've no exp

Re: [PATCH][RESEND] pps: kc: fix non-tickless system config dependency

2016-09-21 Thread Rodolfo Giometti
system which has CONFIG_NO_HZ not set and CONFIG_NO_HZ_IDLE (or CONFIG_NO_HZ_FULL) set. Signed-off-by: Maciej S. Szmigiero --- This is a resend without functional changes since previous submission and resubmission weren't picked up. @Rodolfo: Could you ack this, please? Acked-by: Rodolfo Gio

Re: [PATCH -resend] pps: do not crash when failed to register

2016-07-14 Thread Rodolfo Giometti
NULL in the test there too. Signed-off-by: Jiri Slaby Cc: Rodolfo Giometti --- drivers/pps/clients/pps_parport.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/pps/clients/pps_parport.c b/drivers/pps/clients/pps_parport.c index 38a8bbe74810..83797d89c30f 100644 --

Re: [PATCH] pps: kc: fix non-tickless system config dependency

2016-03-21 Thread Rodolfo Giometti
On Mon, Mar 21, 2016 at 03:28:26PM +0100, Maciej S. Szmigiero wrote: > Hi Rodolfo, > > On 06.03.2016 20:13, Rodolfo Giometti wrote: > > On Sat, Mar 05, 2016 at 06:37:39PM +0100, Maciej S. Szmigiero wrote: > >> CONFIG_NO_HZ currently only sets the default value > >

Re: [PATCH] pps: kc: fix non-tickless system config dependency

2016-03-06 Thread Rodolfo Giometti
On Sat, Mar 05, 2016 at 06:37:39PM +0100, Maciej S. Szmigiero wrote: > CONFIG_NO_HZ currently only sets the default value > of dynticks config so if PPS kernel consumer needs > periodic timer ticks it should depend on > !CONFIG_NO_HZ_COMMON instead of !CONFIG_NO_HZ. > > Otherwise it is possible to

Re: [PATCH] Doc: pps: Fix file name in pps.txt

2015-07-13 Thread Rodolfo Giometti
On Mon, Jul 13, 2015 at 07:21:50AM -0600, Jonathan Corbet wrote: > On Mon, 13 Jul 2015 08:23:15 +0200 > Rodolfo Giometti wrote: > > > On Mon, Jul 13, 2015 at 12:29:11PM +0900, Masanari Iida wrote: > > > This patch fix a file name of example code. > > > &

Re: [PATCH] Doc: pps: Fix file name in pps.txt

2015-07-12 Thread Rodolfo Giometti
On Mon, Jul 13, 2015 at 12:29:11PM +0900, Masanari Iida wrote: > This patch fix a file name of example code. > > Signed-off-by: Masanari Iida Signed-off-by: Rodolfo Giometti -- HCE Engineering e-mail: giome...@hce-engineering.com GNU/Linux

Re: [PATCH] Doc:pps: Fix typo in pps.txt

2015-06-10 Thread Rodolfo Giometti
On Wed, Jun 10, 2015 at 12:53:47PM +0900, Masanari Iida wrote: > This patch fix a spelling typo in Documentation/pps/pps.txt > > Signed-off-by: Masanari Iida Acked-by: Rodolfo Giometti -- HCE Engineering e-mail: giome...@hce-engineering.com GNU/Linux

Re: [PATCH] pps: Add support for read operations and return a useful value in poll

2015-04-01 Thread Rodolfo Giometti
On Tue, Mar 31, 2015 at 11:31:22PM +0200, Christian Riesch wrote: > The PPS_FETCH ioctl in drivers/pps/pps.c blocks until a new PPS event > occurs, then returns the time stamp data. While this is fine for > lots of applications, sometimes it would be nice if the poll system > call and a subsequent

Re: [PATCH v2 15/15] ARM: at91/dt: at91-cosino_mega2560 remove useless tsadcc node

2014-03-17 Thread Rodolfo Giometti
On Mon, Mar 17, 2014 at 02:57:08PM +0100, Alexandre Belloni wrote: > The tsadcc node is useless as it doesn't refer to anything and the touchscreen > is handled by the adc0 node. > > Signed-off-by: Alexandre Belloni > Cc: Rodolfo Giometti Acked. Rodolfo --

Re: [PATCH] pps : add non blocking option to PPS_FETCH ioctl.

2013-10-16 Thread Rodolfo Giometti
On Wed, Oct 16, 2013 at 08:52:47AM +0200, Paul Chavent wrote: > > I would also prefer the separate ioctl. As you said it, it's a bit > annoying to switch from blocking mode to non blocking mode if we > need both mode. But i was not sure about the preferences of the > maintainer : (i) change the ap

Re: [PATCH] pps : add non blocking option to PPS_FETCH ioctl.

2013-10-15 Thread Rodolfo Giometti
On Fri, Oct 11, 2013 at 12:47:20PM -0700, Andrew Morton wrote: > On Fri, 11 Oct 2013 14:40:32 +0200 Paul Chavent wrote: > > > The PPS_FETCH ioctl is blocking still the reception of a PPS > > event. But, in some case, one may immediately need the last event > > date. This patch allow to get the re

Re: [PATCH] pps : add non blocking option to PPS_FETCH ioctl.

2013-10-11 Thread Rodolfo Giometti
On Fri, Oct 11, 2013 at 10:42:56AM +0200, Paul Chavent wrote: > Hi. > > I haven't had any feedback for weeks. I join a patch with more > details in the commit message if needed. If I well remember I already acked this patch. So please add my Acked-by line to the patch. > Do you know someone resp

Re: [PATCH 2/2] USB : serial : invoke dcd_change ldisc's handler.

2013-09-16 Thread Rodolfo Giometti
as the uart drivers, so > document this point too. > > Signed-off-by: Paul Chavent Acked-by: Rodolfo Giometti -- GNU/Linux Solutions e-mail: giome...@enneenne.com Linux Device Driver giome...@linux.it Embedded Systems

Re: [PATCH 2/3] USB : serial : invoke dcd_change ldisc's handler.

2013-09-13 Thread Rodolfo Giometti
On Fri, Sep 13, 2013 at 05:35:12PM +0200, Paul Chavent wrote: > Signed-off-by: Paul Chavent Acked-by: Rodolfo Giometti -- GNU/Linux Solutions e-mail: giome...@enneenne.com Linux Device Driver giome...@linux.it Embedded Syst

Re: [PATCH 0/5] Enable PPS reporting for USB serial devices

2013-09-12 Thread Rodolfo Giometti
it into PPS documentation and/or (better) in configuration menu! :) Then you can add my "Acked-by: Rodolfo Giometti " Ciao, Rodolfo -- GNU/Linux Solutions e-mail: giome...@enneenne.com Linux Device Driver giome...@linux.it Embedded Systems

Re: [PATCH 0/5] Enable PPS reporting for USB serial devices

2013-09-10 Thread Rodolfo Giometti
On Mon, Sep 09, 2013 at 06:01:15PM +0200, Paul Chavent wrote: > Hi. > > This series enable the PPS reporting for USB serial devices. I have nothing against with this solution but consider that reporting a PPS signal through USB bus may add unknown delays that may vanish clock stability... as far

Re: [PATCH] pps : add non blocking option to PPS_FETCH ioctl.

2013-09-06 Thread Rodolfo Giometti
On Wed, Sep 04, 2013 at 10:20:38AM +0200, Paul Chavent wrote: > Signed-off-by: Paul Chavent Acked: Rodolfo Giometti -- GNU/Linux Solutions e-mail: giome...@enneenne.com Linux Device Driver giome...@linux.it Embedded Systems ph

Re: [PATCH v2] pps-gpio: add device-tree binding and support

2013-06-03 Thread Rodolfo Giometti
direction. > > Also use dev_err instead of pr_err in the changed code. > > Signed-off-by: Jan Luebbe Acked-by: Rodolfo Giometti -- GNU/Linux Solutions e-mail: giome...@enneenne.com Linux Device Driver giome...@linux.it Embedded Systems

Re: [PATCH] pps-gpio: add device-tree binding and support

2013-06-03 Thread Rodolfo Giometti
On Fri, May 31, 2013 at 08:46:55PM +0200, Jan Luebbe wrote: > Signed-off-by: Jan Luebbe Acked-by: Rodolfo Giometti -- GNU/Linux Solutions e-mail: giome...@enneenne.com Linux Device Driver giome...@linux.it Embedded Systems ph

Re: [PATCH 1/2] pps-gpio: convert to devm_* helpers

2013-04-23 Thread Rodolfo Giometti
On Mon, Apr 22, 2013 at 10:38:24AM +0200, Jan Lübbe wrote: > Rodolfo, would you take these patches? They are ok for me. Sorry for the delay... :'( ROdolfo -- GNU/Linux Solutions e-mail: giome...@enneenne.com Linux Device Driver giome...@linux.it Embed

Re: [PATCH 1/2] pps-gpio: convert to devm_* helpers

2013-04-22 Thread Rodolfo Giometti
On Mon, Apr 22, 2013 at 10:38:24AM +0200, Jan Lübbe wrote: > Rodolfo, would you take these patches? Yes! Sorry for delay... I'm going to review them today. Ciao, Rodolfo -- GNU/Linux Solutions e-mail: giome...@enneenne.com Linux Device Driver giome...

Re: [PATCH v2 0/9] 3.8-rc regression with pps-ldisc due to 70ece7a731

2013-02-13 Thread Rodolfo Giometti
On Wed, Feb 13, 2013 at 08:45:47AM -0800, Greg KH wrote: > > I can take these through my tty tree, but it would be very good if I > actually had the ack from the PPS maintainer... > > Rodolfo, any objection for me taking these? No. It's ok for me. If you wish you can add my «Acked-by» sign. Cia

[PATCH] PPS: parallel port clients support.

2008-02-22 Thread Rodolfo Giometti
Adds support for the PPS sources connected with the interrupt pin of a parallel port. Signed-off-by: Rodolfo Giometti <[EMAIL PROTECTED]> --- drivers/char/lp.c | 61 +++ drivers/pps/clients/Kconfig | 10 +++ include/linux/par

[PATCH] PPS: LinuxPPS clients support.

2008-02-22 Thread Rodolfo Giometti
ent that can be useful to test system integrity on real systems. Signed-off-by: Rodolfo Giometti <[EMAIL PROTECTED]> --- drivers/pps/Kconfig |2 + drivers/pps/Makefile |1 + drivers/pps/clients/Kconfig | 18 +++ drivers/pps/clients/Makefile |9 +++ drivers

[PATCH] PPS: userland header file for PPS API.

2008-02-22 Thread Rodolfo Giometti
This patch adds into the PPS's documentation directory a possible implementation of the PPS API (RFC 2783) by using the LinuxPPS's char devices. This file is not just an example but it can be used into real systems. :) Signed-off-by: Rodolfo Giometti <[EMAIL PROTECTED]> --- D

[PATCH] LinuxPPS core support.

2008-02-22 Thread Rodolfo Giometti
n 1.0) which in part is implemented by this patch. It provides a set of chars devices, one per PPS source, which can be used to get the time signal. The RFC's functions can be implemented by accessing to these char devices. Signed-off-by: Rodolfo Giometti <[EMAIL PROTECTED]> --- Docu

[PATCH] PPS: serial clients support.

2008-02-22 Thread Rodolfo Giometti
Adds support for the PPS sources connected with the CD (Carrier Detect) pin of a serial port. Signed-off-by: Rodolfo Giometti <[EMAIL PROTECTED]> --- drivers/pps/clients/Kconfig | 10 ++ drivers/serial/8250.c|2 + drivers/serial/serial_core.c

[PATCH] PPS: example program to enable PPS support on serial ports.

2008-02-22 Thread Rodolfo Giometti
Signed-off-by: Rodolfo Giometti <[EMAIL PROTECTED]> --- Documentation/pps/Makefile |2 +- Documentation/pps/ppsctl.c | 62 2 files changed, 63 insertions(+), 1 deletions(-) create mode 100644 Documentation/pps/ppsctl.c diff -

[PATCH] PPS: documentation programs and examples.

2008-02-22 Thread Rodolfo Giometti
Here some utilities and examples about the PPS API and the LinuxPPS support. * ppstest.c implements an useful testing program, while * ppsfind tries to help the user into finding a specific PPS source by using its name or path. Signed-off-by: Rodolfo Giometti <[EMAIL PROTEC

LinuxPPS (RESUBMIT): the PPS Linux implementation.

2008-02-22 Thread Rodolfo Giometti
This patch set adds the PPS support into Linux. PPS means "pulse per second" and its API is specified by RFC 2783 (Pulse-Per-Second API for UNIX-like Operating Systems, Version 1.0). The code has been tested with the NTPD program (http://www.eecis.udel.edu/~mills/ntp/html/index.html) and several

[PATCH] PPS: serial clients support.

2008-02-18 Thread Rodolfo Giometti
Adds support for the PPS sources connected with the CD (Carrier Detect) pin of a serial port. Signed-off-by: Rodolfo Giometti <[EMAIL PROTECTED]> --- drivers/pps/clients/Kconfig | 10 ++ drivers/serial/8250.c|2 + drivers/serial/serial_core.c

[PATCH] LinuxPPS core support.

2008-02-18 Thread Rodolfo Giometti
n 1.0) which in part is implemented by this patch. It provides a set of chars devices, one per PPS source, which can be used to get the time signal. The RFC's functions can be implemented by accessing to these char devices. Signed-off-by: Rodolfo Giometti <[EMAIL PROTECTED]> --- Docu

[PATCH] PPS: userland header file for PPS API.

2008-02-18 Thread Rodolfo Giometti
This patch adds into the PPS's documentation directory a possible implementation of the PPS API (RFC 2783) by using the LinuxPPS's char devices. This file is not just an example but it can be used into real systems. :) Signed-off-by: Rodolfo Giometti <[EMAIL PROTECTED]> --- D

[PATCH] PPS: documentation programs and examples.

2008-02-18 Thread Rodolfo Giometti
Here some utilities and examples about the PPS API and the LinuxPPS support. * ppstest.c implements an useful testing program, while * ppsfind tries to help the user into finding a specific PPS source by using its name or path. Signed-off-by: Rodolfo Giometti <[EMAIL PROTEC

[PATCH] PPS: example program to enable PPS support on serial ports.

2008-02-18 Thread Rodolfo Giometti
Signed-off-by: Rodolfo Giometti <[EMAIL PROTECTED]> --- Documentation/pps/Makefile |2 +- Documentation/pps/ppsctl.c | 62 2 files changed, 63 insertions(+), 1 deletions(-) create mode 100644 Documentation/pps/ppsctl.c diff -

[PATCH] PPS: LinuxPPS clients support.

2008-02-18 Thread Rodolfo Giometti
ent that can be useful to test system integrity on real systems. Signed-off-by: Rodolfo Giometti <[EMAIL PROTECTED]> --- drivers/pps/Kconfig |2 + drivers/pps/Makefile |1 + drivers/pps/clients/Kconfig | 18 +++ drivers/pps/clients/Makefile |9 +++ drivers

LinuxPPS (RESUBMIT): the PPS Linux implementation.

2008-02-18 Thread Rodolfo Giometti
This patch set adds the PPS support into Linux. PPS means "pulse per second" and its API is specified by RFC 2783 (Pulse-Per-Second API for UNIX-like Operating Systems, Version 1.0). The code has been tested with the NTPD program (http://www.eecis.udel.edu/~mills/ntp/html/index.html) and several

[PATCH] PPS: parallel port clients support.

2008-02-18 Thread Rodolfo Giometti
Adds support for the PPS sources connected with the interrupt pin of a parallel port. Signed-off-by: Rodolfo Giometti <[EMAIL PROTECTED]> --- drivers/char/lp.c | 61 +++ drivers/pps/clients/Kconfig | 10 +++ include/linux/par

Re: [PATCH 1/8] LinuxPPS core support.

2008-02-11 Thread Rodolfo Giometti
On Mon, Feb 11, 2008 at 07:31:53PM +0100, Rodolfo Giometti wrote: > Do you think is better create these files anyway and in case report a > NULL string? Is that ok? /* * Attribute functions */ static ssize_t pps_show_assert(struct device *dev,

Re: [PATCH 1/8] LinuxPPS core support.

2008-02-11 Thread Rodolfo Giometti
On Mon, Feb 11, 2008 at 07:25:42AM -0800, Greg KH wrote: > On Mon, Feb 11, 2008 at 10:53:46AM +0100, Rodolfo Giometti wrote: > > On Fri, Feb 08, 2008 at 10:45:31PM -0800, Greg KH wrote: > > > > > > As you are adding sysfs files, please also describe them in > >

Re: [PATCH 1/8] LinuxPPS core support.

2008-02-11 Thread Rodolfo Giometti
On Fri, Feb 08, 2008 at 10:45:31PM -0800, Greg KH wrote: > > As you are adding sysfs files, please also describe them in > Documentation/ABI/ in this same series of patches. Ok. I'm going to add them under "testing". > Why not use a default attribute group? > > That way the files are created be

[PATCH 1/8] LinuxPPS core support.

2008-02-08 Thread Rodolfo Giometti
n 1.0) which in part is implemented by this patch. It provides a set of chars devices, one per PPS source, which can be used to get the time signal. The RFC's functions can be implemented by accessing to these char devices. Signed-off-by: Rodolfo Giometti <[EMAIL PROTECTED]> --- MAINTAI

[PATCH 6/8] PPS: serial clients support.

2008-02-08 Thread Rodolfo Giometti
Adds support for the PPS sources connected with the CD (Carrier Detect) pin of a serial port. Signed-off-by: Rodolfo Giometti <[EMAIL PROTECTED]> --- drivers/pps/clients/Kconfig | 10 ++ drivers/serial/8250.c|2 + drivers/serial/serial_core.c

[PATCH 2/8] PPS: basic documentation file.

2008-02-08 Thread Rodolfo Giometti
Signed-off-by: Rodolfo Giometti <[EMAIL PROTECTED]> --- Documentation/pps/pps.txt | 170 + 1 files changed, 170 insertions(+), 0 deletions(-) create mode 100644 Documentation/pps/pps.txt diff --git a/Documentation/pps/pps.txt b/Documentati

[PATCH 5/8] PPS: LinuxPPS clients support.

2008-02-08 Thread Rodolfo Giometti
ent that can be useful to test system integrity on real systems. Signed-off-by: Rodolfo Giometti <[EMAIL PROTECTED]> --- drivers/pps/Kconfig |2 + drivers/pps/Makefile |1 + drivers/pps/clients/Kconfig | 18 +++ drivers/pps/clients/Makefile |9 +++ drivers

[PATCH 3/8] PPS: userland header file for PPS API.

2008-02-08 Thread Rodolfo Giometti
This patch adds into the PPS's documentation directory a possible implementation of the PPS API (RFC 2783) by using the LinuxPPS's char devices. This file is not just an example but it can be used into real systems. :) Signed-off-by: Rodolfo Giometti <[EMAIL PROTECTED]> --- D

[PATCH 7/8] PPS: example program to enable PPS support on serial ports.

2008-02-08 Thread Rodolfo Giometti
Signed-off-by: Rodolfo Giometti <[EMAIL PROTECTED]> --- Documentation/pps/Makefile |2 +- Documentation/pps/ppsctl.c | 62 2 files changed, 63 insertions(+), 1 deletions(-) create mode 100644 Documentation/pps/ppsctl.c diff -

[PATCH 8/8] PPS: parallel port clients support.

2008-02-08 Thread Rodolfo Giometti
Adds support for the PPS sources connected with the interrupt pin of a parallel port. Signed-off-by: Rodolfo Giometti <[EMAIL PROTECTED]> --- drivers/char/lp.c | 61 +++ drivers/pps/clients/Kconfig | 10 +++ include/linux/par

[PATCH 4/8] PPS: documentation programs and examples.

2008-02-08 Thread Rodolfo Giometti
Here some utilities and examples about the PPS API and the LinuxPPS support. * ppsctl.c implements an useful testing program, while * ppsfind tries to help the user into finding a specific PPS source by using its name or path. Signed-off-by: Rodolfo Giometti <[EMAIL PROTEC

LinuxPPS: the PPS Linux implementation.

2008-02-08 Thread Rodolfo Giometti
This patch set adds the PPS support into Linux. PPS means "pulse per second" and its API is specified by RFC 2783 (Pulse-Per-Second API for UNIX-like Operating Systems, Version 1.0). The code has been tested with the NTPD program (http://www.eecis.udel.edu/~mills/ntp/html/index.html) and several

Re: LinuxPPS patch

2008-01-28 Thread Rodolfo Giometti
On Mon, Jan 28, 2008 at 08:43:08AM -0500, Lennart Sorensen wrote: > I would test it except I am still on 2.6.18 and probably will be for > quite a while. I am still using an older patch of yours which works > great for now, although I know the interface has a changed a bit since. Yes, you are re

LinuxPPS patch

2008-01-27 Thread Rodolfo Giometti
Hello, here my patch for PPS support against kernel 2.6.24: http://ftp.enneenne.com/pub/misc/linuxpps/patches/ntp-pps-2.6.24.diff The code has been tested by some guys intered in PPS stuff and it seems ok. :) Please report your suggestions in order to get kernel inclusion. Thanks in advance

heartbeat for leds on i2c bus

2007-12-03 Thread Rodolfo Giometti
Hello, on my custom board I have some leds mapped on a dedicated chip connected via i2c bus. If I try to use the heartbeat trigger I hang the system... I suppose it is due the fact the led_set_brightness() function is called inside a kernel timer. How I can solve this problem? I thought to modif

  1   2   3   >