SIM power patch

Aleksander Morgado aleksander at aleksander.es
Mon Feb 13 16:57:52 UTC 2017


On Mon, Feb 13, 2017 at 5:09 PM, Kasper Holtze <kasper at holtze.dk> wrote:
>>>> I have updated the patch - it is attached to this mail.
>>>
>>> The patch doesn't apply ("git am 0001-...") on top of libqmi git
>>> master; could you check why?
>>>
>> It applies to the current master now. There's two warnings regarding
>> whitespaces _above_ the code I added, in the JSON file (it complains about
>> trailing whitespaces, guess it has to do with the tabs in the lines
>> above..).
>>
>> $ git apply --check 0001-Added-SIM-power-on-off-functionality.patch
>>
>> $ git am 0001-Added-SIM-power-on-off-functionality.patch
>> Applying: Added-SIM-power-on-off-functionality
>> .git/rebase-apply/patch:16: trailing whitespace.
>>
>> .git/rebase-apply/patch:32: trailing whitespace.
>>
>> warning: 2 lines add whitespace errors.
>>
>> Kasper
>
> ...now with "since" tags...

The trailing whitespaces are actually new in your patch :) I fixed
those before pushing to git master.

I've also added the missing documentation links in a follow up commit.

And last, I've ended up separating the on/off operations into 2
different commands in qmicli... --uim-sim-power-off and
--uim-sim-power-on; basically so that we keep the same logic of one
qmicli operation per QMI message supported.

All in git master now.

Thanks!

-- 
Aleksander
https://aleksander.es


More information about the libqmi-devel mailing list