Received-SPF: pass (google.com: domain of linux-kernel+bounces-55068-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
From: Javier Carrasco <javier.carrasco@wolfvision.net>
Date: Tue, 06 Feb 2024 14:59:30 +0100
Subject: [PATCH v3 2/7] usb: misc: onboard_dev: add support for non-hub
 devices
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
Message-Id: <20240206-onboard_xvf3500-v3-2-f85b04116688@wolfvision.net>
References: <20240206-onboard_xvf3500-v3-0-f85b04116688@wolfvision.net>
In-Reply-To: <20240206-onboard_xvf3500-v3-0-f85b04116688@wolfvision.net>
To: Liam Girdwood <lgirdwood@gmail.com>, Mark Brown <broonie@kernel.org>, 
 Rob Herring <robh+dt@kernel.org>, 
 Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>, 
 Conor Dooley <conor+dt@kernel.org>, Matthias Kaehlcke <mka@chromium.org>, 
 Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: linux-sound@vger.kernel.org, devicetree@vger.kernel.org, 
 linux-kernel@vger.kernel.org, linux-usb@vger.kernel.org, 
 Javier Carrasco <javier.carrasco@wolfvision.net>
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?L3ROa0F6Z0xvMDRKdWVPdlYzbTJKTlBQRmFwOVZGMmJrQloyOFRDSFB5dVk2?=
 =?utf-8?B?d3VVQ25PQitmRGZkMXhOWjZob0hORXBubWE5dDIrNDdtQmRpMzdLUXVmUGlS?=
 =?utf-8?B?LzRWems3SjVFZWwxU1dVZTFLcGRQeWk0ZVgxMjR5SGltQzArY3FCUUovNk1i?=
 =?utf-8?B?RXFYSVdiRlEzR2pVemcvVkZVcUZIMlVzdzVlQVBhb1JEVy9pOHFHWUlmZDN2?=
 =?utf-8?B?L0w0dzlDVkJQSFFFWFJCTm5ScEg5Sm5SS29oY0dIa29zbnJhdGdXeXpRZW1G?=
 =?utf-8?B?K0gyNW16eFpwaDI2UktJekg1T0RMdi9tb3BHVElWd0ZBSyswb043bEt1bEta?=
 =?utf-8?B?NnlTdWFuU2pISkxEV2k0aVhOQzVhTUJiREorK1NGTndZM1NiQm1NTlFHUFhl?=
 =?utf-8?B?dGFWejhyeWFjUkJLQXltdUQzb01xRDVxQkMwSXBQendBalV6ayt4TFUxZXE2?=
 =?utf-8?B?U1k0aFhTQ0Y0WTYvckhsRm16QlpHWnI0aFphT0tMOGpQUzJOR0hGV1YwamtR?=
 =?utf-8?B?S1RRU2RkS1laWWN0Rld3T0czTXpOY29qSXh3QkFySUpYUFpHdGFRS3FvcnFI?=
 =?utf-8?B?UTA0ODRXSFBRTlZUUGcwZVpCbEVDd1UxZVNZenZZQ0lPNC9SMEw5Mm1nUmFK?=
 =?utf-8?B?d3JmcWJZSSt6aHE3UlVyNnpYWkwvTnFoV3lzbWE2VE5ac3BCWXpKSUwzTFRR?=
 =?utf-8?B?WnNYNGNaWEpQeFRRRjJlekF4a01tYVVSWU5hM1FYSTIyWURMU0dvc0s4RjRm?=
 =?utf-8?B?ekpMcjByMzRIRlFwNUM0MUdNSHhsVTBFWlRreDVXWGt4NExqc2RNQkhWT1Q2?=
 =?utf-8?B?TVFmUTErYTdqYTBCTVJCemE3VzZKSEJjTlYwdGd0VXJJUnloYnc1WEFFcEY1?=
 =?utf-8?B?MW44WWJYVU1ld0FEdU1abDI5b2xwbDlpcVZLdFk1M2pQS2cxWHh4eDhyZVF2?=
 =?utf-8?B?T08za3hxc1U1am80ZVJYREk2RWtvMzRkYkZZc2krZlAxcFRLSmJNRENMU3Va?=
 =?utf-8?B?Z1BuSTkydHdDdS8zaXZON204ZmUvL0VjcmZ4SzQ4VlRQakoxNFJFeEExRmdI?=
 =?utf-8?B?aWE3SHNBM05Ic2JnTmhhdG9KK3BTUTRISXVjMWNCSXpNMXQ4SmE2T0VSc3RU?=
 =?utf-8?B?L1Ruamx2RTVKTTZoV0xEMzBRNXgyeGMrNjlYSzZCSTh1S2RPSXA1L1BDMHdy?=
 =?utf-8?B?cU9UTndQRk0wclZtK1hBVGh5U01kMlNoa1JnajJ3Z292RHVERmt0NXZpQmRK?=
 =?utf-8?B?NE5YMEdSU2VXd3lxd0tWSzAxK1UyUjNaRE9lR1hSUFovREdNN1A4V1NRYWhP?=
 =?utf-8?B?QUxCQkJCQUVMeEpZMks1NGhiaTk0RGY1MWkrWXloZE0zdmExRXJtdnozVTJo?=
 =?utf-8?B?bEYyRjZFbVBGQWQ2K1NZMFBmSHZQYVk5ZnZYTTlnUXNzN3p3cHNZM250QnlH?=
 =?utf-8?B?WXdOTXdERG1iRlorNnZtS0pscm02eWY5ek9PK0ZvTytXRzdMN2t1RkxROHpk?=
 =?utf-8?B?Rk1ka2lkVStoelFjcU4veHZUdDJTS1ZFUTJFV3lXUEtzZTZDTkZ5c2Jod3JZ?=
 =?utf-8?B?TnhsaGRBaFR2VzBGQk0wbWhUTjMwUnBrZkRlVEU2VThnNDRxS1BHQkIvdVh2?=
 =?utf-8?B?R0RUbjlCWEExamRJcjg5UkpmS2ErUXlBT0MwcDRlczlLUndHZDFIMzVpb0Qz?=
 =?utf-8?B?dmVDLzNBUHFpeWoxS1FPTW5UTFViMVpnaGdLR3lPU2xIM2NlNGMveFBlaVhD?=
 =?utf-8?B?WTFZM1RIV0xvZ3lYN0NOVXkzUVVZZDRwV3VvUjMzaFFYOEQzTHpiU25DZDRD?=
 =?utf-8?B?OTBGVlhybnZreDNiUVdVb2pVMVg5Tlh1eUQzZUZqaVcwV2NEU3hveHUxMWdC?=
 =?utf-8?B?OVBNdUJIK2NTTmlxZTY4VG1vNjI3QkpzRXBaN3ZMOE1PWmtGeW12eTFKMFVq?=
 =?utf-8?B?UVBVNEIwZE1XYVZvNW5WQlVFK1dSZ3RXYWxFemhQUVBiZGw4NU0vbHRoTjVJ?=
 =?utf-8?B?UzdMaHRpb1VBWWdLQzg2YzBZSk8weHhTVlJoRlErdWY2b0VINm9qZGozRFJa?=
 =?utf-8?B?T214NzRhek82RWp5WnlJTEp4d01HTVRSV3cyVlVEQ2FYa3F5OEVZVDQwam9n?=
 =?utf-8?B?Q2hEVHpLU2V2blBtdDNtQTg2NTZ2T2JwMFpHWkZnMGFlUUZ3V0JDWk5pYWVL?=
 =?utf-8?Q?aK5yCoNEnS950yFvUi07AfM=3D?=
X-OriginatorOrg: wolfvision.net
X-MS-Exchange-CrossTenant-Network-Message-Id: 99caceb5-12c7-4bef-5d62-08dc271bdb3b
X-MS-Exchange-CrossTenant-AuthSource: VE1PR08MB4974.eurprd08.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 06 Feb 2024 13:59:37.6265
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: e94ec9da-9183-471e-83b3-51baa8eb804f
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: quT0Qiaa5WCD02Q6kxBnY5rhZ+cstGuAMivvKgiTeErJ7P7v3G4ckM8MZI9NaRK11r65YNBInZ6oeklEaUHo1wgco+CYgGo9ygaWMo+/f8Y=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB9PR08MB6330

Most of the functionality this driver provides can be used by non-hub
devices as well.

To account for the hub-specific code, add a flag to the device data
structure and check its value for hub-specific code.

Signed-off-by: Javier Carrasco <javier.carrasco@wolfvision.net>
---
 drivers/usb/misc/onboard_usb_dev.c |  3 +++
 drivers/usb/misc/onboard_usb_dev.h | 10 ++++++++++
 2 files changed, 13 insertions(+)

diff --git a/drivers/usb/misc/onboard_usb_dev.c b/drivers/usb/misc/onboard_usb_dev.c
index e2e1e1e30c1e..3ac21ec38ac0 100644
--- a/drivers/usb/misc/onboard_usb_dev.c
+++ b/drivers/usb/misc/onboard_usb_dev.c
@@ -123,6 +123,9 @@ static int __maybe_unused onboard_dev_suspend(struct device *dev)
 	if (onboard_dev->always_powered_in_suspend)
 		return 0;
 
+	if (!onboard_dev->pdata->is_hub)
+		return onboard_dev_power_off(onboard_dev);
+
 	mutex_lock(&onboard_dev->lock);
 
 	list_for_each_entry(node, &onboard_dev->udev_list, list) {
diff --git a/drivers/usb/misc/onboard_usb_dev.h b/drivers/usb/misc/onboard_usb_dev.h
index f13d11a84371..ebe83e19d818 100644
--- a/drivers/usb/misc/onboard_usb_dev.h
+++ b/drivers/usb/misc/onboard_usb_dev.h
@@ -9,51 +9,61 @@
 struct onboard_dev_pdata {
 	unsigned long reset_us;		/* reset pulse width in us */
 	unsigned int num_supplies;	/* number of supplies */
+	bool is_hub;
 };
 
 static const struct onboard_dev_pdata microchip_usb424_data = {
 	.reset_us = 1,
 	.num_supplies = 1,
+	.is_hub = true,
 };
 
 static const struct onboard_dev_pdata microchip_usb5744_data = {
 	.reset_us = 0,
 	.num_supplies = 2,
+	.is_hub = true,
 };
 
 static const struct onboard_dev_pdata realtek_rts5411_data = {
 	.reset_us = 0,
 	.num_supplies = 1,
+	.is_hub = true,
 };
 
 static const struct onboard_dev_pdata ti_tusb8041_data = {
 	.reset_us = 3000,
 	.num_supplies = 1,
+	.is_hub = true,
 };
 
 static const struct onboard_dev_pdata cypress_hx3_data = {
 	.reset_us = 10000,
 	.num_supplies = 2,
+	.is_hub = true,
 };
 
 static const struct onboard_dev_pdata cypress_hx2vl_data = {
 	.reset_us = 1,
 	.num_supplies = 1,
+	.is_hub = true,
 };
 
 static const struct onboard_dev_pdata genesys_gl850g_data = {
 	.reset_us = 3,
 	.num_supplies = 1,
+	.is_hub = true,
 };
 
 static const struct onboard_dev_pdata genesys_gl852g_data = {
 	.reset_us = 50,
 	.num_supplies = 1,
+	.is_hub = true,
 };
 
 static const struct onboard_dev_pdata vialab_vl817_data = {
 	.reset_us = 10,
 	.num_supplies = 1,
+	.is_hub = true,
 };
 
 static const struct of_device_id onboard_dev_match[] = {

-- 
2.40.1