Received-SPF: pass (google.com: domain of linux-kernel+bounces-73140-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) client-ip=147.75.48.161;
From: Javier Carrasco <javier.carrasco@wolfvision.net>
Subject: [PATCH v4 0/8] usb: misc: onboard_hub: add support for XMOS
 XVF3500
Date: Tue, 20 Feb 2024 15:05:44 +0100
Message-Id: <20240220-onboard_xvf3500-v4-0-dc1617cc5dd4@wolfvision.net>
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
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>, 
 Helen Koike <helen.koike@collabora.com>, 
 Maarten Lankhorst <maarten.lankhorst@linux.intel.com>, 
 Maxime Ripard <mripard@kernel.org>, Thomas Zimmermann <tzimmermann@suse.de>, 
 David Airlie <airlied@gmail.com>, Daniel Vetter <daniel@ffwll.ch>, 
 Catalin Marinas <catalin.marinas@arm.com>, Will Deacon <will@kernel.org>, 
 Russell King <linux@armlinux.org.uk>
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 <javier.carrasco@wolfvision.net>, 
 Matthias Kaehlcke <matthias@kaehlcke.net>
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?Tm9Wd3ZaVEQ2RlMxODNHcFJEWml5UWxJTFNmSVR1dnc5blY0YnNOZ0w4eFo1?=
 =?utf-8?B?S25Ea0NHODZpTmQxVGF1clp0eVViMDEwTHREcG0zaStkdzFRZkpvUG9sdjJP?=
 =?utf-8?B?cEZ0a0FVcGcrdUZGbHByRGtzKzFkQ3U2MzFwTEdKU09Xc2dZZU5PNzdRaVgx?=
 =?utf-8?B?bEJsWW1JTlhhSjBnSDRVOE54T1VjK0JqMENZdE1tSmhQaUlsZE1vc0ZkUGYw?=
 =?utf-8?B?MEMvM3FuQ0NTeWpUb0VXNU9yL0FFOUJ2RVlETXpGNTBxS0dyRXIvVVI1Y0Vu?=
 =?utf-8?B?NUc1cEJrYm05SVlUVzZtVnpuZk9hYzVaQlRyKytOLzROVTJvaDZMSUZrRWtF?=
 =?utf-8?B?ci92MkJlejBoaG8rYWswWlk1K3ZuakVJcUorRmMwVmFVYW1jYkhRNFM3UUhP?=
 =?utf-8?B?Z3RQNGZwYXZSRUIvdDJ3ZHg5RWZTZ2VXL0RWV2tOdTl3emlzamRkSTllMlY2?=
 =?utf-8?B?dnB0bFVLMzRnNVdSZDZROGZtYjkvaXR2cWVtMFFVd1ZsRW8xOCtLa0VkNzB4?=
 =?utf-8?B?MkQ4RWlPdXN0emFlMzJQZ2gwekc0RDVhRFVSaUpoU2h6RUhHTWlJMGxuMjlu?=
 =?utf-8?B?azA0RmNYRFZGZGhlT3dsVlFmaUEySXRYdEhMbS83Mm03NXBydnVJUFd5Q3ln?=
 =?utf-8?B?Zm1hMjlteFRTUXNmeTltRTdJZjNvRXdhSTBMYzJua2V3eHpjMEpCQjJlU0ZY?=
 =?utf-8?B?S05zRHFhTVJPMDVZdGtOSkhEbCtVSHdtZTl5NTlFWVNiRzNpSXF4Uk9neHZV?=
 =?utf-8?B?bWJSWXU4OXVjaGZIYnhKMUVQOWYweDQ4UlVqMHNObGpiUnpWdEVyRUN2OXZG?=
 =?utf-8?B?dGIzRkYremkrR0ZuQ1pBVXhzRUtpRDFZeDdWSWFBRUdqSEpNbzZrV0x6aGhK?=
 =?utf-8?B?bHUxS010L0ZVZEsxVEhPZEd6WFoxVVpLL1pwNXNnL1lmUmRCejIyK3M0cVI2?=
 =?utf-8?B?NXg2K3IxWmQzNkZET0ZGNy9zdlp5Tlh4VnpHbkYvZlpXalhhR2ZYU05oWVY0?=
 =?utf-8?B?OHQ5NE9CejJWTk4xTkcvTHNnUUtmeHVlMXZOZU92SDlUbkEzSUZzUWhXc1ZY?=
 =?utf-8?B?T2JQQVJFQlZqdis0cThjRjZremM5Wkw5MVRRaXNubmVoMUpUblFBVlI3YWdS?=
 =?utf-8?B?MXUvUU1OR2EybWR1MzNVNHBJZGpHYkR5UTNLMG10azJGOFRxb05iMWo4Yzg5?=
 =?utf-8?B?NVpqV2NKaUFnU1hOOFRkbU1vQjB3b3pib015N3ZTankxY1AxKy9EQkhlMTR0?=
 =?utf-8?B?UEtlOGZzTy9TZkk3T3JaUHZqRjhuRjQzVU1abTF1OFRYSjZUQkNYSVMrZkZT?=
 =?utf-8?B?QUxreUdraE4xdEF6blhKdVNTaTdYcUQ4MVBDbm55aDJxT3libHYwR1c5WkJT?=
 =?utf-8?B?VE5zeEVuT2hweCsvbkJwNC8wUjNWRnJpQlliakhtZ3NYZm9XbVNVQ2hyMExK?=
 =?utf-8?B?L0R3YUoxYU5CTndybnd2azJ1aENIN2ZsZUtEYU1NSDVIK0FZNlZyVjY3TVVU?=
 =?utf-8?B?VWsrMFpUbUluaVFmWmVzMnVFeGZsTmFyOW5MblpZSTVyRmZ2WW1nQjFsblhW?=
 =?utf-8?B?Z3VPYXdnNjBUdGl6ZHM1RFFCcnNIcllEK2NKaUp5L0p2bmdRbG1XN01MODQx?=
 =?utf-8?B?M0kvOVlHTWRNTmMvaFE1M1hTUjhSOXJJZ2xTdFdjNXlYY3g1eXNEOHJFZW85?=
 =?utf-8?B?ODBaWUFNSmZpVFVSN2RBQlorWDRSQThwd2lDWWhvNjJoNEJzMUd5SW9lbzEy?=
 =?utf-8?B?dEUrQTdlRVNES2lRVElSclp4NDlOd0Q0QkJDSEpaZnlDVlg1M0kza3JjbjBm?=
 =?utf-8?B?RFZ6L0NxdWhMQUFWcUowTEVtRUJNUVVpdjNIRlRFVmpVU21OYllNalByeExE?=
 =?utf-8?B?UzhPd0xlaElmRmRJeEJsNXJSVnFhUXFNY3JBeG54ajhYNk1HYWJnTmxpTk5I?=
 =?utf-8?B?VUh1T2V1d1JMdU85WjFENS8xZGs0RHBLbm5iVy9iUElka1JOVFF3SU5yWmh5?=
 =?utf-8?B?NXdaR3M3aCtialc2RDFtb1h0T3N6cWdmaitQZm1iUW81TERUNDdPR2VIdWdy?=
 =?utf-8?B?S2RIZ3YzeTc2bzVYOFB0Q2pXQmlPK3VmZG8xbmhVdDVWRm9UUnhrV0FwSG9F?=
 =?utf-8?B?c3Z3WUYzRnNRWlFNUUs2NVJEeHZiT29YOVhzMGdUREJWODFaRFgzbytCZmtu?=
 =?utf-8?B?QWc9PQ==?=
X-OriginatorOrg: wolfvision.net
X-MS-Exchange-CrossTenant-Network-Message-Id: 6c85b2b0-7a9a-4bd1-dc0a-08dc321d0b74
X-MS-Exchange-CrossTenant-AuthSource: VE1PR08MB4974.eurprd08.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 20 Feb 2024 14:05:50.8779
 (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: a2/8WlNgNJSyRoP3nP0nlYyQPIWZqteXlf+oVpI2jVFMkvCxi6qIHNWR3CjMmpOjKhKtNnqAuZ4yT52uT8GVbRMnhiIUbi7ItIOC5/0Ptww=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PR3PR08MB5577

This series adds support for the XMOS XVF3500 VocalFusion Voice
Processor[1], a low-latency, 32-bit multicore controller for voice
processing.

The XVF3500 requires a specific power sequence, which consists of
enabling the regulators that control the 3V3 and 1V0 device supplies,
and a reset de-assertion after a delay of at least 100ns. Once in normal
operation, the XVF3500 registers itself as a regular USB device and no
device-specific management is required.

The power management provided by onboard_usb_hub is not specific for hubs
and any other USB device with the same power sequence could profit from
that driver, provided that the device does not have any specific
requirements beyond the power management. To account for non-hub devices,
the driver has been renamed and an extra flag has been added to identify
hubs and provide their specific functionality. Support for
device-specific power suply names has been added, keeping generic names
for backwards compatibility and as a fallback mechanism.

The references to onboard_usb_hub in the core and config files have been
updated as well.

The diff is way much bulkier than the actual code addition because of the
file renaming, so in order to ease reviews and catch hub-specific code
that might still affect non-hub devices, the complete renaming was moved
to a single commit.

The device binding has been added to sound/ because it is the subsystem
that covers its functionality (voice processing) during normal
operation. If it should reside under usb/ instead, it will be moved as
required.

This series has been tested with a Rockchip-based SoC and an XMOS
XVF3500-FB167-C.

[1] https://www.xmos.com/xvf3500/

To: Liam Girdwood <lgirdwood@gmail.com>
To: Mark Brown <broonie@kernel.org>
To: Rob Herring <robh+dt@kernel.org>
To: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
To: Conor Dooley <conor+dt@kernel.org>
To: Matthias Kaehlcke <mka@chromium.org>
To: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
To: Helen Koike <helen.koike@collabora.com>
To: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
To: Maxime Ripard <mripard@kernel.org>
To: Thomas Zimmermann <tzimmermann@suse.de>
To: David Airlie <airlied@gmail.com>
To: Daniel Vetter <daniel@ffwll.ch>
To: Catalin Marinas <catalin.marinas@arm.com>
To: Will Deacon <will@kernel.org>
To: Russell King <linux@armlinux.org.uk>
Cc: linux-sound@vger.kernel.org
Cc: devicetree@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Cc: linux-usb@vger.kernel.org
Cc: dri-devel@lists.freedesktop.org
Cc: linux-arm-kernel@lists.infradead.org
Signed-off-by: Javier Carrasco <javier.carrasco@wolfvision.net>

Changes in v4:
- General: use device supply names and generics as fallback.
- onbord_usb_dev.c: fix suspend callback for non-hub devices.
- onboard_usb_dev.c: fix typos.

- Link to v3: https://lore.kernel.org/r/20240206-onboard_xvf3500-v3-0-f85b04116688@wolfvision.net

Changes in v3:
- onboard_usb_hub: rename to onboard_usb_dev to include non-hub devices.
- onboard_hub_dev: add flag to identify hubs and provide their extra
  functionality.
- dt-bindings: add reference to usb-device.yaml and usb node in the
  example.
- dt-bindings: generic node name.
- Link to v2: https://lore.kernel.org/r/20240130-onboard_xvf3500-v1-0-51b5398406cb@wolfvision.net

Changes in v2:
- general: add support in onboard_usb_hub instead of using a dedicated
  driver.
- dt-bindings: use generic usb-device compatible ("usbVID,PID").
- Link to v1: https://lore.kernel.org/all/20240115-feature-xvf3500_driver-v1-0-ed9cfb48bb85@wolfvision.net/

---
Javier Carrasco (8):
      usb: misc: onboard_hub: rename to onboard_dev
      usb: misc: onboard_dev: add support for non-hub devices
      drm: ci: arm64.config: update ONBOARD_USB_HUB to ONBOARD_USB_DEV
      arm64: defconfig: update ONBOARD_USB_HUB to ONBOARD_USB_DEV
      ARM: multi_v7_defconfig: update ONBOARD_USB_HUB name
      usb: misc: onboard_dev: use device supply names
      ASoC: dt-bindings: xmos,xvf3500: add XMOS XVF3500 voice processor
      usb: misc: onboard_hub: add support for XMOS XVF3500

 ...-usb-hub => sysfs-bus-platform-onboard-usb-dev} |   4 +-
 .../devicetree/bindings/sound/xmos,xvf3500.yaml    |  63 +++
 MAINTAINERS                                        |   4 +-
 arch/arm/configs/multi_v7_defconfig                |   2 +-
 arch/arm64/configs/defconfig                       |   2 +-
 drivers/gpu/drm/ci/arm64.config                    |   4 +-
 drivers/usb/core/Makefile                          |   4 +-
 drivers/usb/core/hub.c                             |   8 +-
 drivers/usb/core/hub.h                             |   2 +-
 drivers/usb/misc/Kconfig                           |  16 +-
 drivers/usb/misc/Makefile                          |   2 +-
 drivers/usb/misc/onboard_usb_dev.c                 | 525 +++++++++++++++++++++
 .../misc/{onboard_usb_hub.h => onboard_usb_dev.h}  |  69 ++-
 ...ard_usb_hub_pdevs.c => onboard_usb_dev_pdevs.c} |  47 +-
 drivers/usb/misc/onboard_usb_hub.c                 | 501 --------------------
 include/linux/usb/onboard_dev.h                    |  18 +
 include/linux/usb/onboard_hub.h                    |  18 -
 17 files changed, 709 insertions(+), 580 deletions(-)
---
base-commit: 41bccc98fb7931d63d03f326a746ac4d429c1dd3
change-id: 20240130-onboard_xvf3500-6c0e78d11a1b

Best regards,
-- 
Javier Carrasco <javier.carrasco@wolfvision.net>