RE: [PATCH 3/9] soc: fsl: set rcpm bit for FTM

From: Yinbo Zhu
Date: Mon May 14 2018 - 03:47:59 EST




-----Original Message-----
From: Leo Li
Sent: 2018å5æ12æ 1:00
To: Yinbo Zhu <yinbo.zhu@xxxxxxx>; Yinbo Zhu <yinbo.zhu@xxxxxxx>; Rob Herring <robh+dt@xxxxxxxxxx>; Mark Rutland <mark.rutland@xxxxxxx>; Catalin Marinas ) <catalin.marinas@xxxxxxx>; Will Deacon ) <will.deacon@xxxxxxx>; Lorenzo Pieralisi ) <lorenzo.pieralisi@xxxxxxx>
Cc: Xiaobo Xie <xiaobo.xie@xxxxxxx>; Ran Wang <ran.wang_1@xxxxxxx>; Daniel Lezcano <daniel.lezcano@xxxxxxxxxx>; Thomas Gleixner <tglx@xxxxxxxxxxxxx>; Shawn Guo <shawnguo@xxxxxxxxxx>; Madalin-cristian Bucur <madalin.bucur@xxxxxxx>; Z.q. Hou <zhiqiang.hou@xxxxxxx>; Jerry Huang <jerry.huang@xxxxxxx>; M.h. Lian <minghuan.lian@xxxxxxx>; Qiang Zhao <qiang.zhao@xxxxxxx>; Fabio Estevam <fabio.estevam@xxxxxxx>; Jiaheng Fan <jiaheng.fan@xxxxxxx>; Po Liu <po.liu@xxxxxxx>; Nipun Gupta <nipun.gupta@xxxxxxx>; Horia GeantÄ <horia.geanta@xxxxxxx>; Priyanka Jain <priyanka.jain@xxxxxxx>; Sumit Garg <sumit.garg@xxxxxxx>; costi <constantin.tudor@xxxxxxxxxxxxx>; Bogdan Purcareata <bogdan.purcareata@xxxxxxx>; open list:CLOCKSOURCE, CLOCKEVENT DRIVERS <linux-kernel@xxxxxxxxxxxxxxx>; open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS <devicetree@xxxxxxxxxxxxxxx>; linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; open list:FREESCALE SOC DRIVERS <linuxppc-dev@xxxxxxxxxxxxxxxx>; Andy Tang <andy.tang@xxxxxxx>; Ying Zhang <ying.zhang22455@xxxxxxx>
Subject: RE: [PATCH 3/9] soc: fsl: set rcpm bit for FTM



> -----Original Message-----
> From: Yinbo Zhu [mailto:yinbo.zhu@xxxxxxx]
> Sent: Thursday, May 10, 2018 10:35 PM
> To: Yinbo Zhu <yinbo.zhu@xxxxxxx>; Rob Herring <robh+dt@xxxxxxxxxx>;
> Mark Rutland <mark.rutland@xxxxxxx>; Catalin Marinas )
> <catalin.marinas@xxxxxxx>; Will Deacon ) <will.deacon@xxxxxxx>;
> Lorenzo Pieralisi ) <lorenzo.pieralisi@xxxxxxx>; Leo Li
> <leoyang.li@xxxxxxx>
> Cc: Xiaobo Xie <xiaobo.xie@xxxxxxx>; Ran Wang <ran.wang_1@xxxxxxx>;
> Daniel Lezcano <daniel.lezcano@xxxxxxxxxx>; Thomas Gleixner
> <tglx@xxxxxxxxxxxxx>; Shawn Guo <shawnguo@xxxxxxxxxx>;
> Madalin-cristian Bucur <madalin.bucur@xxxxxxx>; Z.q. Hou
> <zhiqiang.hou@xxxxxxx>; Jerry Huang <jerry.huang@xxxxxxx>; M.h. Lian
> <minghuan.lian@xxxxxxx>; Qiang Zhao <qiang.zhao@xxxxxxx>; Fabio
> Estevam <fabio.estevam@xxxxxxx>; Jiaheng Fan <jiaheng.fan@xxxxxxx>; Po
> Liu <po.liu@xxxxxxx>; Nipun Gupta <nipun.gupta@xxxxxxx>; Horia GeantÄ
> <horia.geanta@xxxxxxx>; Priyanka Jain <priyanka.jain@xxxxxxx>; Sumit
> Garg <sumit.garg@xxxxxxx>; costi <constantin.tudor@xxxxxxxxxxxxx>;
> Bogdan Purcareata <bogdan.purcareata@xxxxxxx>; Meng Yi
> <meng.yi@xxxxxxx>; Wang Dongsheng <dongsheng.wang@xxxxxxx>; open
> list:CLOCKSOURCE, CLOCKEVENT DRIVERS <linux-kernel@xxxxxxxxxxxxxxx>;
> open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS
> <devicetree@xxxxxxxxxxxxxxx>; linux-arm-kernel@xxxxxxxxxxxxxxxxxxx;
> open list:FREESCALE SOC DRIVERS <linuxppc-dev@xxxxxxxxxxxxxxxx>; Andy
> Tang <andy.tang@xxxxxxx>; Ying Zhang <ying.zhang22455@xxxxxxx>
> Subject: [PATCH 3/9] soc: fsl: set rcpm bit for FTM
>
> From: Zhang Ying-22455 <ying.zhang22455@xxxxxxx>
>
> 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.zhang22455@xxxxxxx>
> Signed-off-by: Yinbo Zhu <yinbo.zhu@xxxxxxx>
> ---
> .../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.