Re: [PATCH v2 0/6] Strip whitespace and empty strings from DBus input

2015-07-17 Thread Jukka Rissanen 
Hi Jaakko,

On ke, 2015-07-15 at 15:26 +0300, Jaakko Hannikainen wrote:
 Hello,
 
 These patches remove empty strings and leading/trailing whitespace from DBus  
   
 input for Domains.Configuration, Proxy.Configuration, 
 Timeservers.Configuration 
 and Nameservers.Configuration.
   
   
   
 Previously, Nameservers.Configuration accepted invalid values such as 
   
 192.0.2.1  invalid-ip 8.8.8.8  and then split them with space as delimiter, 
   
 resulting in Nameservers.Configuration containing 
   
 [192.0.2.1, , invalid-ip, 8.8.8.8, ] rather than the valid value  
   
 [192.0.2.1, 8.8.8.8]. 
   
   
   
 The v2 implements remove_empty_strings much more sensibly, fixes some style   
   
 issues and adds a patch for Nameservers.Configuration.
 
 Managed to fudge the cover letter first time around, sorry.
 
 Jaakko Hannikainen (6):
   service: Add function to remove empty strings
   service: Remove empty strings from Domains
   service: Remove empty strings from Proxy
   service: Strip whitespace from Proxy URL
   service: Remove empty strings from Timeservers
   service: Remove empty strings from Nameservers
 
  src/service.c | 109 
 +++---
  1 file changed, 74 insertions(+), 35 deletions(-)
 

All patches have been applied, thanks!


Cheers,
Jukka


___
connman mailing list
connman@connman.net
https://lists.connman.net/mailman/listinfo/connman

RE: cellular technology is not getting listed

2015-07-17 Thread Kallumari Nagaraja Rao, RammohanX 
Hello Jukka,

I found out the issue.
ConnMan was registering the cellular driver.

During the ConnMan init process it calls up the plugins/ofono/ ofono_init 
function and registers the cellular driver.

That is not happening in the earlier daemon where cellular technology not 
getting listed.

Issue:
I have 2 copies of the connmand one with size around 770kb and other around 
2400kb.
With the small size daemon, ofono plugin does not seem to work.
However, with the daemon of size 2400kb, connman is able to register the 
cellular driver and
Cellular technology is visible in connmanctl.

Please let me know if I am missing again something.

Regards,
Ram.

-Original Message-
From: Jukka Rissanen [mailto:jukka.rissa...@linux.intel.com] 
Sent: Thursday, July 16, 2015 3:25 PM
To: connman@connman.net
Cc: Kallumari Nagaraja Rao, RammohanX
Subject: Re: cellular technology is not getting listed

Hi Ram,

On to, 2015-07-16 at 08:24 +, Kallumari Nagaraja Rao, RammohanX
wrote:
 All,
 
 Following things are done,
 
 1./var/lib/connman cellular added

What do you mean by this?

 
 2./etc/connman/main.conf has wifi and cellular as preferred technologies

PreferredTechnologies only affects how ConnMan does autoconnect.

 
 3.connmand is running

I assume you have ofono plugin compiled and used by ConnMan? Are you using 
Yocto based distro or have you compiled ConnMan yourself?

 
 4.ofonod running
 
 5.cellular context was created after enabling the he910 modem

So ofono shows the cellular context correctly if you invoke list-contexts 
script from ofono, is it so?
Do you have APN and PIN correctly setup?

 
 6.ifconfig list ppp0

What does the above command show?

 
 however, connmanctl does not list the cellular technology.

So the cellular technology is not shown which indicates that ConnMan does not 
know anything about the modem. If the cellular service is not shown (but 
technology is), then the cellular context is not properly setup.

What ConnMan version are you using?

 
 After doing ifconfig, ppp0 does not have inet addr listed.
 
 Please let me know if there is something missing.
 
 Regards,
 Ram.


Cheers,
Jukka


___
connman mailing list
connman@connman.net
https://lists.connman.net/mailman/listinfo/connman

Re: [PATCH] doc/technology-api.txt: Fix typo

2015-07-17 Thread Jukka Rissanen 
Hi Marko,

On ke, 2015-07-15 at 15:03 +0200, Marko Sulejic wrote:
 ---
  doc/technology-api.txt | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)
 
 diff --git a/doc/technology-api.txt b/doc/technology-api.txt
 index f97eac0..f3703c0 100644
 --- a/doc/technology-api.txt
 +++ b/doc/technology-api.txt
 @@ -56,7 +56,7 @@ Properties  boolean Powered [readwrite]
  
   Boolean representing if a technology is connected.
  
 - This is just a convience property for allowing the
 + This is just a convenience property for allowing the
   UI to easily show if this technology has an active
   connection or not.
  

patch has been applied, thanks!


Cheers,
Jukka


___
connman mailing list
connman@connman.net
https://lists.connman.net/mailman/listinfo/connman