From patchwork Wed Feb 28 13:51:33 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Javier Carrasco X-Patchwork-Id: 13575512 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 74161C54E4A for ; Wed, 28 Feb 2024 13:52:32 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Cc:To:In-Reply-To: References:Message-Id:Subject:Date:From:Reply-To:Content-ID: Content-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc :Resent-Message-ID:List-Owner; bh=J8JFjOszqvOoml8Th3KsZSCTahJWqqGtGDfbOUOfRNA=; b=ZS9kSqnPPPs3QDUQA+soIp+Jb7 zBcBI/J9tM6DfVsFb9gdPXnm3nBuLX35hiG56TT8LCJX0yqXCrvAahK3ZZHcqRXMRAxSW4wba3t48 I2ygu2+0gVfRPJoYv/B35NNWmuRA94X0QmIFXHlxgPA9o6ynnrl3QWBIFWuH0Y0LKvn94wO0VhlPz Y4sOaK0S57h3H7MgWAc9F/9CLKqYhvHIWwWSrSL1z0GRUtAwLuROQe/S8H4Krcyo4KJRkTmpr2o5d SKMTLQnyL2nh/O46Cnt3f5mRA2Dz7lX0l2b66y31WX/Nb8nvvh4uwjm8hf+w/43MNCMowz720kuzb xLDsfipQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1rfKM2-00000009YEo-2dpF; Wed, 28 Feb 2024 13:52:14 +0000 Received: from mail-db3eur04on072f.outbound.protection.outlook.com ([2a01:111:f400:fe0c::72f] helo=EUR04-DB3-obe.outbound.protection.outlook.com) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1rfKLk-00000009Xxe-471r for linux-arm-kernel@lists.infradead.org; Wed, 28 Feb 2024 13:51:58 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=LeF8qy+uiXA3HI55U4qIV18yxDbiapz0WM9BRdotXj1eCelM8xUHNt2mDfgqKzkDSiOtlIWCAxHd3LFFV8G5dw0zLob3GQX7cJzq2nMuAA0AZWEYvKfCz6meZ4Qmy1SOue0DN99LEuk8CHTDGdo5fO2ogUffpzDHMNFdDuGR89b5LHtIVKMlj1xH7UBGhackZ63M9pD6fSSryhrWOt8jVR3k/8iPOMG+scMZXT7jpeHL51PVF8m11AkOc7t3XyOedsQgyAEWFH/4qr33ZQKtgn+xJeKYurQrveZiXDxDV72sL9sgNoHGZiWyJ6Qj/hCbtfdpOyBYRvASmDq/1T7Pig== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=k5SXUlle2NVEXwDFcWjTn+XnAibneyXcGO3oVO7aYIU=; b=aNf+S8+AJ1vPzeYD3D3/jOQ/O5MRsNoPJOAbVSbQ/OjWRw019/ObxBVCJTj9QJZn5VskBzBC3VLasBhsSzZAG2tZW24r/vAXLAkJcrHmzRnciWxV9p+f2IAGeaIS2KYXgdvmvL8086k7D41tH6+yP0+VzW2JNK7wIW3jNJXNFpsjA8nCL0cJbdiiCqux9QI80STRijB8NuRalCy209Wzlyq9VK8D1HMI/tG+LOPUAKSwrN5OGyiB5E2/oEUje0tgrTWVoDux+t90LyMrPhG0VWic/m2oFyB0W8/Fg6VTNep10SBVWwrlx0zel+bPkXVod0XMlSbwT7JjnkO2hWpj/A== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=wolfvision.net; dmarc=pass action=none header.from=wolfvision.net; dkim=pass header.d=wolfvision.net; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=wolfvision.net; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=k5SXUlle2NVEXwDFcWjTn+XnAibneyXcGO3oVO7aYIU=; b=nksWASWUJtTZfr4ZUl9jF1YGQ0L5j+W9xizP5TWqGdErX3biKgwUJoiMbdVJb1mvRIE+iM1dyCONuE3Fcpvkp+bZygV9qjf1Wy8sPxYVsk9SxMM8cF40NREVBecVJHKoqZHWougYUNL18qpPs63I6UrX6250Kjw1+mDuCId0c1Y= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=wolfvision.net; Received: from VE1PR08MB4974.eurprd08.prod.outlook.com (2603:10a6:803:111::15) by GV1PR08MB10523.eurprd08.prod.outlook.com (2603:10a6:150:16a::22) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7316.40; Wed, 28 Feb 2024 13:51:43 +0000 Received: from VE1PR08MB4974.eurprd08.prod.outlook.com ([fe80::9e35:6de9:e4fc:843f]) by VE1PR08MB4974.eurprd08.prod.outlook.com ([fe80::9e35:6de9:e4fc:843f%6]) with mapi id 15.20.7316.035; Wed, 28 Feb 2024 13:51:43 +0000 From: Javier Carrasco Date: Wed, 28 Feb 2024 14:51:33 +0100 Subject: [PATCH v5 6/8] usb: misc: onboard_dev: add support for non-hub devices Message-Id: <20240228-onboard_xvf3500-v5-6-76b805fd3fe6@wolfvision.net> References: <20240228-onboard_xvf3500-v5-0-76b805fd3fe6@wolfvision.net> In-Reply-To: <20240228-onboard_xvf3500-v5-0-76b805fd3fe6@wolfvision.net> To: Liam Girdwood , Mark Brown , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Matthias Kaehlcke , Greg Kroah-Hartman , Helen Koike , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Daniel Vetter , Catalin Marinas , Will Deacon , Russell King Cc: linux-sound@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-usb@vger.kernel.org, dri-devel@lists.freedesktop.org, linux-arm-kernel@lists.infradead.org, Javier Carrasco X-Mailer: b4 0.14-dev X-Developer-Signature: v=1; a=ed25519-sha256; t=1709128293; l=4013; i=javier.carrasco@wolfvision.net; s=20230509; h=from:subject:message-id; bh=moj7dNPzywLu5KBQuWEF56AL6ondXwteo2SwhW8gJKY=; b=QRj/qc1Eh7opso6M3JwpKRV4SYDiDZ+bYYZa0ss103gWaRUSBk97Zg/2pfEyJ0JJP5DvFa+c0 n4qoet/jCZnB82ocwcpygGVBJn7ddafJJWuTAAlzBMKhXULBtiJYw/O X-Developer-Key: i=javier.carrasco@wolfvision.net; a=ed25519; pk=tIGJV7M+tCizagNijF0eGMBGcOsPD+0cWGfKjl4h6K8= X-ClientProxiedBy: FR3P281CA0184.DEUP281.PROD.OUTLOOK.COM (2603:10a6:d10:a4::19) To VE1PR08MB4974.eurprd08.prod.outlook.com (2603:10a6:803:111::15) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: VE1PR08MB4974:EE_|GV1PR08MB10523:EE_ X-MS-Office365-Filtering-Correlation-Id: 07085d1b-f809-472b-bcb0-08dc3864656f X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: l+xdbm7iEuH4vU3B6xmAgcnVSZ+BzLEQ4BYVFlwDGJAGFIIMm1iJy1L/6xMVj1npe0U97GNlG4XeuAueUxypNcX95OolgWapvgGyJPMtMaCuDy+olOCyB3TDLLNeDwnN8zOsOCBSq7kvS7IsbwkXG5DRJCcQzKMvvuVIFMKvMsHcBa0jf3wZLdGT1GAGUDTbDRpbqMLx3koYr+jXvFkLry2VL5XykmsqbY5UE7Eg56vLv/lP8hSs2q/5PBOJXWzMYF3fIkMSBF3i+5jv0KXSWM3RhGeeX5P04xd7Tv0/cIO5qtm4fi5B7dPorH/8v3mXQvW5Wh3A3kHyYrYEFoepCKWwy2+rSUUTT9A6Z+A/jCqeqjFA4MLjWfxeFwzttZF8k2brCmlmWhuaACsc5N+9LOalAJMwqyZMRzq4FnMgbPriWLYB9Trj8uHAHLQ6VKqCEkWjTYoIPFi7PEQ6xF65mxfk61UBdZyI4fpvwpypnPg3nT2LFwzAD7K24eEciUGIIrDuqHFQ4XXEmPbwKDdWCq+sNVcnNMdlhzGN/6XBSBpKAELS2YOOCaXHfbt/gludCKyTatp0hwQeYSBJs0dbLpqRL1uFDnobsFfHvgh/K5oWEouJKMasm2PXwNB3TwVYyPs049slGWN6Iadojh04Kagf0E44oBmZskgvHcG0IDkflKJ4qj5ma6tt4L01raRXSrjKKeY5wlfzevtCjf/D9SE5qRII7D3t0pTWBv9cl0E= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:VE1PR08MB4974.eurprd08.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230031)(921011)(38350700005);DIR:OUT;SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?q?GqWNqurMkfRfFhGizlH0qgs7P74B?= =?utf-8?q?k9cQC79rGMIOg0HJ0R/6a7tqDKtcZCDQ1N3WQJdyFpaVh7o3TciPSGXRYWr18nmLg?= =?utf-8?q?skkdi2wJSlYLvVyO6SrfVh3OFVqNwfA5nR/c3LJhiSSS7RD5MAF28PwO2ziddLxOf?= =?utf-8?q?cFrl7rzaccsWIA0Vk3Sqeg8tk1QhaSH29ThL2F4lVd2zGHiKngfpNSFTKdaVGYm6T?= =?utf-8?q?GFy1IcbCvzqQBy+0TszIoBpJgcksVQ29j3sYGA/pLL4nBdUU9XuJ/bm5qI7fs2MGl?= =?utf-8?q?SzlRQcUy6W0F5U/SJDs8pPPp8SkjB46pLj3/H5Osl132Ov2Gum1ZnXfoRihP8ysbg?= =?utf-8?q?3xzaSMn4bAt8eksKrOm41LTwP/93UTQMu5IpXtSzEMAKxS6zbiTPkU9XKk4+hikhK?= =?utf-8?q?AnndLMUUqjx+dAgfcEzwz22tRoU8WZQFWEEwEmQFNtJzpRde4GTBiRYYuogvuQ/tH?= =?utf-8?q?HEBlRDKs5aYsas7KOyNA7rW9HLnddkeK99HZMl3hcYh+Pp6lntYDS7LMERgTM5a4Y?= =?utf-8?q?CVDq0BIxaCfGDOzubXpXtRNNb1+DDiZbSQeL4gje7BYcS0uDg5V2Ju3kpkVpx2JPB?= =?utf-8?q?cPLNWNEwg+i7vAOkm5aEj/elVvNy2KYXBWhpNdIbQUYpIbcOwTeJ7UkyenE6YWpP7?= =?utf-8?q?Tm+YedQl2CdsIykPbriTD4cmyyS6LC+rI6/b8Znr19cdGJFk8gjpUeVkA3A/URgbe?= =?utf-8?q?Z3ZYhLPK5WkrNOB2CtgL4P73seEvb6Xjc6yaGEYAGHwQMQ8/sSadPZIqXgsPxgPpl?= =?utf-8?q?1sol+jltrE1CQityr9KNIE50vzR0iikaV4ICTLc96mngtoR7iTOZWYRMAw2/zCQCR?= =?utf-8?q?xkkpBKRxA2spVIQu9PoAvtc5zwIQncKa8OVSBFs02FhXo88fgFK9PXN46bo4atOXE?= =?utf-8?q?aNXAqT7/f0gdmUL+CaM9ToHCdt2S8fZps1JmfDwRuK6NrCwImG94AHBVgV87nDajn?= =?utf-8?q?xJGLzkeoBPIVCxddhNc17uz7363nZnGBRf4XEtysJYBPtJxWz5QYL3lUjiMZv4tD9?= =?utf-8?q?hWWAmULAlbk6TYLZBzyirnWmhIIl3YRzjHsmT30fe8oLLZMZG5O5PH7JSq8JEZdOP?= =?utf-8?q?/jc0C0xI+4NGaSJmx9eQ4UwO+VC20o2RHfMoWGtQBt0oWZrqRipJ6A3k0SH0Xr/h+?= =?utf-8?q?iNGOQBYmllDQYho8G/VfuW7hE1Bjn+y6VGVB2z8OCIa/T/4YMWWMRiBArePG3kcHE?= =?utf-8?q?HYlVzd1gyyScqUci3IVsg3OXQGw6C2DoN1ZkSqc2ZZmusqFocxHiEYkJ/mmO4qZBG?= =?utf-8?q?TkMd+Ib2/+AY2ce8q9gIfQbCJ0jmQSuv4sTdUSCAFHUb2beiZT4MOxgfxKwVljOwV?= =?utf-8?q?Iy2aob33SriakPG8eIMk4fLR9JnS8zcZNdRPKoQJDtk0txj33Ov5V3uFQj+ZuvdhG?= =?utf-8?q?GCgCmf3Gk1xE6nYAHY551QM6xWp5qC3B8jVkPvDXaWyucNImaXLuMjye41CAeGLt0?= =?utf-8?q?W3Ki60rLuVk8VDDHi94EuuoQAnIdd4N3+wTQbdvn6kMr9bG2o77JnBckVnE5ITIlH?= =?utf-8?q?c9i4nqBVBm/FeYwBUNUExikS8XEd+URY09o2AR/1T8RL3SplJ/aKkOo=3D?= X-OriginatorOrg: wolfvision.net X-MS-Exchange-CrossTenant-Network-Message-Id: 07085d1b-f809-472b-bcb0-08dc3864656f X-MS-Exchange-CrossTenant-AuthSource: VE1PR08MB4974.eurprd08.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 28 Feb 2024 13:51:43.1062 (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: oOxr4soj1bARsuWJ1vmKOO8S3DxJoXtun0O1lD77L6D5cTE32NIcZF57V6NBM1NVemtI1jJTqnaI9Bia8ftJntYiG5Bd7x/18hFLGj2IWiU= X-MS-Exchange-Transport-CrossTenantHeadersStamped: GV1PR08MB10523 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240228_055157_212179_4D9CC80B X-CRM114-Status: GOOD ( 19.35 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org 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. The 'always_powered_in_supend' attribute is only available for hub devices, keeping the driver's default behavior for non-hub devices (keep on in suspend). Signed-off-by: Javier Carrasco --- drivers/usb/misc/onboard_usb_dev.c | 25 +++++++++++++++++++++++-- drivers/usb/misc/onboard_usb_dev.h | 10 ++++++++++ 2 files changed, 33 insertions(+), 2 deletions(-) diff --git a/drivers/usb/misc/onboard_usb_dev.c b/drivers/usb/misc/onboard_usb_dev.c index e1779bd2d126..df0ed172c7ec 100644 --- a/drivers/usb/misc/onboard_usb_dev.c +++ b/drivers/usb/misc/onboard_usb_dev.c @@ -132,7 +132,8 @@ static int __maybe_unused onboard_dev_suspend(struct device *dev) struct usbdev_node *node; bool power_off = true; - if (onboard_dev->always_powered_in_suspend) + if (onboard_dev->always_powered_in_suspend && + !onboard_dev->pdata->is_hub) return 0; mutex_lock(&onboard_dev->lock); @@ -262,7 +263,27 @@ static struct attribute *onboard_dev_attrs[] = { &dev_attr_always_powered_in_suspend.attr, NULL, }; -ATTRIBUTE_GROUPS(onboard_dev); + +static umode_t onboard_dev_attrs_are_visible(struct kobject *kobj, + struct attribute *attr, + int n) +{ + struct device *dev = kobj_to_dev(kobj); + struct onboard_dev *onboard_dev = dev_get_drvdata(dev); + + if (attr == &dev_attr_always_powered_in_suspend.attr && + !onboard_dev->pdata->is_hub) + return 0; + + return attr->mode; +} + +static const struct attribute_group onboard_dev_group = { + .is_visible = onboard_dev_attrs_are_visible, + .attrs = onboard_dev_attrs, +}; +__ATTRIBUTE_GROUPS(onboard_dev); + static void onboard_dev_attach_usb_driver(struct work_struct *work) { diff --git a/drivers/usb/misc/onboard_usb_dev.h b/drivers/usb/misc/onboard_usb_dev.h index 470736483cdf..106480ce72b5 100644 --- a/drivers/usb/misc/onboard_usb_dev.h +++ b/drivers/usb/misc/onboard_usb_dev.h @@ -12,60 +12,70 @@ struct onboard_dev_pdata { unsigned long reset_us; /* reset pulse width in us */ unsigned int num_supplies; /* number of supplies */ const char * const supply_names[MAX_SUPPLIES]; /* use the real names */ + bool is_hub; /* true if the device is a HUB */ }; static const struct onboard_dev_pdata microchip_usb424_data = { .reset_us = 1, .num_supplies = 1, .supply_names = { "vdd" }, + .is_hub = true, }; static const struct onboard_dev_pdata microchip_usb5744_data = { .reset_us = 0, .num_supplies = 2, .supply_names = { "vdd", "vdd2" }, + .is_hub = true, }; static const struct onboard_dev_pdata realtek_rts5411_data = { .reset_us = 0, .num_supplies = 1, .supply_names = { "vdd" }, + .is_hub = true, }; static const struct onboard_dev_pdata ti_tusb8041_data = { .reset_us = 3000, .num_supplies = 1, .supply_names = { "vdd" }, + .is_hub = true, }; static const struct onboard_dev_pdata cypress_hx3_data = { .reset_us = 10000, .num_supplies = 2, .supply_names = { "vdd", "vdd2" }, + .is_hub = true, }; static const struct onboard_dev_pdata cypress_hx2vl_data = { .reset_us = 1, .num_supplies = 1, .supply_names = { "vdd" }, + .is_hub = true, }; static const struct onboard_dev_pdata genesys_gl850g_data = { .reset_us = 3, .num_supplies = 1, .supply_names = { "vdd" }, + .is_hub = true, }; static const struct onboard_dev_pdata genesys_gl852g_data = { .reset_us = 50, .num_supplies = 1, .supply_names = { "vdd" }, + .is_hub = true, }; static const struct onboard_dev_pdata vialab_vl817_data = { .reset_us = 10, .num_supplies = 1, .supply_names = { "vdd" }, + .is_hub = true, }; static const struct of_device_id onboard_dev_match[] = {