| Message ID | cover.1694722339.git.christophe.jaillet@wanadoo.fr (mailing list archive) |
|---|---|
| Headers | show |
| Series | clk: imx: imx8: Fix some error handling paths | expand |
> > This serie fix some error handling paths. It is split in different patches 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=05%7C01%7Cshengjiu.wang%40nxp.com%7C5134bf2d7d924e4ca9ae > 08dbb561881c%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C6383 > 03202724496811%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiL > CJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&s > data=f0SvmM0H1Ki%2Bp3Qqsya8u87CbVowC4bXym2i%2FSBCgJc%3D&reser > ved=0 > > 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
On Thu, 14 Sep 2023 22:31:01 +0200, Christophe JAILLET wrote: > This serie fix some error handling paths. It is split in different patches 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. > > [...] Applied, thanks! [1/5] clk: imx: imx8: Fix an error handling path in clk_imx_acm_attach_pm_domains() commit: 156624e2cf815ce98fad5a24f04370f4459ae6f4 [2/5] clk: imx: imx8: Fix an error handling path if devm_clk_hw_register_mux_parent_data_table() fails commit: 9a0108acdb1b6189dcc8f9318edfc6b7e0281df4 [3/5] clk: imx: imx8: Fix an error handling path in imx8_acm_clk_probe() commit: e9a164e367f039629fd5466a79b9f495646e1261 [4/5] clk: imx: imx8: Add a message in case of devm_clk_hw_register_mux_parent_data_table() error commit: ef23d44b7fcbd19773dcedcb50ca2f625107604f [5/5] clk: imx: imx8: Simplify clk_imx_acm_detach_pm_domains() commit: 3af29a89de2c6e49737a1201b7d6fd5ad3f635a4 Best regards,
This serie fix some error handling paths. It is split in different patches 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. v1: https://lore.kernel.org/all/cover.1693126687.git.christophe.jaillet@wanadoo.fr/ 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(-)