Re: [PATCH 3/6] si4713 : Bug fix for si4713_tx_tune_power() method in the i2c driver

2013-09-02 Thread Hans Verkuil
On 09/02/2013 08:59 AM, Hans Verkuil wrote: > On 09/01/2013 04:57 PM, edubez...@gmail.com wrote: >> Hello Hans, >> >> On Sun, Sep 1, 2013 at 7:04 AM, Hans Verkuil wrote: >>> On 08/31/2013 01:49 PM, edubez...@gmail.com wrote: Hi Dinesh, On Fri, Aug 30, 2013 at 7:28 AM, Dinesh Ram wr

Re: [PATCH 3/6] si4713 : Bug fix for si4713_tx_tune_power() method in the i2c driver

2013-09-02 Thread Hans Verkuil
On 09/01/2013 04:57 PM, edubez...@gmail.com wrote: > Hello Hans, > > On Sun, Sep 1, 2013 at 7:04 AM, Hans Verkuil wrote: >> On 08/31/2013 01:49 PM, edubez...@gmail.com wrote: >>> Hi Dinesh, >>> >>> On Fri, Aug 30, 2013 at 7:28 AM, Dinesh Ram wrote: In the si4713_tx_tune_power() method, the

Re: [PATCH 3/6] si4713 : Bug fix for si4713_tx_tune_power() method in the i2c driver

2013-09-01 Thread edubez...@gmail.com
Hello Hans, On Sun, Sep 1, 2013 at 7:04 AM, Hans Verkuil wrote: > On 08/31/2013 01:49 PM, edubez...@gmail.com wrote: >> Hi Dinesh, >> >> On Fri, Aug 30, 2013 at 7:28 AM, Dinesh Ram wrote: >>> In the si4713_tx_tune_power() method, the args array element 'power' can >>> take values between >>> SI

Re: [PATCH 3/6] si4713 : Bug fix for si4713_tx_tune_power() method in the i2c driver

2013-09-01 Thread Hans Verkuil
On 08/31/2013 01:49 PM, edubez...@gmail.com wrote: > Hi Dinesh, > > On Fri, Aug 30, 2013 at 7:28 AM, Dinesh Ram wrote: >> In the si4713_tx_tune_power() method, the args array element 'power' can >> take values between >> SI4713_MIN_POWER and SI4713_MAX_POWER. power = 0 is also valid. >> All the

Re: [PATCH 3/6] si4713 : Bug fix for si4713_tx_tune_power() method in the i2c driver

2013-08-31 Thread edubez...@gmail.com
Hi Dinesh, On Fri, Aug 30, 2013 at 7:28 AM, Dinesh Ram wrote: > In the si4713_tx_tune_power() method, the args array element 'power' can take > values between > SI4713_MIN_POWER and SI4713_MAX_POWER. power = 0 is also valid. > All the values (0 > power < SI4713_MIN_POWER) are illegal and hence >

[PATCH 3/6] si4713 : Bug fix for si4713_tx_tune_power() method in the i2c driver

2013-08-30 Thread Dinesh Ram
In the si4713_tx_tune_power() method, the args array element 'power' can take values between SI4713_MIN_POWER and SI4713_MAX_POWER. power = 0 is also valid. All the values (0 > power < SI4713_MIN_POWER) are illegal and hence are all mapped to SI4713_MIN_POWER. Signed-off-by: Dinesh Ram --- driv