From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 7C826C433EF for ; Mon, 11 Apr 2022 00:54:42 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S237000AbiDKA4u (ORCPT ); Sun, 10 Apr 2022 20:56:50 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:55448 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S237530AbiDKA4t (ORCPT ); Sun, 10 Apr 2022 20:56:49 -0400 Received: from ams.source.kernel.org (ams.source.kernel.org [145.40.68.75]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 1D0161572D; Sun, 10 Apr 2022 17:54:36 -0700 (PDT) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ams.source.kernel.org (Postfix) with ESMTPS id 63168B80ED5; Mon, 11 Apr 2022 00:54:35 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 9EF8EC385A4; Mon, 11 Apr 2022 00:54:32 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1649638474; bh=rHmqtbKujz1oumBfQyH6US6UQG40oo7hEHJvzLUxX4w=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=Qqw1DzrX8HDF4Z8nDWHUPix5K+JLU8m7wz3curLSEb8A6dPWFanpmUvRALJzJ4rwb eLrQPp195BXch5ljoCiN+7mTFo3itcQFSlsucNmTopPdeGElp/PIvPHaVwTvPG68ll sqAeCYSJ7QU6fZnLCNu8mTvVZoNbC3KeSS7uxTo4ZbuiXGSHbAnfVL4oJh2Fah/d+e dgBNCVgBUaQ/GukWqW8wA5lZoy2lsQ90OO4Tt0z9hhj++Bj3T1THztmdoz7uE1M2zS A3aQ39OwkZwksmEtjN8+wLlk8hZQ5hF2MSfJfS0LnkjHx9klG/OYA42qwdSGeb3hb3 Bs5zFMFf7ziXQ== Date: Mon, 11 Apr 2022 08:54:28 +0800 From: Shawn Guo To: Michael Walle Cc: Li Yang , Rob Herring , Krzysztof Kozlowski , linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] arm64: dts: ls1028a: default to OTG mode for USB Message-ID: <20220411005428.GV129381@dragon> References: <20220330113442.3402940-1-michael@walle.cc> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20220330113442.3402940-1-michael@walle.cc> Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org On Wed, Mar 30, 2022 at 01:34:42PM +0200, Michael Walle wrote: > At the moment, the dtsi will force the dr_mode to host. This is > problematic because it will always turn on the Vbus voltage regardless > if the port is host or device. This might lead to a "shortcut" between > the two USB endpoints because both might have their Vbus supplies > enabled. Therefore, the default should be "otg" for any ports which > aren't host only (from a SoC point of view) and have a user of the dtsi > file overwrite that explicitly. > > Move the 'dr_mode = "host";' into the board dts. Now that the dtsi > doesn't set the dr_mode anymore, we can also drop the 'dr_mode = "otg";' > in the board dts because that is the default value if dr_mode is not > set. > > Signed-off-by: Michael Walle Applied, thanks!