Hi Enric, Thank you for the patch.
On Saturday, 13 October 2018 14:18:44 EEST Enric Balletbo i Serra wrote: > This patch avoids that building the bridge/analogix source code with > smatch triggers complaints about inconsistent indenting. > > Signed-off-by: Enric Balletbo i Serra <[email protected]> > --- > > drivers/gpu/drm/bridge/analogix/analogix_dp_core.c | 12 ++++++------ > 1 file changed, 6 insertions(+), 6 deletions(-) > > diff --git a/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c > b/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c index > d68986cea132..75e2649d7515 100644 > --- a/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c > +++ b/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c > @@ -1219,12 +1219,12 @@ static int analogix_dp_bridge_attach(struct > drm_bridge *bridge) * plat_data->attch return, that's why we record the > connector > * point after plat attached. > */ > - if (dp->plat_data->attach) { > - ret = dp->plat_data->attach(dp->plat_data, bridge, connector); > - if (ret) { > - DRM_ERROR("Failed at platform attch func\n"); > - return ret; > - } > + if (dp->plat_data->attach) { > + ret = dp->plat_data->attach(dp->plat_data, bridge, connector); > + if (ret) { > + DRM_ERROR("Failed at platform attch func\n"); > + return ret; > + } > } This looks good to me. While at it, could you s/attch/attach/ ? With that fixed, Reviewed-by: Laurent Pinchart <[email protected]> > if (dp->plat_data->panel) { -- Regards, Laurent Pinchart _______________________________________________ dri-devel mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/dri-devel
