Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752971Ab2KFDk2 (ORCPT ); Mon, 5 Nov 2012 22:40:28 -0500 Received: from avon.wwwdotorg.org ([70.85.31.133]:46649 "EHLO avon.wwwdotorg.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751697Ab2KFDk1 (ORCPT ); Mon, 5 Nov 2012 22:40:27 -0500 Message-ID: <509886A2.5060509@wwwdotorg.org> Date: Mon, 05 Nov 2012 20:40:18 -0700 From: Stephen Warren User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:15.0) Gecko/20120827 Thunderbird/15.0 MIME-Version: 1.0 To: Omar Ramirez Luna CC: Greg Kroah-Hartman , Russell King , Tony Lindgren , Janusz Krzysztofik , Loic PALLARDY , Arnd Bergmann , Ohad Ben-Cohen , Mark Brown , Dom Cobley , Wim Van Sebroeck , Linus Walleij , Suman Anna , Juan Gutierrez , Felipe Contreras , Tejun Heo , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-omap@vger.kernel.org Subject: Re: [PATCH v2 1/2] mailbox: OMAP: introduce mailbox framework References: <1352170552-29564-1-git-send-email-omar.luna@linaro.org> <1352170552-29564-2-git-send-email-omar.luna@linaro.org> In-Reply-To: <1352170552-29564-2-git-send-email-omar.luna@linaro.org> X-Enigmail-Version: 1.4.4 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1255 Lines: 25 On 11/05/2012 07:55 PM, Omar Ramirez Luna wrote: > Actually moving it from plat-omap, as this framework/driver code is > supposed to be under drivers/ folder. The framework should work with > the current supported OMAP processors (OMAP1+) that have mailbox and > can be used as a method of interprocessor communication. > > The mailbox hardware (in OMAP) uses a queued mailbox-interrupt mechanism > that provides a communication channel between processors through a set of > registers and their associated interrupt signals by sending and receiving > messages. > diff --git a/drivers/mailbox/mailbox.h b/drivers/mailbox/mailbox.h Is this a public interface to the driver? If so, shouldn't the header be in include/linux somewhere? Is this a generic interface to any mailbox driver? If so, then I don't think having "omap" in the symbol names is appropriate. If the header is specific to the OMAP driver, I don't think using the very generic filename "mailbox.h" is appropriate; use omap_mailbox.h instead? -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/