Received: by 2002:ac0:bc90:0:0:0:0:0 with SMTP id a16csp1203834img; Tue, 19 Mar 2019 02:42:28 -0700 (PDT) X-Google-Smtp-Source: APXvYqx/JUbBKXfFr0goIPJRguBC2HcfJmspKLgkcdbvIMtOp3jIS1geGVOAKno9NYkvAeX7UGIB X-Received: by 2002:a63:5349:: with SMTP id t9mr957768pgl.262.1552988548317; Tue, 19 Mar 2019 02:42:28 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1552988548; cv=none; d=google.com; s=arc-20160816; b=t8ucuhzTbForNJ5rqS1/o+iAxlB9ijVQqxhsore3E0kNVitp8xO6Uqp2Q3kRfYtrcd lg7SjGmH8Tujp/0OFzg7AF1jVgBBRmX1utvYh/TZZfieBEmnWSTUnzKiKkKpy3tWgpTr ZalVNcAvauXNkBp8y8NwKPcnHYChZ0GsuSBSTRXKAWs3uLOtRnnNxAjqg9eLD74DHkXf ZvazFYIrvKnwwKuElwoQACJja6dmIVrr21g9q86UtXnqbRObnXPrpQiC+HuVbs/LU1NA C99pR/zIQFMUDfU32n4Zq7rbsxsrtbMAEHdNRmNZvM01m3RTiTIGPlMkvRE1cwvmA0Dp 1vQQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding :content-language:in-reply-to:mime-version:user-agent:date :message-id:from:references:cc:to:subject:ironport-sdr :dkim-signature; bh=zr2EgeiLdWo8n1CSmCAhFBzgK/m8e4zcy2RThU8WN+g=; b=0aNDRTB8ETEYTZ2whYwDgxSGcVtE1ZgTEMZYLZD4cXl6mbiNoe80j28Kvk67WMzWvL pvxyHywsYGjL1dYPB8DEnu+6U7a/HlwXRIeQm1wt6Pf7WHooBPBy0LZwcQcGOh+vnqlA YDPc9+Tb4m/PGjAVzkmUmQp0sZSMmurt9T1G/uXh2afpO6ri+NZLIErIfvu4xTCEjCHq bkSbC9n4geHzItSTPtFO4IeNfuHqZvdA+tfFtbBas+rA0Kop3Si5cpYqm3qQySixlDnO LvJ2droYlSmfDJ1duzB2N5p7nfZVBf4OocpwRUI2VJbB1SJWQAiCgKS6+/RgC1sUmPwO Psew== ARC-Authentication-Results: i=1; mx.google.com; dkim=fail header.i=@skidata.com header.s=selector1 header.b="JibBT9P/"; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id j71si11902397pfc.280.2019.03.19.02.42.13; Tue, 19 Mar 2019 02:42:28 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=fail header.i=@skidata.com header.s=selector1 header.b="JibBT9P/"; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727599AbfCSJk5 (ORCPT + 99 others); Tue, 19 Mar 2019 05:40:57 -0400 Received: from mail2.skidata.com ([91.230.2.91]:11344 "EHLO mail2.skidata.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725862AbfCSJk5 (ORCPT ); Tue, 19 Mar 2019 05:40:57 -0400 X-Greylist: delayed 431 seconds by postgrey-1.27 at vger.kernel.org; Tue, 19 Mar 2019 05:40:56 EDT DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=skidata.com; i=@skidata.com; q=dns/txt; s=selector1; t=1552988456; x=1584524456; h=subject:to:cc:references:from:message-id:date: mime-version:in-reply-to:content-transfer-encoding; bh=pH1iyEMKmz7n5W/eUIYRFrO/gL7CW34g/2pLR7o03TU=; b=JibBT9P/0Qk0EjNBRn2I8pSv5yasSmwrCOuNKyzSpCzloAAqIVdz94yS SsnZafY9QNZ3Ot126Af4sEQfIXCkrH4H8Fj+I9R++G3pdhNMC5h9Gp4gr wJqvDk+5PMM4xQPToTQmPXUFQZ31YcNbUap5XboINKP+3BUbvLRps0u7x C1LSFr34ecqHAdqHawRF2nOp1r5mYac7tDcACwHRTPX4ZBLX0lsRRa+/V wsbiTUTM63pT5cEESMlC8NiIhYYC8yLmB7v8zhKwXhOZT4cTIw8UIKdZb +pc9zciFwMX5ggoFtEogf1BaEL6c4ohdtH8Q5wtu3ihAicPhH/CMlLu1z A==; IronPort-SDR: YBFGLWCn7zelsJjs+IGaQWVWySPCJczBPq8KdUDCtDjbXvUv4ltTTdmNFa9fPDWzB/ARz9x/xo vSprZ2vyaHBOsULn0/cC+ZGd+4lSdGTo74bXIq0x/YbGm1AWA02y3vA4GR2IjOdQVjPFasnne7 Ci7Y555MxOk4Q/tzPwd9qLffrl3+WhRpWPequWAtC8s2YSbAdeMjL22FFfyC0czW/HSAaoWbiG LTGzsIO+CHlbJ1nDZIRBkonYA0W3kShVn6X2Vuaz49uw28Cihvf6Qiwn3DedIQ+HFrndYhyGiT 1T4= X-IronPort-AV: E=Sophos;i="5.58,497,1544482800"; d="scan'208";a="2041749" Subject: Re: [PATCH] usb: usb251xb: Remove unnecessary comparison of unsigned integer with >= 0 To: Marco Felsch CC: "Gustavo A. R. Silva" , Greg Kroah-Hartman , , References: <20190318145024.GA572@embeddedor> <20190318153425.rxx3fwq2hi6tfocr@pengutronix.de> <20190318164811.quofap3b5mjlj5tc@pengutronix.de> From: Richard Leitner Message-ID: Date: Tue, 19 Mar 2019 10:33:43 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.5.1 MIME-Version: 1.0 In-Reply-To: <20190318164811.quofap3b5mjlj5tc@pengutronix.de> Content-Type: text/plain; charset="utf-8"; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit X-Originating-IP: [192.168.111.22] X-ClientProxiedBy: sdex4srv.skidata.net (192.168.111.82) To sdex5srv.skidata.net (192.168.111.83) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 18/03/2019 17:48, Marco Felsch wrote: > On 19-03-18 16:46, Richard Leitner wrote: >> A small note on further improvement from my side: >> >> According the datasheet, when you set swap-dx-lanes to 0 (-> setting port to >> 0 -> setting bit 0 of PRTSP) " the upstream port DP/DM is swapped". > > Yes I know that's not intuitve and may confuses the users. > >> IMHO this is not straight-forward, so maybe we want to additionally >> >> a) mention it somewhere in the documentation > > IMHO this is the correct place. > >> b) check on port >= 1 > > I think this isn't doable since we shouldn't break the existing dt. ACK. @Marco/Gustavo: does one of you wants to provide a patch for this? > > Regards, > Marco > >> c) something else? >> >> Any opinions on that from your side? ... I'd prefer "a"... regards;Richard.L