[lkml]   [2022]   [Dec]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH 3/7] staging: vc04_services: Drop VCHIQ_ERROR usage
Hi Stefan,

On 12/15/22 5:00 PM, Stefan Wahren wrote:
> Hi Umang,
> thanks for taking care of this.
> Am 15.12.22 um 07:58 schrieb Umang Jain:
>> Drop the usage of VCHIQ_ERROR vchiq_status enum type. Replace it with
>> -EINVAL to report the error.
> I would prefer to use adequate error codes like ENOMEM and so on. I
> agree simply replacing all VCHIQ_ERROR with -EINVAL is easier, but
> wasn't the real intention of the TODO item.

I agree with you on that front - I am still trying to understand the
bits of pieces of the vchiq driver as you can see it's not obvious to me.

I will try to address this is in v2 - after the series a bit more feedback.
> The rest of the series looks good to me.

Okay :-)
Thanks for prompt review.
> Best regards

 \ /
  Last update: 2022-12-15 14:27    [W:1.291 / U:0.012 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site