Linux-OMAP Archive on lore.kernel.org
 help / color / Atom feed
* [PATCH 4/5] power: supply: cpcap-charger: fix small mistake in current to register conversion
@ 2021-01-17 21:47 Carl Philipp Klemm
  2021-04-04  7:23 ` Tony Lindgren
  0 siblings, 1 reply; 3+ messages in thread
From: Carl Philipp Klemm @ 2021-01-17 21:47 UTC (permalink / raw)
  To: Sebastian Reichel
  Cc: linux-pm, linux-omap, Arthur Demchenkov, Tony Lindgren,
	Merlijn Wajer, Pavel Machek

Signed-off-by: Carl Philipp Klemm <philipp@uvos.xyz>
---
 drivers/power/supply/cpcap-charger.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/power/supply/cpcap-charger.c b/drivers/power/supply/cpcap-charger.c
index 152090faf5b2..be7d1da7a5a5 100644
--- a/drivers/power/supply/cpcap-charger.c
+++ b/drivers/power/supply/cpcap-charger.c
@@ -300,7 +300,7 @@ static int cpcap_charger_current_to_regval(int microamp)
 		return CPCAP_REG_CRM_ICHRG(0x0);
 	if (miliamp < 177)
 		return CPCAP_REG_CRM_ICHRG(0x1);
-	if (miliamp > 1596)
+	if (miliamp >= 1596)
 		return CPCAP_REG_CRM_ICHRG(0xe);
 
 	res = microamp / 88666;
-- 
2.29.2


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH 4/5] power: supply: cpcap-charger: fix small mistake in current to register conversion
  2021-01-17 21:47 [PATCH 4/5] power: supply: cpcap-charger: fix small mistake in current to register conversion Carl Philipp Klemm
@ 2021-04-04  7:23 ` Tony Lindgren
  2021-04-05  8:22   ` Sebastian Reichel
  0 siblings, 1 reply; 3+ messages in thread
From: Tony Lindgren @ 2021-04-04  7:23 UTC (permalink / raw)
  To: Carl Philipp Klemm
  Cc: Sebastian Reichel, linux-pm, linux-omap, Arthur Demchenkov,
	Merlijn Wajer, Pavel Machek

* Carl Philipp Klemm <philipp@uvos.xyz> [210117 23:47]:
> Signed-off-by: Carl Philipp Klemm <philipp@uvos.xyz>

Acked-by: Tony Lindgren <tony@atomide.com>

> ---
>  drivers/power/supply/cpcap-charger.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/power/supply/cpcap-charger.c b/drivers/power/supply/cpcap-charger.c
> index 152090faf5b2..be7d1da7a5a5 100644
> --- a/drivers/power/supply/cpcap-charger.c
> +++ b/drivers/power/supply/cpcap-charger.c
> @@ -300,7 +300,7 @@ static int cpcap_charger_current_to_regval(int microamp)
>  		return CPCAP_REG_CRM_ICHRG(0x0);
>  	if (miliamp < 177)
>  		return CPCAP_REG_CRM_ICHRG(0x1);
> -	if (miliamp > 1596)
> +	if (miliamp >= 1596)
>  		return CPCAP_REG_CRM_ICHRG(0xe);
>  
>  	res = microamp / 88666;
> -- 
> 2.29.2
> 

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH 4/5] power: supply: cpcap-charger: fix small mistake in current to register conversion
  2021-04-04  7:23 ` Tony Lindgren
@ 2021-04-05  8:22   ` Sebastian Reichel
  0 siblings, 0 replies; 3+ messages in thread
From: Sebastian Reichel @ 2021-04-05  8:22 UTC (permalink / raw)
  To: Tony Lindgren
  Cc: Carl Philipp Klemm, linux-pm, linux-omap, Arthur Demchenkov,
	Merlijn Wajer, Pavel Machek


[-- Attachment #1: Type: text/plain, Size: 978 bytes --]

Hi,

On Sun, Apr 04, 2021 at 10:23:27AM +0300, Tony Lindgren wrote:
> * Carl Philipp Klemm <philipp@uvos.xyz> [210117 23:47]:
> > Signed-off-by: Carl Philipp Klemm <philipp@uvos.xyz>
> 
> Acked-by: Tony Lindgren <tony@atomide.com>

Thanks, queued.

-- Sebastian

> > ---
> >  drivers/power/supply/cpcap-charger.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/drivers/power/supply/cpcap-charger.c b/drivers/power/supply/cpcap-charger.c
> > index 152090faf5b2..be7d1da7a5a5 100644
> > --- a/drivers/power/supply/cpcap-charger.c
> > +++ b/drivers/power/supply/cpcap-charger.c
> > @@ -300,7 +300,7 @@ static int cpcap_charger_current_to_regval(int microamp)
> >  		return CPCAP_REG_CRM_ICHRG(0x0);
> >  	if (miliamp < 177)
> >  		return CPCAP_REG_CRM_ICHRG(0x1);
> > -	if (miliamp > 1596)
> > +	if (miliamp >= 1596)
> >  		return CPCAP_REG_CRM_ICHRG(0xe);
> >  
> >  	res = microamp / 88666;
> > -- 
> > 2.29.2
> > 

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, back to index

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-01-17 21:47 [PATCH 4/5] power: supply: cpcap-charger: fix small mistake in current to register conversion Carl Philipp Klemm
2021-04-04  7:23 ` Tony Lindgren
2021-04-05  8:22   ` Sebastian Reichel

Linux-OMAP Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-omap/0 linux-omap/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-omap linux-omap/ https://lore.kernel.org/linux-omap \
		linux-omap@vger.kernel.org
	public-inbox-index linux-omap

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.vger.linux-omap


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git