From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934200AbcAZLBf (ORCPT ); Tue, 26 Jan 2016 06:01:35 -0500 Received: from mga14.intel.com ([192.55.52.115]:56260 "EHLO mga14.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756886AbcAZLB0 (ORCPT ); Tue, 26 Jan 2016 06:01:26 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.22,349,1449561600"; d="scan'208";a="641567330" Subject: Re: [PATCH v6 10/12] usb: dbc: add handshake between debug target and host To: Julia Lawall References: <201601261754.HqYDuGWL%fengguang.wu@intel.com> Cc: kbuild-all@01.org, Greg Kroah-Hartman , linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org From: Lu Baolu Message-ID: <56A75204.8090108@linux.intel.com> Date: Tue, 26 Jan 2016 19:01:24 +0800 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.5.1 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 01/26/2016 06:40 PM, Julia Lawall wrote: > Please check. As far as I can see, the call to early_xdbc_read ends up at > xdbc_bulk_transfer, which return negative error codes on failure. Good catch. Thank you. I will correct it. > > julia > > On Tue, 26 Jan 2016, kbuild test robot wrote: > >> CC: kbuild-all@01.org >> In-Reply-To: <1453781665-4714-11-git-send-email-baolu.lu@linux.intel.com> >> TO: Lu Baolu >> CC: Greg Kroah-Hartman >> CC: linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org, Lu Baolu >> >> Hi Lu, >> >> [auto build test WARNING on usb/usb-testing] >> [also build test WARNING on v4.5-rc1 next-20160125] >> [if your patch is applied to the wrong git tree, please drop us a note to help improving the system] >> >> url: https://github.com/0day-ci/linux/commits/Lu-Baolu/usb-early-add-support-for-early-printk-through-USB3-debug-port/20160126-122049 >> base: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/usb.git usb-testing >> :::::: branch date: 5 hours ago >> :::::: commit date: 5 hours ago >> >>>> drivers/usb/early/xhci-dbc.c:917:6-10: WARNING: Unsigned expression compared with zero: size > 0 >> git remote add linux-review https://github.com/0day-ci/linux >> git remote update linux-review >> git checkout cdae956781925bbc269faa7f40c931a2825be11a >> vim +917 drivers/usb/early/xhci-dbc.c >> >> 5319db53 Lu Baolu 2016-01-26 901 >> 5319db53 Lu Baolu 2016-01-26 902 /* hand over the owner of host from BIOS */ >> 5319db53 Lu Baolu 2016-01-26 903 xdbc_bios_handoff(); >> 5319db53 Lu Baolu 2016-01-26 904 >> 5319db53 Lu Baolu 2016-01-26 905 ret = xdbc_setup(); >> 5319db53 Lu Baolu 2016-01-26 906 if (ret < 0) { >> 5319db53 Lu Baolu 2016-01-26 907 pr_notice("failed to setup xHCI DbC connection\n"); >> 5319db53 Lu Baolu 2016-01-26 908 xdbcp->xhci_base = NULL; >> 5319db53 Lu Baolu 2016-01-26 909 xdbcp->xdbc_reg = NULL; >> 12cd5775 Lu Baolu 2016-01-26 910 xdbc_dump_debug_buffer(); >> 5319db53 Lu Baolu 2016-01-26 911 return ret; >> 5319db53 Lu Baolu 2016-01-26 912 } >> 5319db53 Lu Baolu 2016-01-26 913 >> cdae9567 Lu Baolu 2016-01-26 914 while (retry > 0) { >> cdae9567 Lu Baolu 2016-01-26 915 early_xdbc_write(NULL, ping, strlen(ping)); >> cdae9567 Lu Baolu 2016-01-26 916 size = early_xdbc_read(NULL, pong, 64); >> cdae9567 Lu Baolu 2016-01-26 @917 if (size > 0) { >> cdae9567 Lu Baolu 2016-01-26 918 xdbc_trace("%s: pong message: %s\n", __func__, pong); >> cdae9567 Lu Baolu 2016-01-26 919 if (pong[0] == 'Y' || pong[0] == 'y') >> cdae9567 Lu Baolu 2016-01-26 920 break; >> cdae9567 Lu Baolu 2016-01-26 921 } else { >> cdae9567 Lu Baolu 2016-01-26 922 xdbc_trace("%s: pong message error %d\n", >> cdae9567 Lu Baolu 2016-01-26 923 __func__, size); >> cdae9567 Lu Baolu 2016-01-26 924 } >> cdae9567 Lu Baolu 2016-01-26 925 >> >> --- >> 0-DAY kernel test infrastructure Open Source Technology Center >> https://lists.01.org/pipermail/kbuild-all Intel Corporation >>