[PATCH 3/3] drm/tegra: Use __drm_atomic_helper_reset_connector for subclassing connector state.

Thierry Reding thierry.reding at gmail.com
Mon Dec 7 02:02:25 PST 2015


On Tue, Nov 24, 2015 at 02:35:34PM +0100, Maarten Lankhorst wrote:
> Signed-off-by: Maarten Lankhorst <maarten.lankhorst at linux.intel.com>
> ---
>  drivers/gpu/drm/tegra/dsi.c | 9 +++------
>  1 file changed, 3 insertions(+), 6 deletions(-)
> 
> diff --git a/drivers/gpu/drm/tegra/dsi.c b/drivers/gpu/drm/tegra/dsi.c
> index f0a138ef68ce..33ad50487f2e 100644
> --- a/drivers/gpu/drm/tegra/dsi.c
> +++ b/drivers/gpu/drm/tegra/dsi.c
> @@ -745,14 +745,11 @@ static void tegra_dsi_soft_reset(struct tegra_dsi *dsi)
>  
>  static void tegra_dsi_connector_reset(struct drm_connector *connector)
>  {
> -	struct tegra_dsi_state *state;
> +	struct tegra_dsi_state *state =
> +		kzalloc(sizeof(*state), GFP_KERNEL);

I think this could use a check just for safety. It's unlikely to ever
happen, but just in case, better allow to fail gracefully than crash.

Thierry
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20151207/73d5cb9b/attachment-0001.sig>


More information about the dri-devel mailing list