From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751825AbdBFLzx (ORCPT ); Mon, 6 Feb 2017 06:55:53 -0500 Received: from mail.linuxfoundation.org ([140.211.169.12]:46916 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751001AbdBFLzw (ORCPT ); Mon, 6 Feb 2017 06:55:52 -0500 Date: Mon, 6 Feb 2017 12:55:52 +0100 From: Greg KH To: Richard Leitner Cc: linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, robh+dt@kernel.org, mark.rutland@arm.com, dev@g0hl1n.net Subject: Re: [PATCH v2] usb: misc: add USB251xB/xBi Hi-Speed Hub Controller Driver Message-ID: <20170206115552.GA31808@kroah.com> References: <1486119324-9362-1-git-send-email-richard.leitner@skidata.com> <20170205074216.GA314@kroah.com> <686ee231-57b2-036f-f97a-dad508e94e39@skidata.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <686ee231-57b2-036f-f97a-dad508e94e39@skidata.com> User-Agent: Mutt/1.7.2 (2016-11-26) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Feb 06, 2017 at 12:19:40PM +0100, Richard Leitner wrote: > On 02/05/2017 08:42 AM, Greg KH wrote: > > On Fri, Feb 03, 2017 at 11:55:24AM +0100, Richard Leitner wrote: > >> +/** > >> + * ascii2utf16le() - Helper routine for producing UTF-16LE string descriptors > >> + * @s: Null-terminated ASCII (actually ISO-8859-1) string > >> + * @buf: Buffer for UTF-16LE string > >> + * @len: Length (in bytes; may be odd) of UTF-16LE buffer. > >> + * > >> + * Return: The number of bytes filled in: 2*strlen(s) or @len, whichever is less > >> + * > >> + * Note: > >> + * The UTF-16LE USB String descriptors can contain at most 31 characters (as > >> + * specified in the datasheet); input strings longer than that are truncated. > >> + * > >> + * Based on ascii2desc from drivers/usb/core/hcd.c > >> + */ > > > > Don't we have a kernel function for this already? If we need to export > > ascii2desc() from the USB core, we can do that, or better yet, move both > > of them to a string library function in the core part of the kernel. We > > shouldn't have to duplicate this type of thin in an individual driver. > > Ok. So I'll move the ascii2utf16le function to lib/string.c (?) and call > it from ascii2desc in drivers/usb/core/hcd.c (due to the fact ascii2desc > also prepends 2 bytes) and the USB251xB driver? Would that be OK? That sounds good to me. > >> --- /dev/null > >> +++ b/include/linux/platform_data/usb251xb.h > >> @@ -0,0 +1,33 @@ > > > > Do you need a platform data structure here? Shouldn't we be only using > > DT now? > > I don't need the platform data support at all. I just added it because > it is also available in the USB3503 driver. So if it's OK for you I'd be > glad to remove it. Great, please remove it! :) thanks, greg k-h