Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752509AbbHEKwR (ORCPT ); Wed, 5 Aug 2015 06:52:17 -0400 Received: from mail-ig0-f172.google.com ([209.85.213.172]:38758 "EHLO mail-ig0-f172.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752300AbbHEKwD (ORCPT ); Wed, 5 Aug 2015 06:52:03 -0400 MIME-Version: 1.0 In-Reply-To: <1438564390-28111-1-git-send-email-leo.yan@linaro.org> References: <1438564390-28111-1-git-send-email-leo.yan@linaro.org> Date: Wed, 5 Aug 2015 16:22:01 +0530 Message-ID: Subject: Re: [RFC PATCH 0/3] mailbox: hisilicon: add mailbox driver From: Jassi Brar To: Leo Yan Cc: Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Catalin Marinas , Will Deacon , Wei Xu , Bintian Wang , Haojian Zhuang , Devicetree List , Linux Kernel Mailing List , "linux-arm-kernel@lists.infradead.org" , Guodong Xu , Jian Zhang , Zhenwei Wang , Haoju Mo , Dan Zhao , kongfei@hisilicon.com, Guangyue Zeng Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1704 Lines: 40 On Mon, Aug 3, 2015 at 6:43 AM, Leo Yan wrote: > This patch series is to implement Hisilicon's mailbox driver and enable > the mailbox controller on Hi6220. > Cool. > The Hisilicon mailbox supports up to 32 channels. Each channel is > unidirectional with a maximum message size of 8 words. I/O is performed > using register access (there is no DMA) and the cell raises an interrupt > when messages are received. > FYI, none of other platforms has mailbox with DMA. 8-word deep fifo, RX interrupt and optional TX interrupt is pretty common. So cool still. > For easily extending for Hisilicon series SoCs (SoCs may have difference > for register's definition with each other), so firstly implement common > mailbox driver; this common mailbox driver provides three mainly > functionality: > > - help register channels into framework; > - hook low level callback functions for register's operations; > - Enhance rx channel's message queue, which is based on the code in > drivers/mailbox/omap-mailbox.c. > Not cool. Please don't reinvent the wheel by having platform specific implementation of the mailbox api. Which vendor doesn't plan to roll out new SoCs, and hence variations of mailbox controllers? The OMAP stack predates the common api, and was actually supposed to be converted over eventually. Please implement just the drivers/mailbox/hi6220-mailbox.c (preferably by the name of the mailbox controller, if any) Thanks. -- 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/