[PATCH 1/4] doc: Add ModemAccessTechnology property

2010-10-28 Thread Rajesh Kadhiravan Nagaiah
---
 doc/modem-api.txt |9 +
 1 files changed, 9 insertions(+), 0 deletions(-)

diff --git a/doc/modem-api.txt b/doc/modem-api.txt
index c48375e..f5a3026 100644
--- a/doc/modem-api.txt
+++ b/doc/modem-api.txt
@@ -43,6 +43,15 @@ Properties   boolean Powered [readwrite]
modem. The Emergency is true if an emergency call or
related operation is currently active.
 
+   string ModemAccessTechnology [readonly]
+
+   Contains current access techonology of the modem.
+   Possible values are:
+   "gsm" - the current modem access technology is GSM
+   "cdma" - the current modem access technology is CDMA
+
+   By default oFono uses "gsm" setting.
+
string Name [readonly, optional]
 
Friendly name of the modem device.
-- 
1.7.0.4

___
ofono mailing list
ofono@ofono.org
http://lists.ofono.org/listinfo/ofono


Re: [PATCH 1/4] doc: Add ModemAccessTechnology property

2010-10-28 Thread Marcel Holtmann
Hi Rajesh,

>  doc/modem-api.txt |9 +
>  1 files changed, 9 insertions(+), 0 deletions(-)
> 
> diff --git a/doc/modem-api.txt b/doc/modem-api.txt
> index c48375e..f5a3026 100644
> --- a/doc/modem-api.txt
> +++ b/doc/modem-api.txt
> @@ -43,6 +43,15 @@ Properties boolean Powered [readwrite]
>   modem. The Emergency is true if an emergency call or
>   related operation is currently active.
>  
> + string ModemAccessTechnology [readonly]
> +
> + Contains current access techonology of the modem.
> + Possible values are:
> + "gsm" - the current modem access technology is GSM
> + "cdma" - the current modem access technology is CDMA
> +
> + By default oFono uses "gsm" setting.
> +

so you are in the org.ofono.Modem interface and its properties and you
wanna prefix the property with "Modem". That is rather pointless
duplication of information. So just strip that.

The obvious question we have to discuss, if we really wanna do it this
way. What are the pros and what are the cons?

Regards

Marcel


___
ofono mailing list
ofono@ofono.org
http://lists.ofono.org/listinfo/ofono


RE: [PATCH 1/4] doc: Add ModemAccessTechnology property

2010-10-28 Thread Rajesh.Nagaiah
Hi Marcel, 
 
> Hi Rajesh,
> 
> >  doc/modem-api.txt |9 +
> >  1 files changed, 9 insertions(+), 0 deletions(-)
> > 
> > diff --git a/doc/modem-api.txt b/doc/modem-api.txt index 
> > c48375e..f5a3026 100644
> > --- a/doc/modem-api.txt
> > +++ b/doc/modem-api.txt
> > @@ -43,6 +43,15 @@ Properties   boolean Powered [readwrite]
> > modem. The Emergency is true if an 
> emergency call or
> > related operation is currently active.
> >  
> > +   string ModemAccessTechnology [readonly]
> > +
> > +   Contains current access techonology of 
> the modem.
> > +   Possible values are:
> > +   "gsm" - the current modem access 
> technology is GSM
> > +   "cdma" - the current modem access 
> technology is CDMA
> > +
> > +   By default oFono uses "gsm" setting.
> > +
> 
> so you are in the org.ofono.Modem interface and its 
> properties and you wanna prefix the property with "Modem". 
> That is rather pointless duplication of information. So just 
> strip that.

I prefix it with Modem just not to mix with the Network Access
Technology. From the property name prespective I agree with you, its an
redundant info as we are in org.ofono.Modem interface

> The obvious question we have to discuss, if we really wanna 
> do it this way. What are the pros and what are the cons?

I quite didnt get this query, pros and cons about this property usage or
?

BR,
Rajesh
___
ofono mailing list
ofono@ofono.org
http://lists.ofono.org/listinfo/ofono


RE: [PATCH 1/4] doc: Add ModemAccessTechnology property

2010-11-05 Thread Rajesh.Nagaiah
Hi Denis/Marcel, 
>  
> > Hi Rajesh,
> > 
> > >  doc/modem-api.txt |9 +
> > >  1 files changed, 9 insertions(+), 0 deletions(-)
> > > 
> > > diff --git a/doc/modem-api.txt b/doc/modem-api.txt index
> > > c48375e..f5a3026 100644
> > > --- a/doc/modem-api.txt
> > > +++ b/doc/modem-api.txt
> > > @@ -43,6 +43,15 @@ Properties boolean Powered [readwrite]
> > >   modem. The Emergency is true if an
> > emergency call or
> > >   related operation is currently active.
> > >  
> > > + string ModemAccessTechnology [readonly]
> > > +
> > > + Contains current access techonology of
> > the modem.
> > > + Possible values are:
> > > + "gsm" - the current modem access
> > technology is GSM
> > > + "cdma" - the current modem access
> > technology is CDMA
> > > +
> > > + By default oFono uses "gsm" setting.
> > > +
> > 
> > so you are in the org.ofono.Modem interface and its 
> properties and you 
> > wanna prefix the property with "Modem".
> > That is rather pointless duplication of information. So just strip 
> > that.
> 
> I prefix it with Modem just not to mix with the Network 
> Access Technology. From the property name prespective I agree 
> with you, its an redundant info as we are in org.ofono.Modem interface
> 
> > The obvious question we have to discuss, if we really wanna 
> do it this 
> > way. What are the pros and what are the cons?
> 
> I quite didnt get this query, pros and cons about this 
> property usage or ?

Any update on this ? If Modem prefix is the only comment, I can
remove and resubmit the patch again. or is there more comments ?

BR,
Rajesh
___
ofono mailing list
ofono@ofono.org
http://lists.ofono.org/listinfo/ofono