Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Leo Li <leoyang.li@nxp.com>
To:     Vladimir Oltean <olteanv@gmail.com>
CC:     Shawn Guo <shawnguo@kernel.org>,
        "linux-arm-kernel@lists.infradead.org" 
        <linux-arm-kernel@lists.infradead.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        Biwen Li <biwen.li@nxp.com>
Subject: RE: [PATCH v3 3/8] arm64: dts: ls1028a: add flextimer based pwm nodes
Thread-Topic: [PATCH v3 3/8] arm64: dts: ls1028a: add flextimer based pwm
 nodes
Thread-Index: AQHX8M2Qp2fR5PuKkEeWcFzMp8XOI6zuxrMAgAAZn+CAAAU2gIAABtXg
Date:   Wed, 13 Apr 2022 19:10:32 +0000
Message-ID: <DU2PR04MB89497E16D2E9A01BF763D92B8FEC9@DU2PR04MB8949.eurprd04.prod.outlook.com>
References: <20211214093240.23320-1-leoyang.li@nxp.com>
 <20211214093240.23320-4-leoyang.li@nxp.com>
 <20220413163306.hp5qyxxgdlekczet@skbuf>
 <DU2PR04MB894984A336667EF01DF709988FEC9@DU2PR04MB8949.eurprd04.prod.outlook.com>
 <20220413182327.lbemqze46eq7ulfn@skbuf>
In-Reply-To: <20220413182327.lbemqze46eq7ulfn@skbuf>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?vgywZujoGJQFpitU0sTgeG24vfQzroGDv1K8Xj+S8uenfnWV96pdRl/bYFSw?=
 =?us-ascii?Q?uHHUy6GrgDduthhSXcEaKycvRSOTq9QRKpcc4qTNm59KL33Y5x5tgQYCnt2W?=
 =?us-ascii?Q?I3JkLYB801TWHDT00gMUdOr2HFxpMHL8dMaDedTvKqTs1gP0gMKht+j5yB/h?=
 =?us-ascii?Q?81DzY5vi9cvvMMnaUtSJ3SvWTtXYJmEg9EbbgIttF2ef0UtHSyu42L+peVsz?=
 =?us-ascii?Q?aKeFWJRjbGqQO1rl0l3jHirLdnxjr0rnCec8qXi9/R6EafDWYc0+1z6d/IRK?=
 =?us-ascii?Q?CpXpsb612QC41Eu5tIEgzD+9FiyiHAwrtlwNoqWwIfcQ3qTFm0XNMa9i0oo+?=
 =?us-ascii?Q?4NBDU1erokwCGEUus1Go+oGQB/7TwqeFHOdGbvdei9OTQuPm0EKaXFP6b3kg?=
 =?us-ascii?Q?+J2N0W8sm+i0G6sKQAtP222+md/hKDE3yYpPzkmrySmlFDsrQ4y0hAPvfhKP?=
 =?us-ascii?Q?PYQc3GhcgMETCceut/ClHFFA+/nbKX0iZw5ClvlF8F7oDj0Uzf0jrcPANoRJ?=
 =?us-ascii?Q?6SElM+3pMS2Hg0DR8clicPyDKoW6Q8tbrVbKQ6aeHWUXPxv7h+wDeuxtPppc?=
 =?us-ascii?Q?m9F/5/FyY0VjVJfNCzAQIPDvf1KaEXqLpgwEiuo1GVF8hdgeYWgznWZK8CmZ?=
 =?us-ascii?Q?+aRVeWMchcn1zoOgaZpTQT0OzWDrplRYRRJGjge38LsCuTGxudvxAUMdH8Zg?=
 =?us-ascii?Q?XGwIe9uQhsW9aWwlQdIt91zhMrvSp8E4yAbEydWhwo45wINvg9dtpXVcoiLL?=
 =?us-ascii?Q?1C0s5nwH2TciMGwfe/NmU+XKTwYn3xEhsB9/GJzTkg0hHGSDJgHOefh27Spf?=
 =?us-ascii?Q?K+s8HxHgrpzPRLZgjBtPN9gLTMearJF7WVEpV6kK8Zs6ejWV6aB3B+4LmzwG?=
 =?us-ascii?Q?FRgsjZ0S3Vnf1YFVPUlVyg1HJ5SMYdjRo9BLrw6MeCDzcnACJgBlanYwhpAt?=
 =?us-ascii?Q?ChFtRbkRTdH8B1T1Yvw73umAwh8k0ZpDliXdvUFE7vfDdc5jSM7Kp2vYKtxw?=
 =?us-ascii?Q?bQDZXvJjuNITvM7o8MI5d589YjNGeehqXxNzmYb+f0DLNrG42+roZfZTQ/Ea?=
 =?us-ascii?Q?ESe0bm++m8B5vixp38GSwMvdaGPh2PMocfLUHh6d3Og7BF1eh5vGgQBbho3R?=
 =?us-ascii?Q?9FzfL5RiXGXfYa9K7VXXS4cY5yg/fJ6sMKZEruCI50kgV5JocUWFXWM97pT6?=
 =?us-ascii?Q?3QvnCTvkfmqb6fQpuiHgMNnGuSlsA4hmAf+lCsNmhdfVoA24DMz2e7bYO+Qr?=
 =?us-ascii?Q?XUSAvMwFWksraWRPo2o58QV5Iquinrzoh+JlHUilYZobwE8FLyoAhznNh1uN?=
 =?us-ascii?Q?Nqwe3WHo0skU7ypvV86UUwNSfOQ46r0DBaMVYMlNjJjWYC8Jky/zaPXBRExG?=
 =?us-ascii?Q?FcnxQ2UZ2DQer4IpCQkLei7WnAx9clV2FS20lcLTRebYbGQtfFjNqUE9eRqV?=
 =?us-ascii?Q?UEsGgRQkyu40tV1qnba3sf+OQR02MlW3C+tw2qukIj4UpFD6JJWy1CNbdNn2?=
 =?us-ascii?Q?hXH8bxjzb7Ozy8ixC47nKp2dLedKB4tJZLFOUxX2eF+UFs3b3I/jqiRz+3QQ?=
 =?us-ascii?Q?RB8LvcjDElrkyCYF1fv6KrWY4Z5bW+wqkPqJ8JcWlOnCPY2ECB8uunGre7ur?=
 =?us-ascii?Q?DeNHjZrZKFUcGDvHsQunGyZHEXYqlrEahOs+xWoy0p4nOtfvnk9JlQfQGlmU?=
 =?us-ascii?Q?Ei2euuOUuUJbhgDLhZpgithx9fLVHZVuxWvfBw30p6tqPKkNwOILeuiwgzfg?=
 =?us-ascii?Q?W66+p00CXA=3D=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: DU2PR04MB8949.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 67ccc736-8997-41b1-2c69-08da1d81487b
X-MS-Exchange-CrossTenant-originalarrivaltime: 13 Apr 2022 19:10:32.9427
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: v25rVTY1tmNOmhyyYfXOdc1oMZb4bi3XzuzOmaigRsUutY6BCZZ9py41HLghYXaL8QdoMOhkjhM6ucXFOaTuNg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AS4PR04MB9550
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS,T_SCC_BODY_TEXT_LINE
        autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


> -----Original Message-----
> From: Vladimir Oltean <olteanv@gmail.com>
> Sent: Wednesday, April 13, 2022 1:23 PM
> To: Leo Li <leoyang.li@nxp.com>
> Cc: Shawn Guo <shawnguo@kernel.org>; linux-arm-
> kernel@lists.infradead.org; linux-kernel@vger.kernel.org; Biwen Li
> <biwen.li@nxp.com>
> Subject: Re: [PATCH v3 3/8] arm64: dts: ls1028a: add flextimer based pwm
> nodes
>=20
> On Wed, Apr 13, 2022 at 06:07:20PM +0000, Leo Li wrote:
> > > arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi:1210.21-1219.5:
> > > Warning
> > > (unique_unit_address): /soc/pwm@2800000: duplicate unit-address
> > > (also used in node /soc/timer@2800000)
> > > arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi:1221.21-1230.5:
> > > Warning
> > > (unique_unit_address): /soc/pwm@2810000: duplicate unit-address
> > > (also used in node /soc/timer@2810000)
> >
> > Well, this is similar situation as pcie.  The flextimer controller can
> > be used as timer, PWM or alarm.  We have separate drivers and bindings
> > for these modes which resulted in different nodes for the same
> > controller.
>=20
> I think the mfd framework can address the situation where multiple driver=
s,
> with multiple functionalities, want access to the same memory region?

I know mfd is used for device providing multiple functions at the same time=
.  I'm not sure if it can help dealing with the one function at a time scen=
ario.

Regards,
Leo