device type in apple-mfi-fastcharfe

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

looking at this bug report:

https://bugzilla.kernel.org/show_bug.cgi?id=215799

I had an occasion to look at the way it registers the charger.
In particular:
 .type                   = POWER_SUPPLY_TYPE_BATTERY,

That seems kind of wrong. It does not contain a battery, does it? The
type seems to confound user space. Is there some deeper reason? If not
how about the attached patch? Regards Oliver
From e1e7805585a979f74ba86a407ca8246a51c47a97 Mon Sep 17 00:00:00 2001
From: Oliver Neukum <oneukum@xxxxxxxx>
Date: Tue, 5 Apr 2022 11:40:39 +0200
Subject: [PATCH] apple-mfi-fastcharge: correct type

This is a charger, not a battery.

Signed-off-by: Oliver Neukum <oneukum@xxxxxxxx>
---
 drivers/usb/misc/apple-mfi-fastcharge.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/usb/misc/apple-mfi-fastcharge.c b/drivers/usb/misc/apple-mfi-fastcharge.c
index ac8695195c13..b8b394df622f 100644
--- a/drivers/usb/misc/apple-mfi-fastcharge.c
+++ b/drivers/usb/misc/apple-mfi-fastcharge.c
@@ -157,7 +157,7 @@ static enum power_supply_property apple_mfi_fc_properties[] = {
 
 static const struct power_supply_desc apple_mfi_fc_desc = {
 	.name                   = "apple_mfi_fastcharge",
-	.type                   = POWER_SUPPLY_TYPE_BATTERY,
+	.type                   = POWER_SUPPLY_TYPE_USB,
 	.properties             = apple_mfi_fc_properties,
 	.num_properties         = ARRAY_SIZE(apple_mfi_fc_properties),
 	.get_property           = apple_mfi_fc_get_property,
-- 
2.34.1


[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux