Hi Lucas, On Tue, 2011-01-11 at 16:08 +0200, Lucas De Marchi wrote: > Hi Jussi > > On Tue, Jan 11, 2011 at 10:17 AM,  <Jussi.Kangas(a)tieto.com> wrote: > > Hi, > > > > This is fix to Marit Henriksen's TODO item "Check SIM pin status if sim_change_pin fails". I've discussed with Marit and it's ok for her if I fix the issue. Problem here is that issue could perhaps also be fixed with retry counter solution introduced by Lucas De Marchi couple of tasks ago. That would seem however require some extra implementation in ste modem ( at least I was not able get the ofono show correct values without extra modifications ) and I think that isimodems don't have that sort of retry counter at all. Because of that and since I had this solution already implemented I propose it to be added as well. > > > > I failed to understand why you are hard-coding this and why you'd like > to use retry counters. Can't you just check the pin state by calling > sim_pin_check()? > > Moreover, this is not really implementing what the TODO item says. > > > regards, > Lucas De Marchi Marit already proposed sim_pin_check approach 2010/11/2. Denis Kenzior thought that initializing sim interface would not be a good idea. See mail "Re: [Patch] sim: Check SIM pin status after changing pin" from Friday 5th in November 2010. True, fix proposal does not do exactly what TODO item says. It trusts to returned error value instead of going to check the sim state once more. In my testings this value has been trustable so far and there has not been reason to go check the state. Of course I've tested only with one modem so this might not be a universal truth. I don't particularly wish to use the retry counters. I was just thinking that same information ( PUK required ) could be deduced by counting the retries.