Linux RTC
 help / color / mirror / Atom feed
From: Lee Jones <lee.jones@linaro.org>
To: S Twiss <stwiss.opensource@diasemi.com>
Cc: DEVICETREE <devicetree@vger.kernel.org>,
	Dmitry Torokhov <dmitry.torokhov@gmail.com>,
	Ian Campbell <ijc+devicetree@hellion.org.uk>,
	Kumar Gala <galak@codeaurora.org>,
	LINUXINPUT <linux-input@vger.kernel.org>,
	LINUXKERNEL <linux-kernel@vger.kernel.org>,
	Mark Rutland <mark.rutland@arm.com>,
	Pawel Moll <pawel.moll@arm.com>,
	RTCLINUX <rtc-linux@googlegroups.com>,
	Rob Herring <robh+dt@kernel.org>,
	Samuel Ortiz <sameo@linux.intel.com>,
	David Dajun Chen <david.chen@diasemi.com>,
	Support Opensource <support.opensource@diasemi.com>
Subject: [rtc-linux] Re: [PATCH V3 0/3]  da9062: Add DA9062 OnKey support using the existing DA9063 OnKey driver
Date: Tue, 28 Jul 2015 09:52:16 +0100	[thread overview]
Message-ID: <20150728085216.GP14943@x1> (raw)
In-Reply-To: <cover.1437668261.git.stwiss.opensource@diasemi.com>

> From: S Twiss <stwiss.opensource@diasemi.com>

Steve,

Can you sort out your git configuration please.  Your name should be
represented in full.  No abbreviations or synonyms please.

I would expect to see: 'Steve Twiss <stwiss.opensource@diasemi.com>'

The mailer is less important, but I suggest 'Steve Twiss' would be
more professional than either 'S Twiss' or 'Opensource [Steve Twiss]'.

I appreciate it might be company policy, but is there any chance of
reversing that to read 'Steve Twiss [Opensource]' instead?

> This patch set adds OnKey support for the Dialog DA9062 Power Management =
IC.
> Changes are made to the existing DA9063 OnKey component so that functiona=
lity
> in this device driver can be re-used to support the DA9062 OnKey.
>=20
> This following patch set re-uses the existing kernel OnKey driver for chi=
ps
> whose OnKey blocks are functionally similar to the DA9063 OnKey.
>=20
> The main points for the MFD core and device tree changes are as follows.
>=20
> - Alteration of the DA9063 OnKey Kconfig needs to be updated to depend on
>   both MFD_DA9063 "or" MFD_DA9062. There is no explicit DA9062 OnKey Kcon=
fig.
> - The DA9062 MFD core should add a new OnKey resource as usual and an ent=
ry
>   in the mfd_cell to support a component name and of_compatible for
>   "da9062-onkey" and "dlg,da9062-onkey".
> - The device tree binding support should include a compatible string for
>   "dlg,da9062-onkey"
>=20
> The main points for the OnKey changes are as follows:
>=20
> A generic structure is used (called da906x_chip_config) to hold all gener=
ic
> registers and bitmasks for use with this type of OnKey component.
>=20
> Functions in the DA9063 OnKey will be refactored to use this compatibilit=
y
> struct and all accesses to generic registers/masks will be made through
> this table look-up instead of using defines from the register header file=
s
> directly
>=20
> Linkage between the DA9062 MFD and the DA9063 OnKey driver is created thr=
ough
> the use of an of_match_table entry in the platform_driver structure.
> A re-work of da9063_onkey_probe() is necessary to use the of_match_node()=
 and
> dev_get_regmap() functions: this will provide initialisation of the gener=
ic
> registers and masks and allow access to the regmap according to the corre=
ct
> device tree specification.
>=20
> The addition of a of_device_id table for DA9063 and DA9062 default data
> is created.=20
>=20
> In this patch set the following is provided:
>  - [PATCH V3 1/3]: MFD changes in DA9062 to support OnKey
>  - [PATCH V3 2/3]: Update existing DA9063 OnKey to add DA9062 support
>  - [PATCH V3 3/3]: Device tree bindings for DA9062 OnKey component
>=20
> This patch applies against linux-next and next-20150708=20
>=20
> Thank you,
> Steve Twiss, Dialog Semiconductor Ltd.
>=20
> S Twiss (3):
>   mfd: da9062: Support for the DA9063 OnKey in the DA9062 core
>   onkey: da9063: Add DA9062 OnKey capability to DA9063 OnKey driver
>   devicetree: da9062: Add device tree bindings for DA9062 OnKey
>=20
>  .../devicetree/bindings/input/da9062-onkey.txt     |  36 ++++++
>  Documentation/devicetree/bindings/mfd/da9062.txt   |   3 +
>  drivers/input/misc/Kconfig                         |   8 +-
>  drivers/input/misc/da9063_onkey.c                  | 129 +++++++++++++++=
++----
>  drivers/mfd/da9062-core.c                          |  11 ++
>  5 files changed, 158 insertions(+), 29 deletions(-)
>  create mode 100644 Documentation/devicetree/bindings/input/da9062-onkey.=
txt
>=20

--=20
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org =E2=94=82 Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog

--=20
--=20
You received this message because you are subscribed to "rtc-linux".
Membership options at http://groups.google.com/group/rtc-linux .
Please read http://groups.google.com/group/rtc-linux/web/checklist
before submitting a driver.
---=20
You received this message because you are subscribed to the Google Groups "=
rtc-linux" group.
To unsubscribe from this group and stop receiving emails from it, send an e=
mail to rtc-linux+unsubscribe@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

  parent reply	other threads:[~2015-07-28  8:52 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-07-23 16:17 [rtc-linux] [PATCH V3 0/3] da9062: Add DA9062 OnKey support using the existing DA9063 OnKey driver S Twiss
2015-07-23 16:17 ` [rtc-linux] [PATCH V3 2/3] onkey: da9063: Add DA9062 OnKey capability to " S Twiss
2015-09-15  7:57   ` [rtc-linux] " Stephen Twiss
2015-09-15  8:08   ` Opensource [Steve Twiss]
2015-07-23 16:17 ` [rtc-linux] [PATCH V3 1/3] mfd: da9062: Support for the DA9063 OnKey in the DA9062 core S Twiss
2015-07-28  8:42   ` [rtc-linux] " Lee Jones
2015-07-23 16:17 ` [rtc-linux] [PATCH V3 3/3] devicetree: da9062: Add device tree bindings for DA9062 OnKey S Twiss
2015-07-27 22:43   ` [rtc-linux] " Dmitry Torokhov
2015-07-28  5:40     ` Dmitry Torokhov
2015-07-28  7:42       ` Lee Jones
2015-07-28  7:44         ` [rtc-linux] " Opensource [Steve Twiss]
2015-07-28  7:42       ` Opensource [Steve Twiss]
2015-07-28  8:40       ` [rtc-linux] " Lee Jones
2015-07-28 17:20         ` Dmitry Torokhov
2015-07-28 17:30           ` Lee Jones
2015-09-28  8:19       ` [rtc-linux] " Opensource [Steve Twiss]
2015-09-29 23:54         ` [rtc-linux] " Dmitry Torokhov
2015-09-30  7:45           ` [rtc-linux] " Opensource [Steve Twiss]
2015-09-30 11:47           ` [rtc-linux] " Lee Jones
2015-07-28  8:41   ` Lee Jones
2015-07-28  8:52 ` Lee Jones [this message]
2015-07-28 14:09   ` [rtc-linux] RE: [PATCH V3 0/3] da9062: Add DA9062 OnKey support using the existing DA9063 OnKey driver Opensource [Steve Twiss]

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=20150728085216.GP14943@x1 \
    --to=lee.jones@linaro.org \
    --cc=david.chen@diasemi.com \
    --cc=devicetree@vger.kernel.org \
    --cc=dmitry.torokhov@gmail.com \
    --cc=galak@codeaurora.org \
    --cc=ijc+devicetree@hellion.org.uk \
    --cc=linux-input@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mark.rutland@arm.com \
    --cc=pawel.moll@arm.com \
    --cc=robh+dt@kernel.org \
    --cc=rtc-linux@googlegroups.com \
    --cc=sameo@linux.intel.com \
    --cc=stwiss.opensource@diasemi.com \
    --cc=support.opensource@diasemi.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox