From mboxrd@z Thu Jan 1 00:00:00 1970 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756319AbeAHJhL (ORCPT + 1 other); Mon, 8 Jan 2018 04:37:11 -0500 Received: from mout.kundenserver.de ([212.227.126.134]:50847 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756302AbeAHJhJ (ORCPT ); Mon, 8 Jan 2018 04:37:09 -0500 Subject: Re: [PATCH] Revert "ARM: dts: bcm283x: Fix DTC warnings about missing phy-cells" To: Arnd Bergmann Cc: Eric Anholt , Olof Johansson , Florian Fainelli , DTML , Linux ARM , arm-soc , Linux Kernel Mailing List References: <1515332212-23593-1-git-send-email-stefan.wahren@i2se.com> <87incdjp9k.fsf@anholt.net> From: Stefan Wahren Message-ID: Date: Mon, 8 Jan 2018 10:36:42 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.5.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Content-Language: en-US X-Provags-ID: V03:K0:cphw18VAmfy9sygNdjGvJk44Kk1RXqkDWnk8w4kuHhsjsmQqofw xIJI66dTcPgKsdUYTlmCUeeK8BEM9ax+L8w57HuKBKM+J+J6OVtit64z6B6+K1D9VC2cIwv /E09PmmBJtftl+k/myDFBc0gFY0kYveiNpePzEyk8G/sviIYQzLCVs6/SNbipGNC59JOk8l /9n0ZHVEyagCosZObT/sQ== X-UI-Out-Filterresults: notjunk:1;V01:K0:82bu/6W0qGY=:O1r5I4luEMQAOdn932VL71 0/w5vo2DWTxh8/MqNNjtxLlGyqPERV2OSd3yvgcng47G9Rvr5/j7nJHbXLS++hzDb/B57IoV8 SbuXbxi2Jx5m1Jo+XXSeCRA1gXjdBPSrnuRt9CPhnhB0sRsyzwuzIGjJQ73WBJ0yH098N7a5T 8ctKrNoZd1Oon9xBXfez69QvQG0geWrh15oPMi03YrbSDy+wawcpo5ufBZTHpxD+sTrIg5OTp 1clxoH9euB57Y+t3NIs7Gv8o+VgV+6JP952BucVwwsVEMJdvkTW08/6KAixwsudTbj+xBoYL7 MpDRXFdSqVedBz8ImneMEEG2dtDUUxJuhAzDqxm2nahvx+Okdo3i90y/cLvvw+nBFVSxnV+Ej M9jDAG38RKor9Dm0cGhaHWaURPyf3YjT3Y5NXLwQBtmB1rthXqBjuHD+XuQw9zO+7ATOFmtc8 7iBN2D1a3c0u3kt2rGtg1JNuvggG5cFJwAigl6vKkbIVnb0TzZL6+mVGE5Eus3AW4M4Ep/ZgE 3Fxa8C7VSMynq6znZoTw4/BPK9L1CDtRxS47CKCSk+P9p7SHNaHjPl/0x0C1lcrIiTsXEVZti L9Xui9sR1CEgsKFAGx7bjlWGf6eXgsLF5J5dmCqFxU/ua6Z9ZDLRULAgi3Q0ysN7eKS+3xw2E UnThbyfPSPY2mYqrHSyj//ugMdZ4F4DVS3+ELOuaUFiHISrDJtWLAm4rFcd0Db5tbnxKMbTX1 QdOU3Y0tO2JKhiIwlDmmBfM1LnGynF3lDqmGIVd2hWqZ6VALWH5MVtwJrO0= Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Return-Path: Hi Arnd, Am 08.01.2018 um 10:27 schrieb Arnd Bergmann: > On Mon, Jan 8, 2018 at 10:15 AM, Stefan Wahren wrote: >> Hi Eric, >> Am 07.01.2018 um 23:08 schrieb Eric Anholt: >>> Stefan Wahren writes: >>> >>>> This reverts commit 014d6da6cb2525d7f48fb08c705cb130cc7b5f4a. >>>> >>>> The DT clean up could trigger an endless deferred probe of DWC2 USB >>>> driver >>>> on the Raspberry Pi 2/3. So revert the change until we fixed the probing >>>> issue. >>> Why's that? I found that I needed to enable the generic no-op phy >>> driver, but other than that it was fine. >> >> in order to avoid this regression. Changing the configuration is not a >> solution for the kernelci guys. >> >> Btw >> >> CONFIG_NOP_USB_XCEIV=y >> >> is already enabled in arm64/defconfig and the issue still occured. Do you >> mean a different option? > Obviously we need to fix this, but I really want to understand what exactly > happened so we can fix the code if possible rather than making the > dts file incompatible with the binding again. i fully agree, but dwc2 "hacking" usually requires more time than reverting this change. > > Do you have any more insight into how we get into the deferred probe > situation? I send this bug report [1] on Friday to linux-usb. Stefan [1] - https://marc.info/?l=linux-usb&m=151518314314753&w=2 > > Arnd