From cddd4591404fb4c53dc0b3c0b15b942cdbed4356 Mon Sep 17 00:00:00 2001 From: Yangbo Lu Date: Fri, 10 Apr 2020 10:47:05 +0800 Subject: layerscape: add patches-5.4 Add patches for linux-5.4. The patches are from NXP LSDK-20.04 release which was tagged LSDK-20.04-V5.4. https://source.codeaurora.org/external/qoriq/qoriq-components/linux/ For boards LS1021A-IOT, and Traverse-LS1043 which are not involved in LSDK, port the dts patches from 4.14. The patches are sorted into the following categories: 301-arch-xxxx 302-dts-xxxx 303-core-xxxx 701-net-xxxx 801-audio-xxxx 802-can-xxxx 803-clock-xxxx 804-crypto-xxxx 805-display-xxxx 806-dma-xxxx 807-gpio-xxxx 808-i2c-xxxx 809-jailhouse-xxxx 810-keys-xxxx 811-kvm-xxxx 812-pcie-xxxx 813-pm-xxxx 814-qe-xxxx 815-sata-xxxx 816-sdhc-xxxx 817-spi-xxxx 818-thermal-xxxx 819-uart-xxxx 820-usb-xxxx 821-vfio-xxxx Signed-off-by: Yangbo Lu --- ...2c-imx-Coverity-fix-divide-by-zero-warnin.patch | 99 ++++++++++++++++++++++ 1 file changed, 99 insertions(+) create mode 100644 target/linux/layerscape/patches-5.4/808-i2c-0006-MLK-20368-i2c-imx-Coverity-fix-divide-by-zero-warnin.patch (limited to 'target/linux/layerscape/patches-5.4/808-i2c-0006-MLK-20368-i2c-imx-Coverity-fix-divide-by-zero-warnin.patch') diff --git a/target/linux/layerscape/patches-5.4/808-i2c-0006-MLK-20368-i2c-imx-Coverity-fix-divide-by-zero-warnin.patch b/target/linux/layerscape/patches-5.4/808-i2c-0006-MLK-20368-i2c-imx-Coverity-fix-divide-by-zero-warnin.patch new file mode 100644 index 0000000000..98c5c21486 --- /dev/null +++ b/target/linux/layerscape/patches-5.4/808-i2c-0006-MLK-20368-i2c-imx-Coverity-fix-divide-by-zero-warnin.patch @@ -0,0 +1,99 @@ +From 2c2c7e16ad0401f85dd8ff9bc0789fa10e715c0e Mon Sep 17 00:00:00 2001 +From: Clark Wang +Date: Tue, 15 Jan 2019 10:04:30 +0800 +Subject: [PATCH] MLK-20368 i2c-imx: Coverity: fix divide by zero warning + +"i2c_clk_rate / 2" might be zero when the i2c_clk_rate gets the clock is +0 or 1, so add a judgment to avoid the denominator is equal to 0. + +Signed-off-by: Clark Wang +[Arul: Add support to check return value everywhere in the driver] +Signed-off-by: Arulpandiyan Vadivel + +Signed-off-by: Shrikant Bobade +(cherry picked from commit d382de595bffc0975ab7c0582e08dd4f7afc0c1a) +(cherry picked from commit 456caa9ba270ca8f4f962918a0625d1a8348f316) +--- + drivers/i2c/busses/i2c-imx.c | 31 +++++++++++++++++++++++++------ + 1 file changed, 25 insertions(+), 6 deletions(-) + +--- a/drivers/i2c/busses/i2c-imx.c ++++ b/drivers/i2c/busses/i2c-imx.c +@@ -492,16 +492,24 @@ static int i2c_imx_acked(struct imx_i2c_ + return 0; + } + +-static void i2c_imx_set_clk(struct imx_i2c_struct *i2c_imx, ++static int i2c_imx_set_clk(struct imx_i2c_struct *i2c_imx, + unsigned int i2c_clk_rate) + { + struct imx_i2c_clk_pair *i2c_clk_div = i2c_imx->hwdata->clk_div; + unsigned int div; + int i; + +- /* Divider value calculation */ + if (i2c_imx->cur_clk == i2c_clk_rate) +- return; ++ return 0; ++ ++ /* ++ * Keep the denominator of the following program ++ * always NOT equal to 0. ++ */ ++ ++ /* Divider value calculation */ ++ if (!(i2c_clk_rate / 2)) ++ return -EINVAL; + + i2c_imx->cur_clk = i2c_clk_rate; + +@@ -532,20 +540,23 @@ static void i2c_imx_set_clk(struct imx_i + dev_dbg(&i2c_imx->adapter.dev, "IFDR[IC]=0x%x, REAL DIV=%d\n", + i2c_clk_div[i].val, i2c_clk_div[i].div); + #endif ++ ++ return 0; + } + + static int i2c_imx_clk_notifier_call(struct notifier_block *nb, + unsigned long action, void *data) + { ++ int ret = 0; + struct clk_notifier_data *ndata = data; + struct imx_i2c_struct *i2c_imx = container_of(nb, + struct imx_i2c_struct, + clk_change_nb); + + if (action & POST_RATE_CHANGE) +- i2c_imx_set_clk(i2c_imx, ndata->new_rate); ++ ret = i2c_imx_set_clk(i2c_imx, ndata->new_rate); + +- return NOTIFY_OK; ++ return notifier_from_errno(ret); + } + + static int i2c_imx_start(struct imx_i2c_struct *i2c_imx) +@@ -555,6 +566,10 @@ static int i2c_imx_start(struct imx_i2c_ + + dev_dbg(&i2c_imx->adapter.dev, "<%s>\n", __func__); + ++ result = i2c_imx_set_clk(i2c_imx, clk_get_rate(i2c_imx->clk)); ++ if (result) ++ return result; ++ + imx_i2c_write_reg(i2c_imx->ifdr, i2c_imx, IMX_I2C_IFDR); + /* Enable I2C controller */ + imx_i2c_write_reg(i2c_imx->hwdata->i2sr_clr_opcode, i2c_imx, IMX_I2C_I2SR); +@@ -1175,7 +1190,11 @@ static int i2c_imx_probe(struct platform + i2c_imx->bitrate = pdata->bitrate; + i2c_imx->clk_change_nb.notifier_call = i2c_imx_clk_notifier_call; + clk_notifier_register(i2c_imx->clk, &i2c_imx->clk_change_nb); +- i2c_imx_set_clk(i2c_imx, clk_get_rate(i2c_imx->clk)); ++ ret = i2c_imx_set_clk(i2c_imx, clk_get_rate(i2c_imx->clk)); ++ if (ret < 0) { ++ dev_err(&pdev->dev, "can't get I2C clock\n"); ++ goto clk_notifier_unregister; ++ } + + /* + * This limit caused by an i.MX7D hardware issue(e7805 in Errata). -- cgit v1.2.3