These patches fix cases where a subsequent statement is aligned with the right hand side of an assignment rather than the left hand side. This was done using the following semantic match (http://coccinelle.lip6.fr/). It has a lot of false positives, because Coccinelle doesn't record the difference between a space and a tab, but these issues are easy to skip over.
// <smpl> @r@ expression x,e; statement S; position p0,p1,p2; @@ x@p0 = e@p1; S@p2
@script:ocaml@ p0 << r.p0; p1 << r.p1; p2 << r.p2; @@ if ((List.hd p0).col < (List.hd p1).col && (List.hd p1).col = (List.hd p2).col) then Coccilib.print_main "" p1 // </smpl>
---
drivers/firmware/stratix10-svc.c | 12 ++++++------ drivers/gpu/drm/msm/dsi/phy/dsi_phy.c | 2 +- drivers/iommu/msm_iommu.c | 8 ++++---- drivers/net/ethernet/amazon/ena/ena_eth_com.c | 4 ++-- drivers/net/ethernet/intel/i40e/i40e_main.c | 2 +- drivers/scsi/pm8001/pm8001_sas.c | 2 +- drivers/staging/rtl8192e/rtl8192e/rtl_wx.c | 6 +++--- drivers/usb/gadget/udc/snps_udc_core.c | 17 ++++++++--------- 8 files changed, 26 insertions(+), 27 deletions(-)
Delete tab aligning a statement with the right hand side of a preceding assignment rather than the left hand side.
Found with the help of Coccinelle.
Signed-off-by: Julia Lawall Julia.Lawall@lip6.fr
--- drivers/gpu/drm/msm/dsi/phy/dsi_phy.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/msm/dsi/phy/dsi_phy.c b/drivers/gpu/drm/msm/dsi/phy/dsi_phy.c index 1760483b247e..4d8785282c6c 100644 --- a/drivers/gpu/drm/msm/dsi/phy/dsi_phy.c +++ b/drivers/gpu/drm/msm/dsi/phy/dsi_phy.c @@ -355,7 +355,7 @@ int msm_dsi_dphy_timing_calc_v3(struct msm_dsi_dphy_timing *timing, } else { timing->shared_timings.clk_pre = linear_inter(tmax, tmin, pcnt2, 0, false); - timing->shared_timings.clk_pre_inc_by_2 = 0; + timing->shared_timings.clk_pre_inc_by_2 = 0; }
timing->ta_go = 3;
dri-devel@lists.freedesktop.org
-
Julia Lawall