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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <HE1PR0401MB1945EAD443377803A9F9EA5CE99C0@HE1PR0401MB1945.eurprd04.prod.outlook.com>
Date:   Mon, 14 May 2018 07:47:45 +0000
From:   Yinbo Zhu <yinbo.zhu@....com>
To:     Leo Li <leoyang.li@....com>, Rob Herring <robh+dt@...nel.org>,
        Mark Rutland <mark.rutland@....com>,
        "Catalin Marinas )" <catalin.marinas@....com>,
        "Will Deacon )" <will.deacon@....com>,
        "Lorenzo Pieralisi )" <lorenzo.pieralisi@....com>
CC:     Xiaobo Xie <xiaobo.xie@....com>, Ran Wang <ran.wang_1@....com>,
        Daniel Lezcano <daniel.lezcano@...aro.org>,
        Thomas Gleixner <tglx@...utronix.de>,
        Shawn Guo <shawnguo@...nel.org>,
        Madalin-cristian Bucur <madalin.bucur@....com>,
        "Z.q. Hou" <zhiqiang.hou@....com>,
        Jerry Huang <jerry.huang@....com>,
        "M.h. Lian" <minghuan.lian@....com>,
        Qiang Zhao <qiang.zhao@....com>,
        Fabio Estevam <fabio.estevam@....com>,
        Jiaheng Fan <jiaheng.fan@....com>, Po Liu <po.liu@....com>,
        Nipun Gupta <nipun.gupta@....com>,
        Horia Geantă <horia.geanta@....com>,
        Priyanka Jain <priyanka.jain@....com>,
        Sumit Garg <sumit.garg@....com>,
        costi <constantin.tudor@...escale.com>,
        Bogdan Purcareata <bogdan.purcareata@....com>,
        "open list:CLOCKSOURCE, CLOCKEVENT DRIVERS" 
        <linux-kernel@...r.kernel.org>,
        "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" 
        <devicetree@...r.kernel.org>,
        "linux-arm-kernel@...ts.infradead.org" 
        <linux-arm-kernel@...ts.infradead.org>,
        "open list:FREESCALE SOC DRIVERS" <linuxppc-dev@...ts.ozlabs.org>,
        Andy Tang <andy.tang@....com>,
        Ying Zhang <ying.zhang22455@....com>
Subject: RE: [PATCH 3/9] soc: fsl: set rcpm bit for FTM



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



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