From: Lee Jones <lee.jones@linaro.org>
To: Alexandre Belloni <alexandre.belloni@bootlin.com>
Cc: Frank Wunderlich <frank-w@public-files.de>,
Alessandro Zummo <a.zummo@towertech.it>,
Allison Randal <allison@lohutok.net>,
"David S. Miller" <davem@davemloft.net>,
devicetree@vger.kernel.org,
Eddie Huang <eddie.huang@mediatek.com>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Jonathan Cameron <Jonathan.Cameron@huawei.com>,
Kate Stewart <kstewart@linuxfoundation.org>,
linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org, linux-mediatek@lists.infradead.org,
linux-pm@vger.kernel.org, linux-rtc@vger.kernel.org,
Mark Rutland <mark.rutland@arm.com>,
Matthias Brugger <matthias.bgg@gmail.com>,
Mauro Carvalho Chehab <mchehab+samsung@kernel.org>,
Nicolas Ferre <nicolas.ferre@microchip.com>,
Richard Fontana <rfontana@redhat.com>,
Rob Herring <robh+dt@kernel.o>
Subject: Re: [PATCH v5 03/10] rtc: mt6397: move some common definitions into rtc.h
Date: Tue, 13 Aug 2019 08:20:18 +0100 [thread overview]
Message-ID: <20190813072018.GZ26727@dell> (raw)
In-Reply-To: <20190812153856.GR3600@piout.net>
On Mon, 12 Aug 2019, Alexandre Belloni wrote:
> On 12/08/2019 16:11:29+0100, Lee Jones wrote:
> > On Mon, 12 Aug 2019, Frank Wunderlich wrote:
> >
> > > From: Josef Friedl <josef.friedl@speed.at>
> > >
> > > move code to separate header-file to reuse definitions later
> > > in poweroff-driver (drivers/power/reset/mt6323-poweroff.c)
> > >
> > > Suggested-by: Frank Wunderlich <frank-w@public-files.de>
> > > Signed-off-by: Josef Friedl <josef.friedl@speed.at>
> > > Signed-off-by: Frank Wunderlich <frank-w@public-files.de>
> > > ---
> > > changes since v4: none
> > > changes since v3: none
> > > changes since v2: add missing commit-message
> > > ---
> > > drivers/rtc/rtc-mt6397.c | 55 +-------------------------
> > > include/linux/mfd/mt6397/rtc.h | 71 ++++++++++++++++++++++++++++++++++
> >
> > include/linux/rtc/mt6397.h?
> >
>
> It's just a bunch of registers and bit definitions so it may as well
> stay close to include/linux/mfd/mt6397/registers.h
Sure. It was a weak preference.
--
Lee Jones [李琼斯]
Linaro Services Technical Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
WARNING: multiple messages have this Message-ID (diff)
From: Lee Jones <lee.jones@linaro.org>
To: Alexandre Belloni <alexandre.belloni@bootlin.com>
Cc: Frank Wunderlich <frank-w@public-files.de>,
Alessandro Zummo <a.zummo@towertech.it>,
Allison Randal <allison@lohutok.net>,
"David S. Miller" <davem@davemloft.net>,
devicetree@vger.kernel.org,
Eddie Huang <eddie.huang@mediatek.com>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Jonathan Cameron <Jonathan.Cameron@huawei.com>,
Kate Stewart <kstewart@linuxfoundation.org>,
linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org, linux-mediatek@lists.infradead.org,
linux-pm@vger.kernel.org, linux-rtc@vger.kernel.org,
Mark Rutland <mark.rutland@arm.com>,
Matthias Brugger <matthias.bgg@gmail.com>,
Mauro Carvalho Chehab <mchehab+samsung@kernel.org>,
Nicolas Ferre <nicolas.ferre@microchip.com>,
Richard Fontana <rfontana@redhat.com>,
Rob Herring <robh+dt@kernel.org>,
Sean Wang <sean.wang@mediatek.com>,
Sebastian Reichel <sre@kernel.org>,
Thomas Gleixner <tglx@linutronix.de>,
"Tianping . Fang" <tianping.fang@mediatek.com>,
Josef Friedl <josef.friedl@speed.at>
Subject: Re: [PATCH v5 03/10] rtc: mt6397: move some common definitions into rtc.h
Date: Tue, 13 Aug 2019 08:20:18 +0100 [thread overview]
Message-ID: <20190813072018.GZ26727@dell> (raw)
In-Reply-To: <20190812153856.GR3600@piout.net>
On Mon, 12 Aug 2019, Alexandre Belloni wrote:
> On 12/08/2019 16:11:29+0100, Lee Jones wrote:
> > On Mon, 12 Aug 2019, Frank Wunderlich wrote:
> >
> > > From: Josef Friedl <josef.friedl@speed.at>
> > >
> > > move code to separate header-file to reuse definitions later
> > > in poweroff-driver (drivers/power/reset/mt6323-poweroff.c)
> > >
> > > Suggested-by: Frank Wunderlich <frank-w@public-files.de>
> > > Signed-off-by: Josef Friedl <josef.friedl@speed.at>
> > > Signed-off-by: Frank Wunderlich <frank-w@public-files.de>
> > > ---
> > > changes since v4: none
> > > changes since v3: none
> > > changes since v2: add missing commit-message
> > > ---
> > > drivers/rtc/rtc-mt6397.c | 55 +-------------------------
> > > include/linux/mfd/mt6397/rtc.h | 71 ++++++++++++++++++++++++++++++++++
> >
> > include/linux/rtc/mt6397.h?
> >
>
> It's just a bunch of registers and bit definitions so it may as well
> stay close to include/linux/mfd/mt6397/registers.h
Sure. It was a weak preference.
--
Lee Jones [李琼斯]
Linaro Services Technical Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
WARNING: multiple messages have this Message-ID (diff)
From: Lee Jones <lee.jones@linaro.org>
To: Alexandre Belloni <alexandre.belloni@bootlin.com>
Cc: Kate Stewart <kstewart@linuxfoundation.org>,
Mark Rutland <mark.rutland@arm.com>,
linux-kernel@vger.kernel.org,
Richard Fontana <rfontana@redhat.com>,
Mauro Carvalho Chehab <mchehab+samsung@kernel.org>,
linux-rtc@vger.kernel.org,
Frank Wunderlich <frank-w@public-files.de>,
Allison Randal <allison@lohutok.net>,
devicetree@vger.kernel.org, linux-pm@vger.kernel.org,
Sean Wang <sean.wang@mediatek.com>,
"Tianping . Fang" <tianping.fang@mediatek.com>,
Rob Herring <robh+dt@kernel.org>,
linux-mediatek@lists.infradead.org,
Jonathan Cameron <Jonathan.Cameron@huawei.com>,
Matthias Brugger <matthias.bgg@gmail.com>,
Thomas Gleixner <tglx@linutronix.de>,
Eddie Huang <eddie.huang@mediatek.com>,
linux-arm-kernel@lists.infradead.org,
Alessandro Zummo <a.zummo@towertech.it>,
Josef Friedl <josef.friedl@speed.at>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Sebastian Reichel <sre@kernel.org>,
"David S. Miller" <davem@davemloft.net>
Subject: Re: [PATCH v5 03/10] rtc: mt6397: move some common definitions into rtc.h
Date: Tue, 13 Aug 2019 08:20:18 +0100 [thread overview]
Message-ID: <20190813072018.GZ26727@dell> (raw)
In-Reply-To: <20190812153856.GR3600@piout.net>
On Mon, 12 Aug 2019, Alexandre Belloni wrote:
> On 12/08/2019 16:11:29+0100, Lee Jones wrote:
> > On Mon, 12 Aug 2019, Frank Wunderlich wrote:
> >
> > > From: Josef Friedl <josef.friedl@speed.at>
> > >
> > > move code to separate header-file to reuse definitions later
> > > in poweroff-driver (drivers/power/reset/mt6323-poweroff.c)
> > >
> > > Suggested-by: Frank Wunderlich <frank-w@public-files.de>
> > > Signed-off-by: Josef Friedl <josef.friedl@speed.at>
> > > Signed-off-by: Frank Wunderlich <frank-w@public-files.de>
> > > ---
> > > changes since v4: none
> > > changes since v3: none
> > > changes since v2: add missing commit-message
> > > ---
> > > drivers/rtc/rtc-mt6397.c | 55 +-------------------------
> > > include/linux/mfd/mt6397/rtc.h | 71 ++++++++++++++++++++++++++++++++++
> >
> > include/linux/rtc/mt6397.h?
> >
>
> It's just a bunch of registers and bit definitions so it may as well
> stay close to include/linux/mfd/mt6397/registers.h
Sure. It was a weak preference.
--
Lee Jones [李琼斯]
Linaro Services Technical Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2019-08-13 7:20 UTC|newest]
Thread overview: 54+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-08-12 12:15 [PATCH v5 00/10] implement poweroff for mt6323 / bpi-r2 Frank Wunderlich
2019-08-12 12:15 ` Frank Wunderlich
2019-08-12 12:15 ` Frank Wunderlich
2019-08-12 12:15 ` [PATCH v5 01/10] dt-bindings: add powercontroller Frank Wunderlich
2019-08-12 12:15 ` Frank Wunderlich
2019-08-12 12:15 ` Frank Wunderlich
2019-08-12 15:15 ` Lee Jones
2019-08-12 15:15 ` Lee Jones
2019-08-12 15:15 ` Lee Jones
2019-08-13 15:25 ` Rob Herring
2019-08-13 15:25 ` Rob Herring
2019-08-13 15:25 ` Rob Herring
2019-08-13 20:51 ` SRS0=c+xj=WJ=public-files.de=frank-w
2019-08-13 22:02 ` SRS0=c+xj=WJ=public-files.de=frank-w
[not found] ` <CAL_Jsq+fW=x8MoL3GMj6+ZKC3fd6F6+8Rtx=j5R9vEuL9UHCuQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2019-08-14 5:56 ` Frank Wunderlich
2019-08-12 12:15 ` [PATCH v5 02/10] dt-bindings: add missing mt6397 rtc Frank Wunderlich
2019-08-12 12:15 ` Frank Wunderlich
2019-08-12 12:15 ` Frank Wunderlich
2019-08-12 12:15 ` [PATCH v5 03/10] rtc: mt6397: move some common definitions into rtc.h Frank Wunderlich
2019-08-12 12:15 ` Frank Wunderlich
2019-08-12 12:15 ` Frank Wunderlich
2019-08-12 15:11 ` Lee Jones
2019-08-12 15:11 ` Lee Jones
2019-08-12 15:11 ` Lee Jones
2019-08-12 15:38 ` Alexandre Belloni
2019-08-12 15:38 ` Alexandre Belloni
2019-08-12 15:38 ` Alexandre Belloni
2019-08-13 7:20 ` Lee Jones [this message]
2019-08-13 7:20 ` Lee Jones
2019-08-13 7:20 ` Lee Jones
2019-08-12 12:15 ` [PATCH v5 04/10] rtc: mt6397: improvements of rtc driver Frank Wunderlich
2019-08-12 12:15 ` Frank Wunderlich
2019-08-12 12:15 ` Frank Wunderlich
2019-08-12 12:15 ` [PATCH v5 05/10] mfd: mt6323: some improvements of mt6397-core Frank Wunderlich
2019-08-12 12:15 ` Frank Wunderlich
2019-08-12 12:15 ` Frank Wunderlich
2019-08-12 12:15 ` [PATCH v5 06/10] rtc: mt6397: add compatible for mt6323 Frank Wunderlich
2019-08-12 12:15 ` Frank Wunderlich
2019-08-12 12:15 ` Frank Wunderlich
2019-08-12 12:15 ` [PATCH v5 07/10] mfd: mt6323: add mt6323 rtc+pwrc Frank Wunderlich
2019-08-12 12:15 ` Frank Wunderlich
2019-08-12 12:15 ` Frank Wunderlich
2019-08-12 12:15 ` [PATCH v5 08/10] power: reset: add driver for mt6323 poweroff Frank Wunderlich
2019-08-12 12:15 ` Frank Wunderlich
2019-08-12 12:15 ` Frank Wunderlich
2019-08-12 13:23 ` Lee Jones
2019-08-12 13:23 ` Lee Jones
2019-08-12 13:23 ` Lee Jones
2019-08-12 12:15 ` [PATCH v5 09/10] MAINTAINERS: add Mediatek shutdown drivers Frank Wunderlich
2019-08-12 12:15 ` Frank Wunderlich
2019-08-12 12:15 ` Frank Wunderlich
2019-08-12 12:15 ` [PATCH v5 10/10] arm: dts: mt6323: add keys, power-controller, rtc and codec Frank Wunderlich
2019-08-12 12:15 ` Frank Wunderlich
2019-08-12 12:15 ` Frank Wunderlich
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20190813072018.GZ26727@dell \
--to=lee.jones@linaro.org \
--cc=Jonathan.Cameron@huawei.com \
--cc=a.zummo@towertech.it \
--cc=alexandre.belloni@bootlin.com \
--cc=allison@lohutok.net \
--cc=davem@davemloft.net \
--cc=devicetree@vger.kernel.org \
--cc=eddie.huang@mediatek.com \
--cc=frank-w@public-files.de \
--cc=gregkh@linuxfoundation.org \
--cc=kstewart@linuxfoundation.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mediatek@lists.infradead.org \
--cc=linux-pm@vger.kernel.org \
--cc=linux-rtc@vger.kernel.org \
--cc=mark.rutland@arm.com \
--cc=matthias.bgg@gmail.com \
--cc=mchehab+samsung@kernel.org \
--cc=nicolas.ferre@microchip.com \
--cc=rfontana@redhat.com \
--cc=robh+dt@kernel.o \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.