SIM power patch

Aleksander Morgado aleksander at aleksander.es
Mon Feb 13 19:26:17 UTC 2017


On Mon, Feb 13, 2017 at 7:57 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!
>>
> Alright, thank you! Sorry for all the issues - hope it was a little help
> anyways.

Sorry for what? :) thanks to you for the patch.

-- 
Aleksander
https://aleksander.es


More information about the libqmi-devel mailing list