Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:5 as permitted sender) client-ip=2620:137:e000::3:5;
From:   "S.J. Wang" <shengjiu.wang@nxp.com>
To:     Christophe JAILLET <christophe.jaillet@wanadoo.fr>,
        "abelvesa@kernel.org" <abelvesa@kernel.org>,
        Peng Fan <peng.fan@nxp.com>,
        "mturquette@baylibre.com" <mturquette@baylibre.com>,
        "sboyd@kernel.org" <sboyd@kernel.org>,
        "shawnguo@kernel.org" <shawnguo@kernel.org>,
        "s.hauer@pengutronix.de" <s.hauer@pengutronix.de>,
        "kernel@pengutronix.de" <kernel@pengutronix.de>,
        "festevam@gmail.com" <festevam@gmail.com>,
        dl-linux-imx <linux-imx@nxp.com>
CC:     "linux-clk@vger.kernel.org" <linux-clk@vger.kernel.org>,
        "linux-arm-kernel@lists.infradead.org" 
        <linux-arm-kernel@lists.infradead.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "kernel-janitors@vger.kernel.org" <kernel-janitors@vger.kernel.org>
Subject: RE: [PATCH v2 0/5] clk: imx: imx8: Fix some error handling paths
Thread-Topic: [PATCH v2 0/5] clk: imx: imx8: Fix some error handling paths
Thread-Index: AQHZ56JABBqNp29H4E67KfuIoELhrQ==
Date:   Fri, 15 Sep 2023 07:00:04 +0000
Message-ID: <PA4PR04MB92486B510B8E8F8309F484EDE3F6A@PA4PR04MB9248.eurprd04.prod.outlook.com>
References: <cover.1694722339.git.christophe.jaillet@wanadoo.fr>
In-Reply-To: <cover.1694722339.git.christophe.jaillet@wanadoo.fr>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?D/NdsVlrNZleem6i0bxllKWJqIYazMyO1sZlkG1HTfRbIJtiJcOF6T9unlPR?=
 =?us-ascii?Q?2AeiH/gPvCWzmkHv7Ksn3a5QgsGPERtAc9nuMkDvVD3ilAvN8aYAalVCGau5?=
 =?us-ascii?Q?VwZ0/Q73MJYEvAerPOjZvR2JK1tJBM5wjxTTX5/GV1ZJddwug3+X8i3F4dlw?=
 =?us-ascii?Q?D1b3pGJuMyZyjwLHWHIif4KFNQmY/ks9ftlH0ALt3NRTmVXwGA1ALhp/Xsug?=
 =?us-ascii?Q?rfYE6fTxIIF+KSRkueTXBfuhTpU97l/Rd2+XqnHATDOCs4Ms6TCvqUJ/B/Pz?=
 =?us-ascii?Q?tLt+27uhLST3mlreX2znxXqBNNcfLGa6lAp9DjAJZWNUJUJFXc9q1mHvbYL/?=
 =?us-ascii?Q?RCrhoWDFEqGR+9LFBTyi7TkzSIeI80HZ4OCQjy2YRUuffhVuei7xmSSPKZXQ?=
 =?us-ascii?Q?S6W8EWAjKSodbyMZR0+2Mo4xab+IJ7b/pZey8a2UdEKZs+e8CP6M0EDRj5FJ?=
 =?us-ascii?Q?r0jhPH0EfSkWk5KJdunBJdUCL4ajKQwK5OGNTsQghbGE5WNZ5i8cIuafnSfb?=
 =?us-ascii?Q?mSax9B5x1CTAYp1BQAua5rLBHkGggv1qMp0bRgUUufWEE3kZC1jfe4vlSD5e?=
 =?us-ascii?Q?R2Vf450r+NsmX4dzdBn8qITJXwJ9HbnAgJR1zLsDdgeHgLMTjTcpEtqakgDM?=
 =?us-ascii?Q?GJWsxelm/zUZUt2o40l0smASGFGjD7imNNv/h4VbyYkZ4KmRCGno6f/84lr9?=
 =?us-ascii?Q?Ju4sfUFC7ua1afy3MpcNciYwcylBpGtb7U7UTFR1jHUEKEPHeil3UP7v4ovc?=
 =?us-ascii?Q?oQ0GCRamTr676OJb4ePc9JMQcfyxnGQg15GX+31eyQGZFYSon5ZZZO7u+2RD?=
 =?us-ascii?Q?k+KlVZ8gyNuVCNJ5fvUWRtNlWJixV6JplPskZoIkdBQCzPbUGnQ/iK7ZswBn?=
 =?us-ascii?Q?/wi0HERM9eLL0Vt94rvpy/iERf5a9EvIHEwz3O0WIj1l2wIOGNv71COsZBcT?=
 =?us-ascii?Q?bPtfmDPSuaqdDDfyATadybECCk1uNQLILlmM9ZQjOtxVE4EAkpmCQ3jZbVuz?=
 =?us-ascii?Q?2QjnBlMAfZrpXqgOf4Cr1DiFb9WLXaxfWUTiEwD7bwjvNpxxQE+30UQUK0gu?=
 =?us-ascii?Q?QVRYtxKkcafwXfpn97tUdBLn+tHk3yuH1F9a7eQI7rCx6RlxnjKddgJI2zyo?=
 =?us-ascii?Q?uM7mCBnWuM+S9F8BWeTnE+8DTIO/KIkuECxDdipIcsppR90Axua+94XaSfWi?=
 =?us-ascii?Q?fW/E1CvszeluuKUUojAfurvBWxd6HpDsG9i9XgkkXltFkb2OwqINTHWm5tB2?=
 =?us-ascii?Q?+PPVq/JEOkj0nX8ZE3SOIOboK5ZJpdXH5yldMzQKBdxjM0P//GOnP4RTP5ZM?=
 =?us-ascii?Q?MpS0z5SwcH6MzrvUWG7fAqUwDoBEgek3zK/ZkIdg+EFNAfJcgzxvnHAxQ5kq?=
 =?us-ascii?Q?4cnSWz5RfwzGLt6J0+fCuNgDT8QVBzw+8q6ZDIocZUNJdaRegIJsqAeOMZ3W?=
 =?us-ascii?Q?P2q3KYItj/gl+w0KjgvDWueBW7fWtwY9fMiJ290HcIByX3veROkvElBiYjqo?=
 =?us-ascii?Q?aOow7+sEqvo7DyTEK61H4+/5RgYRVTedaiWhcJ0NdGpMsxgN9Td4UuHRHf1c?=
 =?us-ascii?Q?Jh2v/9zmzj14VZN/ssQnwOA5gyfy9MdejZ6q0za2?=
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: PA4PR04MB9248.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 2669547a-d17b-4d7c-8e5d-08dbb5b96369
X-MS-Exchange-CrossTenant-originalarrivaltime: 15 Sep 2023 07:00:04.4410
 (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: gNkZis3bwKj7xCsBgRvYeJbsJ84Lcai5GnigLXVbVGjIqEPM7AYCup3yJ4fi4mF+slw+pD/7pI5sHNK18AAp8A==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AS5PR04MB9941
X-Spam-Status: No, score=-0.9 required=5.0 tests=DKIM_SIGNED,DKIM_VALID,
	DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,
	SPF_HELO_NONE,SPF_PASS autolearn=unavailable autolearn_force=no
	version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on groat.vger.email
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (groat.vger.email [0.0.0.0]); Fri, 15 Sep 2023 00:00:32 -0700 (PDT)


>
> This serie fix some error handling paths. It is split in different patche=
s to ease
> review because the issues are unrelated and the proposed fixes are maybe
> wrong (I don't have the hardware to test anything)
>
> Some of v1 patches had been incorrectly hand modified. There was a extra =
')'
> in patch 3 and 4 that prevented the merge.
>
> This v2 is a cleanly regenarated serie, compile tested at each step. This=
 time,
> git format-patch did all the work and the patches have NOT been hand
> modified afterwards.
>
>
> Some log messages have also been slightly tweaked.
> R-b tags have been added.


Reviewed-by: Shengjiu Wang <shengjiu.wang@nxp.com>
Tested-by: Shengjiu Wang <shengjiu.wang@nxp.com>

Best regards
Wang Shengjiu
>
> v1:
> https://lore.k/
> ernel.org%2Fall%2Fcover.1693126687.git.christophe.jaillet%40wanadoo.fr%2
> F&data=3D05%7C01%7Cshengjiu.wang%40nxp.com%7C5134bf2d7d924e4ca9ae
> 08dbb561881c%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C6383
> 03202724496811%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiL
> CJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&s
> data=3Df0SvmM0H1Ki%2Bp3Qqsya8u87CbVowC4bXym2i%2FSBCgJc%3D&reser
> ved=3D0
>
> Christophe JAILLET (5):
>   clk: imx: imx8: Fix an error handling path in
>     clk_imx_acm_attach_pm_domains()
>   clk: imx: imx8: Fix an error handling path if
>     devm_clk_hw_register_mux_parent_data_table() fails
>   clk: imx: imx8: Fix an error handling path in imx8_acm_clk_probe()
>   clk: imx: imx8: Add a message in case of
>     devm_clk_hw_register_mux_parent_data_table() error
>   clk: imx: imx8: Simplify clk_imx_acm_detach_pm_domains()
>
>  drivers/clk/imx/clk-imx8-acm.c | 27 +++++++++++++++------------
>  1 file changed, 15 insertions(+), 12 deletions(-)
>
> --
> 2.34.1