drm/bridge: imx: Remove redundant checks on existence of bridge->encoder

Browse source

The checks on the existence of bridge->encoder in the implementation of
drm_bridge_funcs::attach() is not necessary, as it has already been checked
in the drm_bridge_attach() function call by previous bridge or KMS driver.
The drm_bridge_attach() will quit with a negative error code returned if
it fails for some reasons, hence, it is guaranteed that the .encoder member
of the drm_bridge instance is not NULL when various i.MX specific bridge
attach functions are called.

Remove the redundant checking codes "if (!bridge->encoder) { ... }".

Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Sui Jingfeng <sui.jingfeng@linux.dev>
Signed-off-by: Robert Foss <rfoss@kernel.org>
Link: https://patchwork.freedesktop.org/patch/msgid/20240513153109.46786-12-sui.jingfeng@linux.dev

...

This commit is contained in:

Sui Jingfeng 2024-05-13 23:31:08 +08:00 • committed by Robert Foss

parent 91942a37eb

commit ec74951a75

No known key found for this signature in database

GPG key ID: 3EFD900F76D1D784

4 changed files with 0 additions and 20 deletions

Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL

Download patch file Download diff file Expand all files Collapse all files

5

drivers/gpu/drm/bridge/imx/imx-ldb-helper.c

View file

5

drivers/gpu/drm/bridge/imx/imx8qxp-pixel-combiner.c

View file

5

drivers/gpu/drm/bridge/imx/imx8qxp-pixel-link.c

View file

5

drivers/gpu/drm/bridge/imx/imx8qxp-pxl2dpi.c

View file