From mboxrd@z Thu Jan 1 00:00:00 1970 Message-ID: <55D5E0F0.6040904@xenomai.org> Date: Thu, 20 Aug 2015 07:15:12 -0700 From: Jorge Ramirez Ortiz MIME-Version: 1.0 References: <55C17C8E.5000909@gatech.edu> <55C3AEBE.9080603@xenomai.org> <55C3C3F8.1090808@gatech.edu> <55C3C7F4.1030107@xenomai.org> <12F45B9B-70C3-4278-BA0E-DFC26E19DBF0@gatech.edu> <55C495D6.2070706@xenomai.org> <06B32287-9FE2-42E5-BC2B-E9C807D0BD69@gatech.edu> <55C4BD22.9060809@xenomai.org> <55C515F0.8080307@xenomai.org> <45806C53-526E-425C-AEA4-70B2A22EA364@gatech.edu> <55C51965.2020206@xenomai.org> <933604EB-994B-4CE9-BA38-DB404CA0811D@gatech.edu> <55C522AE.8030403@xenomai.org> <55C9EAA6.6030503@gatech.edu> <55D15EDB.4060107@gatech.edu> <55D1E001.5080905@xenomai.org> <55D1EE0A.6030100@gatech.edu> In-Reply-To: <55D1EE0A.6030100@gatech.edu> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Subject: Re: [Xenomai] analogy calibration issue(s) List-Id: Discussions about the Xenomai project List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Yogi A. Patel" Cc: "xenomai@xenomai.org" On 08/17/2015 07:22 AM, Yogi A. Patel wrote: > >>> diff --git a/lib/analogy/calibration.c b/lib/analogy/calibration.c >>> index b50cb2f..d062a00 100644 >>> --- a/lib/analogy/calibration.c >>> +++ b/lib/analogy/calibration.c >>> @@ -105,7 +105,10 @@ static inline int read_int(int *val, struct _dictionary_ >>> *f, const char *subd, >>> >>> *val = iniparser_getint(f, str, 0xFFFF); >>> if (*val == 0xFFFF) >>> + { >>> + *val = -1; >>> ret = -ENOENT; >> this is not correct. >> the coefficient was not found and so an error must be return. > Yes, I agree that an error must be returned. The problem is that *val is > pointing to 0XFFFF - which is nb_elements. So when you check if(nb_elements < > 0), it still proceeds, rather than returning an error as it should. > ACKd. Not sure what I was thinking. > Thanks for sending your file. I am absurdly confused as to why I don't get an > output section..have you tried this with any other cards? > No, I only have access to the 6221. Just enable the debugging information in the kernel config. you can also use ftrace (supported by analogy) -- jro