From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marek Vasut Date: Mon, 18 Jun 2018 09:19:29 +0200 Subject: [U-Boot] [PATCH] usb: sunxi: ohci: make ohci_t the first member in private data In-Reply-To: References: <20180617161342.19228-1-anarsoul@gmail.com> <082e3114-8207-4137-e59f-71956bf4d0d9@denx.de> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 06/18/2018 08:15 AM, Jagan Teki wrote: > On Mon, Jun 18, 2018 at 6:44 AM, Marek Vasut wrote: >> On 06/17/2018 06:13 PM, Vasily Khoruzhick wrote: >>> ohci-hcd casts priv_data pointer to (ohci_t *), thus it must be >>> the first member in private data struct. >>> >>> Fixes 831cc98b1 ("usb: sunxi: Simplify ccm reg base code") >>> >>> Signed-off-by: Vasily Khoruzhick >> >> Sigh, I really wonder how (or if at all!) the original series was >> tested. And then I get flak for scrutinizing patches, right ... > > APAIK, I did basic sanity with possible tests. >From what I see in the other thread, the USB never worked with the series. If the controller returns 0s as EHCI version, something is obviously broken and I don't even understand how that could be an acceptable positive test result. The USB HCD version register can NOT EVER contain zeroes as per the specification. I am really unhappy here. > But one thing for sure is, you should have to wait for sometime to > apply this patch. Applying fast (that to during weekend) making > reviewers or maintainers not giving enough room to work. Putting my USB maintainer hat on, I am quite sure I can evaluate such a simple yet critical bugfix, if only by spending those two minutes to look at the code in ohci_register() . And since you complain about the rate patches get in again, let me remind you how much flak I got for taking my time reviewing your series and not applying it right away. Now I am getting flak for applying stuff too quickly instead. You know, maybe if you spend more time testing the patches you send thoroughly instead of lecturing people on the MLs, we wouldn't be having this conversation. -- Best regards, Marek Vasut