Return-path: Received: from 80-190-117-144.ip-home.de ([80.190.117.144]:50245 "EHLO bu3sch.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751000Ab1BNWwO (ORCPT ); Mon, 14 Feb 2011 17:52:14 -0500 Subject: Re: Notes on ssb specs and implementation From: Michael =?ISO-8859-1?Q?B=FCsch?= To: =?UTF-8?Q?Rafa=C5=82_Mi=C5=82ecki?= Cc: linux-wireless@vger.kernel.org, Larry Finger , b43-dev In-Reply-To: (sfid-20110214_233217_829894_FFFFFFFFD1071FC3) References: <1297282621.9734.5.camel@maggie> <1297720877.5683.3.camel@maggie> (sfid-20110214_233217_829894_FFFFFFFFD1071FC3) Content-Type: text/plain; charset="UTF-8" Date: Mon, 14 Feb 2011 23:52:05 +0100 Message-ID: <1297723925.5683.11.camel@maggie> Mime-Version: 1.0 Sender: linux-wireless-owner@vger.kernel.org List-ID: On Mon, 2011-02-14 at 23:32 +0100, Rafał Miłecki wrote: > Sure. It's not for submission, so you have to expect magic values. That patch doesn't look too bad. > Personally I don't like for example description at > http://bcm-v4.sipsolutions.net/802.11/PHY . It's really messed in "Put > PHY Into Reset". > Section "Put PHY Into Reset" is really "How to reset PHY" and it > duplicates code of takine PHY out of reset. Yeah, well. Maybe I repeat myself for the hundredth time: Do not duplicate the specifications exactly. Apply common sense before implementing the code. If we can do better on certain things, do it. > We also have a lot of magic in ssb_device_enable. We reset SB and PHY > at the same time. I guess it introduces some optimization but makes it > harder to understand, especially if you try to understand implemented > code and look as specs at the same time. Well, it's core specific flags. I don't see anything wrong here. I think the PHY reset might even require an actual core reset to work correctly. So you could never untie both. But that's exactly those magic things we're never going to find out unless we have hardware documentation. -- Greetings Michael.