Return-path: Received: from mail-qw0-f46.google.com ([209.85.216.46]:63155 "EHLO mail-qw0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754920Ab1DRObE convert rfc822-to-8bit (ORCPT ); Mon, 18 Apr 2011 10:31:04 -0400 MIME-Version: 1.0 In-Reply-To: <201104181619.35115.arnd@arndb.de> References: <201104171938.12834.arnd@arndb.de> <201104181619.35115.arnd@arndb.de> Date: Mon, 18 Apr 2011 16:31:02 +0200 Message-ID: Subject: Re: Could I (ab)use bus (struct bus_type) for virtual Broadcom bus? From: =?UTF-8?B?UmFmYcWCIE1pxYJlY2tp?= To: Arnd Bergmann Cc: George Kashperko , Hauke Mehrtens , Russell King , linux-kernel@vger.kernel.org, linux-wireless@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Arend van Spriel , Jonas Gorski , b43-dev@lists.infradead.org, Greg KH , Andy Botting , Larry Finger Content-Type: text/plain; charset=UTF-8 Sender: linux-wireless-owner@vger.kernel.org List-ID: W dniu 18 kwietnia 2011 16:19 użytkownik Arnd Bergmann napisał: > On Monday 18 April 2011, Rafał Miłecki wrote: >> W dniu 17 kwietnia 2011 19:38 użytkownik Arnd Bergmann napisał: >> >> > In general, the bus_type directly relates to how a device gets probed. >> > If broadcom uses the same basic register layout as regular AMBA devices, >> > it should use the amba bus type. >> >> From Broadcom side we *could* use some registers that are AMBA >> specific, they are present... but there is totally no point in doing >> that. Everything we use is Broadcom specific. >> >> >> > I think it would be fine to extend the AMBA bus slightly if there are >> > just minor differences. >> >> As I said, Broadcom specific driver use nothing from AMBA common >> things. Plus we implement routines that are Broadcom specific and no >> other platform will use them. > > You mean the hardware has two sets of registers containing the same > information, one of them the standard registers, and one with broadcom > specific ones? > > Why don't you just use the standard ones then? No. Did you read my first mail in this thread? There is pair of cores for every device. First is AMBA-specific called agent/wrapper and second one is Broadcom-specific. AMBA specific core called agent/wrapper has AMBA specific registers: CID and PID. However we do not ever read that in Broadcom driver, because that are useless for us. On AMBA specific core we use only some Broadcom specific registers to manage (enable/disable) *second* (Broadcom-specific) core. -- Rafał