[lkml]   [2019]   [Aug]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [RFC 1/1] edac: Add a counter parameter for edac_device_handle_ue/ce()

On 8/1/2019 5:17 PM, Robert Richter wrote:
>> Don't you think it'll be confused to have different APIs between EDAC_MC and
>> (in MC the count passed as part of edac_mc_handle_error())
> I don't think edac_mc_handle_error() with 11 function arguments is a
> good reference for somethin we want to adopt. For the majority of
> drivers you just introduce another useless argument with the following
> pattern:
> edac_device_handle_ce(edac_dev, 1, 0, 0, edac_dev_name);
> IMO, the api should be improved when touching it.

Got it, I'll update the patch as you suggested.


> -Robert

 \ /
  Last update: 2019-08-01 16:31    [W:0.050 / U:3.048 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site