[Intel-gfx] [PATCH] i915: correctly handling failed allocation

[Insu Yun]

On Wed, Dec 30, 2015 at 4:16 AM, Jani Nikula <jani.nikula at linux.intel.com>
wrote:

> On Tue, 29 Dec 2015, Insu Yun <wuninsu at gmail.com> wrote:
> > Signed-off-by: Insu Yun <wuninsu at gmail.com>
> > ---
> >  drivers/gpu/drm/i915/intel_dsi_panel_vbt.c | 2 ++
> >  1 file changed, 2 insertions(+)
> >
> > diff --git a/drivers/gpu/drm/i915/intel_dsi_panel_vbt.c
> b/drivers/gpu/drm/i915/intel_dsi_panel_vbt.c
> > index a5e99ac..4e279dd 100644
> > --- a/drivers/gpu/drm/i915/intel_dsi_panel_vbt.c
> > +++ b/drivers/gpu/drm/i915/intel_dsi_panel_vbt.c
> > @@ -666,6 +666,8 @@ struct drm_panel *vbt_panel_init(struct intel_dsi
> *intel_dsi, u16 panel_id)
> >
> >       /* This is cheating a bit with the cleanup. */
> >       vbt_panel = devm_kzalloc(dev->dev, sizeof(*vbt_panel), GFP_KERNEL);
> > +     if (!vbt_pannel)
> > +             return NULL;
>

Oh sorry. There was a type.


>
> We have build bots and CI, but the least you must do is build the code
> you change before submitting patches.
>

Yes. Sorry.


>
> BR,
> Jani.
>
>
> >
> >       vbt_panel->intel_dsi = intel_dsi;
> >       drm_panel_init(&vbt_panel->panel);
>
> --
> Jani Nikula, Intel Open Source Technology Center
>



-- 
Regards
Insu Yun
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/intel-gfx/attachments/20151230/20e02d7e/attachment-0001.html>