Re: [PATCH v6 1/7] drm/stm: ltdc: Fix leak of px clk enable in some error paths

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 07/29/2017 04:32 PM, Archit Taneja wrote:
> Hi Philippe,
> 
> On 07/17/2017 01:10 PM, Philippe CORNU wrote:
>> The pixel clock gets enabled early during init, since it's required
>> in order to read registers. This pixel clock must be disabled if
>> errors during this init phase.
>>
> 
> This patch was pulled in to drm-misc-next, but it lacks your Sign-off.
> It looks like the Ack and the Sign-off got accidentally mixed up
> 

Hi Archit,
This patch is from Eric that is why I put his "signed-off-by" and I 
"acked" it to confirm that the patch is fine to me :-)

Nevertheless, as I am the patch "author" in the git tree, I confirm with 
my my below signed-off-by:

Signed-off-by: Philippe Cornu <philippe.cornu@xxxxxx>

Many thanks
Philippe :-)

> Can you please reply to this mail with your "Signed-off-by" so that
> we have proof of it on dri-devel?
> 
> Thanks,
> Archit
> 
>> Signed-off-by: Eric Anholt <eric@xxxxxxxxxx>
>> Acked-by: Philippe Cornu <philippe.cornu@xxxxxx>
>> ---
>>   drivers/gpu/drm/stm/ltdc.c | 10 ++++++----
>>   1 file changed, 6 insertions(+), 4 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/stm/ltdc.c b/drivers/gpu/drm/stm/ltdc.c
>> index 5331760..7f64d5a 100644
>> --- a/drivers/gpu/drm/stm/ltdc.c
>> +++ b/drivers/gpu/drm/stm/ltdc.c
>> @@ -1045,13 +1045,15 @@ int ltdc_load(struct drm_device *ddev)
>>       if (of_address_to_resource(np, 0, &res)) {
>>           DRM_ERROR("Unable to get resource\n");
>> -        return -ENODEV;
>> +        ret = -ENODEV;
>> +        goto err;
>>       }
>>       ldev->regs = devm_ioremap_resource(dev, &res);
>>       if (IS_ERR(ldev->regs)) {
>>           DRM_ERROR("Unable to get ltdc registers\n");
>> -        return PTR_ERR(ldev->regs);
>> +        ret = PTR_ERR(ldev->regs);
>> +        goto err;
>>       }
>>       for (i = 0; i < MAX_IRQ; i++) {
>> @@ -1064,7 +1066,7 @@ int ltdc_load(struct drm_device *ddev)
>>                           dev_name(dev), ddev);
>>           if (ret) {
>>               DRM_ERROR("Failed to register LTDC interrupt\n");
>> -            return ret;
>> +            goto err;
>>           }
>>       }
>> @@ -1079,7 +1081,7 @@ int ltdc_load(struct drm_device *ddev)
>>       if (ret) {
>>           DRM_ERROR("hardware identifier (0x%08x) not supported!\n",
>>                 ldev->caps.hw_version);
>> -        return ret;
>> +        goto err;
>>       }
>>       DRM_INFO("ltdc hw version 0x%08x - ready\n", 
>> ldev->caps.hw_version);
>>
> 
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux