From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Warren Subject: Re: [PATCH v1 2/9] mailbox: Add NVIDIA Tegra XUSB mailbox driver Date: Wed, 25 Jun 2014 16:02:08 -0600 Message-ID: <53AB46E0.5010908@wwwdotorg.org> References: <1403072180-4944-1-git-send-email-abrestic@chromium.org> <1403072180-4944-3-git-send-email-abrestic@chromium.org> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1403072180-4944-3-git-send-email-abrestic-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org> Sender: linux-usb-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Andrew Bresticker , devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-doc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, linux-usb-u79uwXL29TY76Z2rM5mHXA@public.gmane.org Cc: Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Randy Dunlap , Thierry Reding , Russell King , Linus Walleij , Greg Kroah-Hartman , Mathias Nyman , Grant Likely , Alan Stern , Kishon Vijay Abraham I , Arnd Bergmann List-Id: devicetree@vger.kernel.org On 06/18/2014 12:16 AM, Andrew Bresticker wrote: > The Tegra XHCI controller communicates requests to the host through > a mailbox interface. Host drivers which can handle these requests, > such as the Tegra XUSB pad controller driver and upcoming Tegra XHCI > host controller driver, can send messages and register to be notified > of incoming messages. > diff --git a/include/linux/tegra-xusb-mbox.h b/include/linux/tegra-xusb-mbox.h > +extern int tegra_xusb_mbox_register_notifier(struct tegra_xusb_mbox *mbox, > + struct notifier_block *nb); > +extern void tegra_xusb_mbox_unregister_notifier(struct tegra_xusb_mbox *mbox, > + struct notifier_block *nb); > +extern int tegra_xusb_mbox_send(struct tegra_xusb_mbox *mbox, > + enum tegra_xusb_mbox_cmd cmd, u32 data); > +extern struct tegra_xusb_mbox * > +tegra_xusb_mbox_lookup_by_phandle(struct device_node *np, const char *prop); This seems to use a custom API. I've seen mention of a mailbox subsystem, and I assume that has a standardized API. Should this driver implement that instead? -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html