From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rob Herring Subject: Re: [PATCH v3 3/4] rtc: isl1208: add support for isl1219 with tamper detection Date: Wed, 7 Mar 2018 16:02:50 -0600 Message-ID: <20180307220250.pfaboby7ax7baxp2@rob-hp-laptop> References: <1520246373-19023-1-git-send-email-Denis.Osterland@diehl.com> <1520246373-19023-4-git-send-email-Denis.Osterland@diehl.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <1520246373-19023-4-git-send-email-Denis.Osterland@diehl.com> Sender: linux-kernel-owner@vger.kernel.org To: Denis OSTERLAND Cc: "a.zummo@towertech.it" , "alexandre.belloni@free-electrons.com" , "linux-kernel@vger.kernel.org" , "mgr@pengutronix.de" , "m.grzeschik@pengutronix.de" , "devicetree@vger.kernel.org" , "linux@roeck-us.net" , "jdelvare@suse.com" , "linux-rtc@vger.kernel.org" , "kernel@pengutronix.de" List-Id: devicetree@vger.kernel.org On Mon, Mar 05, 2018 at 10:43:52AM +0000, Denis OSTERLAND wrote: > From: Michael Grzeschik > > We add support for the ISL1219 chip that got an integrated tamper > detection function. This patch implements the feature by adding > an additional timestamp0 file to sysfs device path. > This file contains seconds since epoch, if an event occurred, > or is empty, if none occurred. > > The devicetree documentation for the ISL1219 device tree > binding is added with an short example. It is not a trivial > device, because it supports two interrupt souces. > > Signed-off-by: Michael Grzeschik > Signed-off-by: Denis Osterland > --- > .../devicetree/bindings/rtc/isil,isl1219.txt | 28 ++++ While preferred to separate, I'm not going to ask for that if there are no other changes on the binding. Reviewed-by: Rob Herring > drivers/rtc/rtc-isl1208.c | 160 ++++++++++++++++++--- > 2 files changed, 171 insertions(+), 17 deletions(-) > create mode 100644 Documentation/devicetree/bindings/rtc/isil,isl1219.txt