-----Original Message-----
From: Leo Li 
Sent: 2018年5月12日 1:00
To: Yinbo Zhu <yinbo....@nxp.com>; Yinbo Zhu <yinbo....@nxp.com>; Rob Herring 
<robh...@kernel.org>; Mark Rutland <mark.rutl...@arm.com>; Catalin Marinas ) 
<catalin.mari...@arm.com>; Will Deacon ) <will.dea...@arm.com>; Lorenzo 
Pieralisi ) <lorenzo.pieral...@arm.com>
Cc: Xiaobo Xie <xiaobo....@nxp.com>; Ran Wang <ran.wan...@nxp.com>; Daniel 
Lezcano <daniel.lezc...@linaro.org>; Thomas Gleixner <t...@linutronix.de>; 
Shawn Guo <shawn...@kernel.org>; Madalin-cristian Bucur 
<madalin.bu...@nxp.com>; Z.q. Hou <zhiqiang....@nxp.com>; Jerry Huang 
<jerry.hu...@nxp.com>; M.h. Lian <minghuan.l...@nxp.com>; Qiang Zhao 
<qiang.z...@nxp.com>; Fabio Estevam <fabio.este...@nxp.com>; Jiaheng Fan 
<jiaheng....@nxp.com>; Po Liu <po....@nxp.com>; Nipun Gupta 
<nipun.gu...@nxp.com>; Horia Geantă <horia.gea...@nxp.com>; Priyanka Jain 
<priyanka.j...@nxp.com>; Sumit Garg <sumit.g...@nxp.com>; costi 
<constantin.tu...@freescale.com>; Bogdan Purcareata 
<bogdan.purcare...@nxp.com>; open list:CLOCKSOURCE, CLOCKEVENT DRIVERS 
<linux-ker...@vger.kernel.org>; open list:OPEN FIRMWARE AND FLATTENED DEVICE 
TREE BINDINGS <devicet...@vger.kernel.org>; 
linux-arm-ker...@lists.infradead.org; open list:FREESCALE SOC DRIVERS 
<linuxppc-dev@lists.ozlabs.org>; Andy Tang <andy.t...@nxp.com>; Ying Zhang 
<ying.zhang22...@nxp.com>
Subject: RE: [PATCH 3/9] soc: fsl: set rcpm bit for FTM



> -----Original Message-----
> From: Yinbo Zhu [mailto:yinbo....@nxp.com]
> Sent: Thursday, May 10, 2018 10:35 PM
> To: Yinbo Zhu <yinbo....@nxp.com>; Rob Herring <robh...@kernel.org>; 
> Mark Rutland <mark.rutl...@arm.com>; Catalin Marinas ) 
> <catalin.mari...@arm.com>; Will Deacon ) <will.dea...@arm.com>; 
> Lorenzo Pieralisi ) <lorenzo.pieral...@arm.com>; Leo Li 
> <leoyang...@nxp.com>
> Cc: Xiaobo Xie <xiaobo....@nxp.com>; Ran Wang <ran.wan...@nxp.com>; 
> Daniel Lezcano <daniel.lezc...@linaro.org>; Thomas Gleixner 
> <t...@linutronix.de>; Shawn Guo <shawn...@kernel.org>; 
> Madalin-cristian Bucur <madalin.bu...@nxp.com>; Z.q. Hou 
> <zhiqiang....@nxp.com>; Jerry Huang <jerry.hu...@nxp.com>; M.h. Lian 
> <minghuan.l...@nxp.com>; Qiang Zhao <qiang.z...@nxp.com>; Fabio 
> Estevam <fabio.este...@nxp.com>; Jiaheng Fan <jiaheng....@nxp.com>; Po 
> Liu <po....@nxp.com>; Nipun Gupta <nipun.gu...@nxp.com>; Horia Geantă 
> <horia.gea...@nxp.com>; Priyanka Jain <priyanka.j...@nxp.com>; Sumit 
> Garg <sumit.g...@nxp.com>; costi <constantin.tu...@freescale.com>; 
> Bogdan Purcareata <bogdan.purcare...@nxp.com>; Meng Yi 
> <meng...@nxp.com>; Wang Dongsheng <dongsheng.w...@nxp.com>; open 
> list:CLOCKSOURCE, CLOCKEVENT DRIVERS <linux-ker...@vger.kernel.org>; 
> open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS 
> <devicet...@vger.kernel.org>; linux-arm-ker...@lists.infradead.org; 
> open list:FREESCALE SOC DRIVERS <linuxppc-dev@lists.ozlabs.org>; Andy 
> Tang <andy.t...@nxp.com>; Ying Zhang <ying.zhang22...@nxp.com>
> Subject: [PATCH 3/9] soc: fsl: set rcpm bit for FTM
> 
> From: Zhang Ying-22455 <ying.zhang22...@nxp.com>
> 
> Set RCPM for FTM when using FTM as wakeup source. Because the RCPM 
> module of each platform has different big-end and little-end mode, 
> there need to set RCPM depending on the platform.
> 
> Signed-off-by: Zhang Ying-22455 <ying.zhang22...@nxp.com>
> Signed-off-by: Yinbo Zhu <yinbo....@nxp.com>
> ---
>  .../devicetree/bindings/timer/fsl,ftm-timer.txt    |    7 ++
>  drivers/soc/fsl/layerscape/ftm_alarm.c             |   92 ++++++++++++++++++-
>  2 files changed, 94 insertions(+), 5 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/timer/fsl,ftm-timer.txt
> b/Documentation/devicetree/bindings/timer/fsl,ftm-timer.txt
> index aa8c402..15ead58 100644
> --- a/Documentation/devicetree/bindings/timer/fsl,ftm-timer.txt
> +++ b/Documentation/devicetree/bindings/timer/fsl,ftm-timer.txt
> @@ -3,6 +3,13 @@ Freescale FlexTimer Module (FTM) Timer  Required
> properties:
> 
>  - compatible : should be "fsl,ftm-timer"

>Hi Yingbo,

>This is a change that breaks backward compatibility and not acceptable.
Hi leo,

This patch if I keep the change as inner patch and push it to dash-linnux but I 
will not push it to upstream, It's okay?
As far as I know, there was a other patch and file for replace the file and 
that the patch is already on the upstream 
https://patchwork.kernel.org/patch/9391293/

> + Possible compatibles for ARM:
> +     "fsl,ls1012a-ftm"
> +     "fsl,ls1021a-ftm"
> +     "fsl,ls1043a-ftm"
> +     "fsl,ls1046a-ftm"
> +     "fsl,ls1088a-ftm"
> +     "fsl,ls208xa-ftm"
>  - reg : Specifies base physical address and size of the register sets for the
>    clock event device and clock source device.
>  - interrupts : Should be the clock event device interrupt.
> diff --git a/drivers/soc/fsl/layerscape/ftm_alarm.c
> b/drivers/soc/fsl/layerscape/ftm_alarm.c
> index 6f9882f..811dcfa 100644
> --- a/drivers/soc/fsl/layerscape/ftm_alarm.c
> +++ b/drivers/soc/fsl/layerscape/ftm_alarm.c

>There is no such file in the mainline kernel.  So it looks like the patch set 
>is

> based on some internal git repo instead of the upstream Linux kernel.  This 
> kind of patches

> shouldn't be sent to the upstream mailing list for review.

>Regards,

>Leo
This patch will not to upstream.

Regards,

Yinbo.

Reply via email to