[PATCH] omapdrm: no need to check return value of debugfs_create functions

[Tomi Valkeinen]

Hi,

On 08/08/2019 15:50, Laurent Pinchart wrote:
> Hi Greg,
> 
> Thank you for the patch.
> 
> On Thu, Jun 13, 2019 at 01:57:49PM +0200, Greg Kroah-Hartman wrote:
>> When calling debugfs functions, there is no need to ever check the
>> return value.  The function can work or not, but the code logic should
>> never do something different based on this.
>>
>> Cc: Tomi Valkeinen <tomi.valkeinen at ti.com>
>> Cc: David Airlie <airlied at linux.ie>
>> Cc: Daniel Vetter <daniel at ffwll.ch>
>> Cc: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
>> Cc: Sebastian Reichel <sebastian.reichel at collabora.com>
>> Cc: Jyri Sarha <jsarha at ti.com>
>> Cc: Tony Lindgren <tony at atomide.com>
>> Cc: zhong jiang <zhongjiang at huawei.com>
>> Cc: dri-devel at lists.freedesktop.org
>> Signed-off-by: Greg Kroah-Hartman <gregkh at linuxfoundation.org>
> 
> This makes sense.
> 
> Reviewed-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
> 
> Tomi, will you take this patch in your tree ?

Yes, I'll pick this up. I presume debugfs_create_file() will give a 
warning print if it fails to create the file, instead of being silent.

  Tomi

-- 
Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki.
Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki