Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
From:   Amit Sunil Dhamne <amitsuni@xilinx.com>
To:     Amit Sunil Dhamne <amitsuni@xilinx.com>,
        "mturquette@baylibre.com" <mturquette@baylibre.com>,
        "m.tretter@pengutronix.de" <m.tretter@pengutronix.de>,
        "sboyd@kernel.org" <sboyd@kernel.org>,
        Michal Simek <michals@xilinx.com>,
        "mark.rutland@arm.com" <mark.rutland@arm.com>,
        "linux-clk@vger.kernel.org" <linux-clk@vger.kernel.org>
CC:     Rajan Vaja <RAJANV@xilinx.com>, Jolly Shah <JOLLYS@xilinx.com>,
        Tejas Patel <TEJASP@xilinx.com>,
        "linux-arm-kernel@lists.infradead.org" 
        <linux-arm-kernel@lists.infradead.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: RE: [PATCH v3 0/3] clk: zynqmp: Add firmware specific clock flags
Thread-Topic: [PATCH v3 0/3] clk: zynqmp: Add firmware specific clock flags
Thread-Index: AQHWairFarR+Mi/aekCugQ3RZGdA0qkx7/Vw
Date:   Mon, 10 Aug 2020 22:06:13 +0000
Message-ID: <BYAPR02MB562261B10DBF08A58FF066B2A7440@BYAPR02MB5622.namprd02.prod.outlook.com>
References: <1596523457-40465-1-git-send-email-amit.sunil.dhamne@xilinx.com>
In-Reply-To: <1596523457-40465-1-git-send-email-amit.sunil.dhamne@xilinx.com>
Accept-Language: en-US
Content-Language: en-US
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: cc0cb116-18c0-4315-5052-08d83d799929
X-MS-Exchange-CrossTenant-originalarrivaltime: 10 Aug 2020 22:06:14.0932
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 657af505-d5df-48d0-8300-c31994686c5c
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: 9ZSkRwJ5Mj9DPyjwiveIAhIRFxpyUPhi0ulMvn/hHfbn/AafpFZcn3YpeJkb63NpPx1x/GMQwJm3g00QlwhJcQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BYAPR02MB4503
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi Stephen, Reviewers,
I wanted to gently follow up on the review of this patch.

Thanks,
Amit

> -----Original Message-----
> From: Amit Sunil Dhamne <amit.sunil.dhamne@xilinx.com>
> Sent: Monday, August 3, 2020 11:44 PM
> To: mturquette@baylibre.com; m.tretter@pengutronix.de;
> sboyd@kernel.org; Michal Simek <michals@xilinx.com>;
> mark.rutland@arm.com; linux-clk@vger.kernel.org
> Cc: Rajan Vaja <RAJANV@xilinx.com>; Jolly Shah <JOLLYS@xilinx.com>; Tejas
> Patel <TEJASP@xilinx.com>; linux-arm-kernel@lists.infradead.org; linux-
> kernel@vger.kernel.org; Amit Sunil Dhamne <amitsuni@xilinx.com>
> Subject: [PATCH v3 0/3] clk: zynqmp: Add firmware specific clock flags
>=20
> Currently firmware is maintaining CCF specific flags and provides to CCF =
as it
> is. But CCF flag numbers may change and that shouldn't mean that the
> firmware needs to change. The firmware should have its own 'flag number
> space' that is distinct from the common clk framework's 'flag number spac=
e'.
> So use firmware specific clock flags in ZynqMP clock driver instead of CC=
F
> flags.
>=20
> Changes in v3:
>  - Modify helper function signature to map zynqmp (common)flags with CCF
>  - Add helper function to map zynqmp (mux & divider)flags with CCF flags
>=20
> Changes in v2:
>  - Add helper function to map zynqmp (common)flags with CCF flags.
>  - Mapped zynqmp clock flags with CCF flags from
>    zynqmp_clk_register_*() functions instead of
>    __zynqmp_clock_get_topology() which is changing the flags to struct
>    clk_init_data instead of the struct clock_topology.
>=20
> Rajan Vaja (3):
>   clk: zynqmp: Use firmware specific common clock flags
>   clk: zynqmp: Use firmware specific divider clock flags
>   clk: zynqmp: Use firmware specific mux clock flags
>=20
>  drivers/clk/zynqmp/clk-gate-zynqmp.c |  4 +++-  drivers/clk/zynqmp/clk-
> mux-zynqmp.c  | 26 +++++++++++++++++++++--
>  drivers/clk/zynqmp/clk-zynqmp.h      | 41
> ++++++++++++++++++++++++++++++++++++
>  drivers/clk/zynqmp/clkc.c            | 32 +++++++++++++++++++++++++++-
>  drivers/clk/zynqmp/divider.c         | 29 ++++++++++++++++++++++---
>  drivers/clk/zynqmp/pll.c             |  4 +++-
>  6 files changed, 128 insertions(+), 8 deletions(-)
>=20
> --
> 2.7.4