on these patches. If
> > > you want I resend or something else just let me know please.
> > >
> >
> > I don't remember the specifics but I think the patches have the
> > necessary acks from me to go through the mfd tree (didn't we agree on
>
On Sun, 13 Aug 2017, Wolfram Sang wrote:
> On Tue, May 23, 2017 at 08:17:27AM +0100, Lee Jones wrote:
> > On Mon, 22 May 2017, Wolfram Sang wrote:
> >
> > > include/linux/i2c is not for client devices. Move the header file to a
> > > more appropriate location.
&g
On Sun, 13 Aug 2017, Wolfram Sang wrote:
> On Thu, Jul 06, 2017 at 08:03:52AM +0100, Lee Jones wrote:
> > On Thu, 06 Jul 2017, Thierry Reding wrote:
> >
> > > On Mon, May 22, 2017 at 12:02:10AM +0200, Wolfram Sang wrote:
> > > > include/linux/i2c is not for
ENSE))
> cros_ec_sensors_register(ec_dev);
>
> + /* Check whether this EC has RTC support */
> + if (cros_ec_check_features(ec_dev, EC_FEATURE_RTC))
> + cros_ec_rtc_register(ec_dev);
> +
> if (ec_dev->max_passthru) {
>
didn't know that. So are you saying that this patch will
> break support for devices like Pixel 2? I tested the patches on
> various devices but not on Pixel 2 so could be.
Does this affect the rest of the series?
Or should I review/apply as usual?
--
Lee Jones
Linaro STMicro
eletions(-)
> > rename include/linux/{i2c => mfd}/twl.h (100%)
>
> I didn't see this get applied yet, so just in case anyone was waiting
> for me (this is trivial, so I don't think there's a need):
You're not the last. :)
--
Lee Jones
Linaro STMicroelectron
| 2 +-
For my own reference:
Acked-for-MFD-by: Lee Jones
> drivers/rtc/rtc-dm355evm.c| 2 +-
> include/linux/{i2c => mfd}/dm355evm_msp.h | 0
> 4 files changed, 3 insertions(+), 3 deletions(-)
> rename include/linux/{i2c => mfd}/dm355evm_msp.h (100
twl4030-vibra.c | 2 +-
> drivers/mfd/twl-core.c | 6 +++---
> drivers/mfd/twl4030-audio.c | 2 +-
> drivers/mfd/twl4030-irq.c | 2 +-
> drivers/mfd/twl4030-power.c | 2 +-
> drivers/mfd/twl6030-irq.c | 2 +-
F
+301,10 @@ int cros_ec_register(struct cros_ec_device *ec_dev)
> if (cros_ec_check_features(ec_dev, EC_FEATURE_MOTION_SENSE))
> cros_ec_sensors_register(ec_dev);
>
> + /* Check whether this EC has RTC support */
> + if (cros_ec_check_features(ec_dev, EC_FEATURE_RT
and removes the MFD bits from the character device driver. Also
> makes independent the IIO driver from the character device as also has no
> sense.
>
> Signed-off-by: Enric Balletbo i Serra
> Acked-by: Jonathan Cameron
> ---
>
> As pointed by Lee Jones in this thre
release cycle. This should go through the MFD tree and can be picked
> independently of the other patches. Lee Jones I think this is for you.
>
> Changes since v1:
> - None
>
> include/linux/mfd/cros_ec.h | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
This patch is
On Mon, 03 Apr 2017, Enric Balletbo i Serra wrote:
>
>
> On 03/04/17 16:25, Lee Jones wrote:
> > On Fri, 24 Mar 2017, Enric Balletbo i Serra wrote:
> >
> >> From: Stephen Barber
> >>
> >> If the EC supports RTC host commands, expose an RTC
On Mon, 03 Apr 2017, Enric Balletbo i Serra wrote:
>
>
> On 03/04/17 16:30, Lee Jones wrote:
> > On Mon, 03 Apr 2017, Lee Jones wrote:
> >
> >> On Fri, 24 Mar 2017, Enric Balletbo i Serra wrote:
> >>
> >>> From: Vic Yang
> >>>
&
release cycle. This should go through the MFD tree and can be picked
> independently of the other patches. Lee Jones I think this is for you.
>
> include/linux/mfd/cros_ec.h | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
Applied to -fixes.
> diff --git a/include/linux
On Mon, 03 Apr 2017, Lee Jones wrote:
> On Fri, 24 Mar 2017, Enric Balletbo i Serra wrote:
>
> > From: Vic Yang
> >
> > For SPI, we can get up to 32 additional bytes for response preamble.
> > The current overhead (2 bytes) may cause problems when we try to rece
cros_ec_register(struct cros_ec_device *ec_dev)
> if (cros_ec_check_features(ec_dev, EC_FEATURE_MOTION_SENSE))
> cros_ec_sensors_register(ec_dev);
>
> + /* Check whether this EC has RTC support */
> + if (cros_ec_check_features(ec_dev, EC_FEATURE_RTC))
>
release cycle. This should go through the MFD tree and can be picked
> independently of the other patches. Lee Jones I think this is for you.
>
> include/linux/mfd/cros_ec.h | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
Applied, thanks.
> diff --git a/include/linux
w trivial bits inline, but more comments than anything that needs
> changing.
>
> Acked-by: Jonathan Cameron
>
> Lee, so who gets to do the immutable branch for this one?
I have no strong opinion.
> > ---
> >
> > As pointed by Lee Jones in this thread [1] we
On Wed, 15 Mar 2017, Enric Balletbo i Serra wrote:
> Hi Lee,
>
> On 15/03/17 11:24, Lee Jones wrote:
> > On Tue, 14 Mar 2017, Enric Balletbo i Serra wrote:
> >> On 14/03/17 14:59, Lee Jones wrote:
> >>> On Tue, 14 Feb 2017, Enric Balletbo i Serra wrot
On Tue, 14 Mar 2017, Enric Balletbo i Serra wrote:
> On 14/03/17 14:59, Lee Jones wrote:
> > On Tue, 14 Feb 2017, Enric Balletbo i Serra wrote:
> >
> >> From: Stephen Barber
> >>
> >> If the EC supports RTC host commands, expose an RTC device.
> >&
OTION_SENSE))
> cros_ec_sensors_register(ec);
>
> + /* check whether this EC instance has RTC host command support */
> + if (cros_ec_check_features(ec, EC_FEATURE_RTC))
> + cros_ec_rtc_register(ec);
> +
> return 0;
>
> dev_reg
On Tue, 14 Feb 2017, Enric Balletbo i Serra wrote:
> Hi Lee,
>
> On 23/01/17 13:14, Lee Jones wrote:
> > On Mon, 23 Jan 2017, Alexandre Belloni wrote:
> >
> >> On 19/01/2017 at 13:30:31 +0100, Enric Balletbo i Serra wrote :
> >>> From: Stephen Barber
>
> > Signed-off-by: Stephen Barber
> > Signed-off-by: Enric Balletbo i Serra
>
> Acked-by: Alexandre Belloni
>
> Lee, this can go through the mfd tree. Hopefully, nobody will add a new
> driver nearby ;)
Yes, I can take the set, once all of the Acks are obt
vice_probe(struct platform_device *pdev)
> if (cros_ec_check_features(ec, EC_FEATURE_MOTION_SENSE))
> cros_ec_sensors_register(ec);
>
> + /* check whether this EC instance has RTC host command support */
> + if (cros_ec_check_features(ec, EC_FEATURE_RTC))
> +
gt; if (cros_ec_check_features(ec, EC_FEATURE_MOTION_SENSE))
> > cros_ec_sensors_register(ec);
> >
> > + /* check whether this EC instance has RTC host command support */
> > + if (cros_ec_check_features(ec, EC_FEATURE_RTC))
> > + cros_ec_rtc_regi
gt; ---
> include/linux/mfd/cros_ec_commands.h | 8
> 1 file changed, 8 insertions(+)
Acked-by: Lee Jones
> diff --git a/include/linux/mfd/cros_ec_commands.h
> b/include/linux/mfd/cros_ec_commands.h
> index 80d401d..73f7a62 100644
> --- a/include/linux/mfd/cr
t; drivers/platform/chrome/cros_ec_proto.c | 20
> include/linux/mfd/cros_ec.h | 10 ++
Acked-by: Lee Jones
> 2 files changed, 30 insertions(+)
>
> diff --git a/drivers/platform/chrome/cros_ec_proto.c
> b/drivers/platform/chrome/cros_ec_proto
On Fri, 23 Dec 2016, Vesa Jääskeläinen wrote:
> Texas Instrument's TPS65910 has support for compensating RTC crystal
> inaccuracies. When enabled every hour RTC counter value will be compensated
> with two's complement value.
>
> Acked-by: Lee Jones
> Sign
> ---
> drivers/rtc/rtc-tps65910.c | 132
> +++
> include/linux/mfd/tps65910.h | 1 +
For the MFD change:
Acked-by: Lee Jones
> 2 files changed, 133 insertions(+)
>
> diff --git a/drivers/rtc/rtc-tps65910.c b/drivers/rtc/rtc-tps6
config MFD_MAX77686
> tristate "Maxim Semiconductor MAX77686/802 PMIC Support"
> depends on I2C
> - depends on OF
> + depends on OF || COMPILE_TEST
> select MFD_CORE
> select REGMAP_I2C
> select REGMAP_IRQ
--
Lee Jones
Linaro
++-
drivers/rtc/Kconfig | 4 +-
include/linux/mfd/rk808.h | 154 +++-
7 files changed, 516 insertions(+), 56 deletions(-)
--
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro
| 66 ++
drivers/rtc/rtc-pm8xxx.c | 1 +
include/dt-bindings/mfd/qcom-rpm.h | 22
7 files changed, 157 insertions(+)
--
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
On Thu, 11 Aug 2016, Neil Armstrong wrote:
> In order to support the Qualcomm MDM9615 SoC, add support for the
> RPM regulator entries in the qcom-rpm driver.
>
> Acked-by: Bjorn Andersson
> Acked-by: Lee Jones
> Signed-off-by: Neil Armstrong
Quick note. In future
On Fri, 19 Aug 2016, Wadim Egorov wrote:
> On 18.08.2016 16:34, Lee Jones wrote:
> > On Wed, 10 Aug 2016, Wadim Egorov wrote:
> >
> >> The RK818 chip is a Power Management IC (PMIC) for multimedia and handheld
> >> devices. It contains the following components:
REE,
> + .volatile_reg = rk808_is_volatile_reg,
> +};
> +
> static const struct regmap_config rk808_regmap_config = {
> .reg_bits = 8,
> .val_bits = 8,
> @@ -79,11 +92,21 @@ static const struct mfd_cell rk808s[] = {
> {
> .name = "rk808
create mode 100644 drivers/rtc/rtc-ac100.c
create mode 100644 include/linux/mfd/ac100.h
--
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
--
You received this message because you are subscribed to
.parent_names = parents,
> > + .num_parents = ARRAY_SIZE(parents),
> > + .flags = 0,
> > + };
> > +
> > + clk->regmap = chip->regmap;
> > + clk->offset = AC100_CLKOUT_CTRL1 + i;
> > +
arm_irq_enable,
> > +};
> > +
> > +static int ac100_rtc_probe(struct platform_device *pdev)
> > +{
> > + struct ac100_dev *ac100 = dev_get_drvdata(pdev->dev.parent);
> > + struct ac100_rtc_dev *chip;
> > + int ret;
> > +
> > + chip = d
On Fri, 08 Jul 2016, Chen-Yu Tsai wrote:
> The AC100 is a multifunction device with an audio codec subsystem and
> an RTC subsystem. These two subsystems share a common register space
> and host interface.
>
> Signed-off-by: Chen-Yu Tsai
> Acked-by: Lee Jones
> -
On Mon, 08 Aug 2016, Lee Jones wrote:
> On Fri, 08 Jul 2016, Chen-Yu Tsai wrote:
>
> > The AC100 is a multifunction device with an audio codec subsystem and
> > an RTC subsystem. These two subsystems share a common register space
> > and host interface.
> >
&
gt;
> - Fix interrupt line for ac100_codec in provided example.
>
> ---
> Documentation/devicetree/bindings/mfd/ac100.txt | 54
> +
> 1 file changed, 54 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/mfd/ac100.txt
For my o
ebase and resend the series.
--
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
--
You received this message because you are subscribed to "rtc-linux".
Membership options at http://groups.googl
#clock-cells = <0>;
> + clock-output-names = "4M_adda";
> + };
> +
> + ac100_rtc: rtc {
> + compatible = "x-powers,ac100-rtc";
> + interrupt-parent = <&nmi_intc>;
> + interrupts = <0 I
mfd/qcom-rpm.txt | 1 +
> drivers/mfd/qcom_rpm.c | 51
> ++
> include/dt-bindings/mfd/qcom-rpm.h | 22 ++
> 3 files changed, 74 insertions(+)
For my own reference:
Acked-by: Lee Jones
> diff --git a/Documentation/
insertion(+)
For my own reference:
Acked-by: Lee Jones
> diff --git a/drivers/mfd/pm8921-core.c b/drivers/mfd/pm8921-core.c
> index 1b7ec08..0e3a2ea 100644
> --- a/drivers/mfd/pm8921-core.c
> +++ b/drivers/mfd/pm8921-core.c
> @@ -309,6 +309,7 @@ static const
Signed-off-by: Robert Baldyga
> Reviewed-by: Krzysztof Kozlowski
> Acked-by: Chanwoo Choi
> Acked-by: Bryan Wu
> Acked-by: Lee Jones
> Acked-by: Sebastian Reichel
> Acked-by: Dmitry Torokhov
> Acked-by: Alexandre Belloni
> [k.kozlowski: Collect acks, rebase on v4.6-rc6]
>
On Wed, 08 Jun 2016, Krzysztof Kozlowski wrote:
> On 06/08/2016 04:26 PM, Lee Jones wrote:
> > On Wed, 01 Jun 2016, Krzysztof Kozlowski wrote:
> >
> >> From: Robert Baldyga
> >>
> >> This patch modifies max8997 driver and each associated function drive
On Wed, 15 Jun 2016, Chen-Yu Tsai wrote:
> On Wed, Jun 15, 2016 at 8:36 PM, Lee Jones wrote:
> > On Wed, 15 Jun 2016, Chen-Yu Tsai wrote:
> >
> >> The AC100 is a multifunction device with an audio codec subsystem and
> >> an RTC subsystem. These two subsyst
ine RK808_IRQ_PLUG_IN_INT_MSKBIT(0)
> #define RK808_IRQ_PLUG_OUT_INT_MSK BIT(1)
>
> +/* RK818 IRQ Definitions */
> +#define RK818_IRQ_VOUT_LO0
> +#define RK818_IRQ_VB_LO 1
> +#define RK818_IRQ_PWRON 2
> +#define RK818_IRQ_PWRON_LP 3
> +#define RK818_IRQ_HOTDIE
wers,ac100-rtc";
> + interrupt-parent = <&nmi_intc>;
> + interrupts = <0 IRQ_TYPE_LEVEL_LOW>;
Isn't this the same interrupt as above? Is it shared?
> + #clock-cells = <1>;
> + clock-output-names =
ac100.c
> new file mode 100644
> index ..15d24fa1
> --- /dev/null
> +++ b/drivers/mfd/ac100.c
> @@ -0,0 +1,135 @@
> +/*
> + * ac100.c - MFD core driver for X-Powers' AC100 Audio Codec IC
Nit: Drop the filename here.
Apart from that:
Acked-by: Lee Jones
--
On Thu, 09 Jun 2016, Wadim Egorov wrote:
>
>
> On 09.06.2016 13:12, Lee Jones wrote:
> > On Thu, 09 Jun 2016, Wadim Egorov wrote:
> >> On 08.06.2016 16:17, Lee Jones wrote:
> >>> On Thu, 02 Jun 2016, Wadim Egorov wrote:
> >>>
> >>>
On Thu, 09 Jun 2016, Wadim Egorov wrote:
> On 08.06.2016 16:17, Lee Jones wrote:
> > On Thu, 02 Jun 2016, Wadim Egorov wrote:
> >
> >> The RK818 chip is a power management IC for multimedia and handheld
> > "Power Management IC (PMIC)"
> >
>
Signed-off-by: Robert Baldyga
> Reviewed-by: Krzysztof Kozlowski
>
> Acked-by: Chanwoo Choi
> Acked-by: Bryan Wu
> Acked-by: Lee Jones
> Acked-by: Sebastian Reichel
> Acked-by: Dmitry Torokhov
> Acked-by: Alexandre Belloni
> [k.kozlowski: Collect acks, rebase o
On Thu, 02 Jun 2016, Wadim Egorov wrote:
> Add device tree bindings documentation for Rockchip's RK818 PMIC.
>
> Signed-off-by: Wadim Egorov
> Acked-by: Rob Herring
Acked-by: Lee Jones
> ---
> Documentation/devicetree/bindings/mfd/rk808.txt | 37
> +
ne RK818_IRQ_PLUG_IN8
> +#define RK818_IRQ_PLUG_OUT 9
> +#define RK818_IRQ_CHG_OK 10
> +#define RK818_IRQ_CHG_TE 11
> +#define RK818_IRQ_CHG_TS112
> +#define RK818_IRQ_TS213
> +#define RK818_IRQ_CHG_CVTLIM 14
> +#define RK818_IRQ_DISCHG_ILIM7
>
56 matches
Mail list logo