From: Matthias Kaehlcke <mka@chromium.org> To: Javier Carrasco <javier.carrasco@wolfvision.net> Cc: 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>, 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>, 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 Subject: Re: [PATCH v5 8/8] usb: misc: onboard_hub: add support for XMOS XVF3500 Date: Wed, 28 Feb 2024 19:22:18 +0000 [thread overview] Message-ID: <Zd-H6hVPv1poXoa1@google.com> (raw) In-Reply-To: <20240228-onboard_xvf3500-v5-8-76b805fd3fe6@wolfvision.net> On Wed, Feb 28, 2024 at 02:51:35PM +0100, Javier Carrasco wrote: > The XMOS XVF3500 VocalFusion Voice Processor[1] is a low-latency, 32-bit > multicore controller for voice processing. > > This device 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. Such power > sequence is already supported by the onboard_hub driver, and it can be > reused for non-hub USB devices as well. Please update the commit message, the onboard_hub driver no longer exists as such with the other patches of this series. > Once in normal operation, the XVF3500 registers itself as a USB device, > and it does not require any device-specific operations in the driver. > > [1] https://www.xmos.com/xvf3500/ > > Signed-off-by: Javier Carrasco <javier.carrasco@wolfvision.net> Acked-by: Matthias Kaehlcke <mka@chromium.org> > --- > drivers/usb/misc/onboard_usb_dev.c | 2 ++ > drivers/usb/misc/onboard_usb_dev.h | 8 ++++++++ > 2 files changed, 10 insertions(+) > > diff --git a/drivers/usb/misc/onboard_usb_dev.c b/drivers/usb/misc/onboard_usb_dev.c > index df0ed172c7ec..50f84c5278a2 100644 > --- a/drivers/usb/misc/onboard_usb_dev.c > +++ b/drivers/usb/misc/onboard_usb_dev.c > @@ -405,6 +405,7 @@ static struct platform_driver onboard_dev_driver = { > #define VENDOR_ID_REALTEK 0x0bda > #define VENDOR_ID_TI 0x0451 > #define VENDOR_ID_VIA 0x2109 > +#define VENDOR_ID_XMOS 0x20B1 > > /* > * Returns the onboard_dev platform device that is associated with the USB > @@ -497,6 +498,7 @@ static const struct usb_device_id onboard_dev_id_table[] = { > { USB_DEVICE(VENDOR_ID_TI, 0x8142) }, /* TI USB8041 2.0 */ > { USB_DEVICE(VENDOR_ID_VIA, 0x0817) }, /* VIA VL817 3.1 */ > { USB_DEVICE(VENDOR_ID_VIA, 0x2817) }, /* VIA VL817 2.0 */ > + { USB_DEVICE(VENDOR_ID_XMOS, 0x0013) }, /* XVF3500 */ nit: be a bit more specific? e.g. XVF3500 Voice Processor, or XMOS XVF3500 The other entries were implicitly hubs since this was the 'onboard_hub' driver. It wouldn't be a bad idea to add 'hub' to these entries in the patch that 'renames' the driver.
WARNING: multiple messages have this Message-ID (diff)
From: Matthias Kaehlcke <mka@chromium.org> To: Javier Carrasco <javier.carrasco@wolfvision.net> Cc: 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>, 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>, 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 Subject: Re: [PATCH v5 8/8] usb: misc: onboard_hub: add support for XMOS XVF3500 Date: Wed, 28 Feb 2024 19:22:18 +0000 [thread overview] Message-ID: <Zd-H6hVPv1poXoa1@google.com> (raw) In-Reply-To: <20240228-onboard_xvf3500-v5-8-76b805fd3fe6@wolfvision.net> On Wed, Feb 28, 2024 at 02:51:35PM +0100, Javier Carrasco wrote: > The XMOS XVF3500 VocalFusion Voice Processor[1] is a low-latency, 32-bit > multicore controller for voice processing. > > This device 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. Such power > sequence is already supported by the onboard_hub driver, and it can be > reused for non-hub USB devices as well. Please update the commit message, the onboard_hub driver no longer exists as such with the other patches of this series. > Once in normal operation, the XVF3500 registers itself as a USB device, > and it does not require any device-specific operations in the driver. > > [1] https://www.xmos.com/xvf3500/ > > Signed-off-by: Javier Carrasco <javier.carrasco@wolfvision.net> Acked-by: Matthias Kaehlcke <mka@chromium.org> > --- > drivers/usb/misc/onboard_usb_dev.c | 2 ++ > drivers/usb/misc/onboard_usb_dev.h | 8 ++++++++ > 2 files changed, 10 insertions(+) > > diff --git a/drivers/usb/misc/onboard_usb_dev.c b/drivers/usb/misc/onboard_usb_dev.c > index df0ed172c7ec..50f84c5278a2 100644 > --- a/drivers/usb/misc/onboard_usb_dev.c > +++ b/drivers/usb/misc/onboard_usb_dev.c > @@ -405,6 +405,7 @@ static struct platform_driver onboard_dev_driver = { > #define VENDOR_ID_REALTEK 0x0bda > #define VENDOR_ID_TI 0x0451 > #define VENDOR_ID_VIA 0x2109 > +#define VENDOR_ID_XMOS 0x20B1 > > /* > * Returns the onboard_dev platform device that is associated with the USB > @@ -497,6 +498,7 @@ static const struct usb_device_id onboard_dev_id_table[] = { > { USB_DEVICE(VENDOR_ID_TI, 0x8142) }, /* TI USB8041 2.0 */ > { USB_DEVICE(VENDOR_ID_VIA, 0x0817) }, /* VIA VL817 3.1 */ > { USB_DEVICE(VENDOR_ID_VIA, 0x2817) }, /* VIA VL817 2.0 */ > + { USB_DEVICE(VENDOR_ID_XMOS, 0x0013) }, /* XVF3500 */ nit: be a bit more specific? e.g. XVF3500 Voice Processor, or XMOS XVF3500 The other entries were implicitly hubs since this was the 'onboard_hub' driver. It wouldn't be a bad idea to add 'hub' to these entries in the patch that 'renames' the driver. _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2024-02-28 19:22 UTC|newest] Thread overview: 45+ messages / expand[flat|nested] mbox.gz Atom feed top 2024-02-28 13:51 [PATCH v5 0/8] usb: misc: onboard_hub: add support for XMOS XVF3500 Javier Carrasco 2024-02-28 13:51 ` Javier Carrasco 2024-02-28 13:51 ` [PATCH v5 1/8] usb: misc: onboard_hub: use device supply names Javier Carrasco 2024-02-28 13:51 ` Javier Carrasco 2024-02-28 15:37 ` Matthias Kaehlcke 2024-02-28 15:37 ` Matthias Kaehlcke 2024-02-28 16:02 ` Javier Carrasco 2024-02-28 16:02 ` Javier Carrasco 2024-02-28 16:13 ` Matthias Kaehlcke 2024-02-28 16:13 ` Matthias Kaehlcke 2024-02-28 13:51 ` [PATCH v5 2/8] usb: misc: onboard_hub: rename to onboard_dev Javier Carrasco 2024-02-28 13:51 ` Javier Carrasco 2024-02-28 18:18 ` Matthias Kaehlcke 2024-02-28 18:18 ` Matthias Kaehlcke 2024-02-28 20:10 ` Javier Carrasco 2024-02-28 20:10 ` Javier Carrasco 2024-02-28 13:51 ` [PATCH v5 3/8] drm: ci: arm64.config: update ONBOARD_USB_HUB to ONBOARD_USB_DEV Javier Carrasco 2024-02-28 13:51 ` Javier Carrasco 2024-02-28 13:51 ` [PATCH v5 4/8] arm64: defconfig: " Javier Carrasco 2024-02-28 13:51 ` Javier Carrasco 2024-02-28 13:51 ` [PATCH v5 5/8] ARM: multi_v7_defconfig: update ONBOARD_USB_HUB to ONBOAD_USB_DEV Javier Carrasco 2024-02-28 13:51 ` Javier Carrasco 2024-02-28 13:51 ` [PATCH v5 6/8] usb: misc: onboard_dev: add support for non-hub devices Javier Carrasco 2024-02-28 13:51 ` Javier Carrasco 2024-02-28 18:10 ` Matthias Kaehlcke 2024-02-28 18:10 ` Matthias Kaehlcke 2024-02-28 20:21 ` Javier Carrasco 2024-02-28 20:21 ` Javier Carrasco 2024-02-28 20:41 ` Matthias Kaehlcke 2024-02-28 20:41 ` Matthias Kaehlcke 2024-02-28 20:50 ` Javier Carrasco 2024-02-28 20:50 ` Javier Carrasco 2024-02-28 21:34 ` Matthias Kaehlcke 2024-02-28 21:34 ` Matthias Kaehlcke 2024-02-29 6:38 ` Javier Carrasco 2024-02-29 6:38 ` Javier Carrasco 2024-02-28 13:51 ` [PATCH v5 7/8] ASoC: dt-bindings: xmos,xvf3500: add XMOS XVF3500 voice processor Javier Carrasco 2024-02-28 13:51 ` Javier Carrasco 2024-02-28 14:04 ` Mark Brown 2024-02-28 14:04 ` Mark Brown 2024-02-28 14:04 ` [PATCH v5 7/8] ASoC: dt-bindings: xmos, xvf3500: " Mark Brown 2024-02-28 13:51 ` [PATCH v5 8/8] usb: misc: onboard_hub: add support for XMOS XVF3500 Javier Carrasco 2024-02-28 13:51 ` Javier Carrasco 2024-02-28 19:22 ` Matthias Kaehlcke [this message] 2024-02-28 19:22 ` Matthias Kaehlcke
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=Zd-H6hVPv1poXoa1@google.com \ --to=mka@chromium.org \ --cc=airlied@gmail.com \ --cc=broonie@kernel.org \ --cc=catalin.marinas@arm.com \ --cc=conor+dt@kernel.org \ --cc=daniel@ffwll.ch \ --cc=devicetree@vger.kernel.org \ --cc=dri-devel@lists.freedesktop.org \ --cc=gregkh@linuxfoundation.org \ --cc=helen.koike@collabora.com \ --cc=javier.carrasco@wolfvision.net \ --cc=krzysztof.kozlowski+dt@linaro.org \ --cc=lgirdwood@gmail.com \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-sound@vger.kernel.org \ --cc=linux-usb@vger.kernel.org \ --cc=linux@armlinux.org.uk \ --cc=maarten.lankhorst@linux.intel.com \ --cc=mripard@kernel.org \ --cc=robh+dt@kernel.org \ --cc=tzimmermann@suse.de \ --cc=will@kernel.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.