From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751974AbbG1Woj (ORCPT ); Tue, 28 Jul 2015 18:44:39 -0400 Received: from mail-la0-f52.google.com ([209.85.215.52]:35648 "EHLO mail-la0-f52.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751489AbbG1Woh convert rfc822-to-8bit (ORCPT ); Tue, 28 Jul 2015 18:44:37 -0400 MIME-Version: 1.0 In-Reply-To: <1436922022-17564-1-git-send-email-moritz.fischer@ettus.com> References: <1436922022-17564-1-git-send-email-moritz.fischer@ettus.com> Date: Tue, 28 Jul 2015 15:44:35 -0700 Message-ID: Subject: Re: [PATCHv7 0/2] Adding driver for Xilinx LogiCORE IP mailbox. From: Moritz Fischer To: Jassi Brar Cc: linux-kernel@vger.kernel.org, robh+dt@kernel.org, pawel.moll@arm.com, mark.rutland@arm.com, ijc+devicetree@hellion.org.uk, Kumar Gala , Michal Simek , =?UTF-8?Q?S=C3=B6ren_Brinkmann?= , akpm@linux-foundation.org, Greg KH , mchehab@osg.samsung.com, Arnd Bergmann , joe@perches.com, Jingoo Han , devicetree@vger.kernel.org, linux-arm-kernel , Moritz Fischer Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Jassi, just a ping to see if you're waiting on me to fix things or if this is good to go from your point of view, and you're just waiting some time to give people more time to review. Let me know if you want me to resend with Soeren's Acked-By Cheers, Moritz On Tue, Jul 14, 2015 at 6:00 PM, Moritz Fischer wrote: > Hi all, > > Thanks for hanging in there, hopefully the last round of review ... > > This patchset adds mailbox framework integration for the Xilinx LogiCORE IP > mailbox. The Xilinx LogiCORE IP mailbox is a fpga softcore that allows > interprocessor communication between AXI4 stream / memory mapped > processors. > > Cheers, > > Moritz > > Changes from v6: > - As suggested by Jassi and Sören use #mbox-cells = <1> > > Changes from v5: > - Fixed constness for ops struct > - Removed redundant checks > - Moved ops assignment > - Removed MODULE_ALIAS > > Changes from v4: > - Have separate mbox_ops structs for polling / irq mode > - Moved clk handling to startup / shutdown > - Embedded struct mbox_chan in struct xilinx_mbox > - Fixed mbox-cells in devicetree documentation > - Misc stylistic issues > > Changes from v3: > - Stylistic changes > - Changed reg size in dts to 0x100 > > Changes from v2: > - Fixed error condition for IRQ from >= 0 to > 0 > - Fixed clock enable > - Cleaned up docs > > Changes from v1: > - Added common clock framework support > - Deal with IRQs that happend before driver load, > since HW will not let us know about them when we enable IRQs > > Changes from v0: > - Several stylistic issues > - Dropped superfluous intr_mode member > - Really masking the IRQs on mailbox_shutdown > - No longer using polling by accident in non-IRQ mode > - Swapped doc and driver commits > > > Moritz Fischer (2): > dts: Adding docs for Xilinx LogiCORE IP mailbox driver. > mailbox: Adding driver for Xilinx LogiCORE IP mailbox. > > .../devicetree/bindings/mailbox/xilinx-mailbox.txt | 44 +++ > MAINTAINERS | 7 + > drivers/mailbox/Kconfig | 7 + > drivers/mailbox/Makefile | 2 + > drivers/mailbox/mailbox-xilinx.c | 367 +++++++++++++++++++++ > 5 files changed, 427 insertions(+) > create mode 100644 Documentation/devicetree/bindings/mailbox/xilinx-mailbox.txt > create mode 100644 drivers/mailbox/mailbox-xilinx.c > > -- > 2.4.3 > From mboxrd@z Thu Jan 1 00:00:00 1970 From: Moritz Fischer Subject: Re: [PATCHv7 0/2] Adding driver for Xilinx LogiCORE IP mailbox. Date: Tue, 28 Jul 2015 15:44:35 -0700 Message-ID: References: <1436922022-17564-1-git-send-email-moritz.fischer@ettus.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: In-Reply-To: <1436922022-17564-1-git-send-email-moritz.fischer-+aYTwkv1SeIAvxtiuMwx3w@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Jassi Brar Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, pawel.moll-5wv7dgnIgG8@public.gmane.org, mark.rutland-5wv7dgnIgG8@public.gmane.org, ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org, Kumar Gala , Michal Simek , =?UTF-8?Q?S=C3=B6ren_Brinkmann?= , akpm-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org, Greg KH , mchehab-JPH+aEBZ4P+UEJcrhfAQsw@public.gmane.org, Arnd Bergmann , joe-6d6DIl74uiNBDgjK7y7TUQ@public.gmane.org, Jingoo Han , devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel , Moritz Fischer List-Id: devicetree@vger.kernel.org Hi Jassi, just a ping to see if you're waiting on me to fix things or if this is good to go from your point of view, and you're just waiting some time to give people more time to review. Let me know if you want me to resend with Soeren's Acked-By Cheers, Moritz On Tue, Jul 14, 2015 at 6:00 PM, Moritz Fischer wrote: > Hi all, > > Thanks for hanging in there, hopefully the last round of review ... > > This patchset adds mailbox framework integration for the Xilinx LogiC= ORE IP > mailbox. The Xilinx LogiCORE IP mailbox is a fpga softcore that allo= ws > interprocessor communication between AXI4 stream / memory mapped > processors. > > Cheers, > > Moritz > > Changes from v6: > - As suggested by Jassi and S=C3=B6ren use #mbox-cells =3D <1> > > Changes from v5: > - Fixed constness for ops struct > - Removed redundant checks > - Moved ops assignment > - Removed MODULE_ALIAS > > Changes from v4: > - Have separate mbox_ops structs for polling / irq mode > - Moved clk handling to startup / shutdown > - Embedded struct mbox_chan in struct xilinx_mbox > - Fixed mbox-cells in devicetree documentation > - Misc stylistic issues > > Changes from v3: > - Stylistic changes > - Changed reg size in dts to 0x100 > > Changes from v2: > - Fixed error condition for IRQ from >=3D 0 to > 0 > - Fixed clock enable > - Cleaned up docs > > Changes from v1: > - Added common clock framework support > - Deal with IRQs that happend before driver load, > since HW will not let us know about them when we enable IRQs > > Changes from v0: > - Several stylistic issues > - Dropped superfluous intr_mode member > - Really masking the IRQs on mailbox_shutdown > - No longer using polling by accident in non-IRQ mode > - Swapped doc and driver commits > > > Moritz Fischer (2): > dts: Adding docs for Xilinx LogiCORE IP mailbox driver. > mailbox: Adding driver for Xilinx LogiCORE IP mailbox. > > .../devicetree/bindings/mailbox/xilinx-mailbox.txt | 44 +++ > MAINTAINERS | 7 + > drivers/mailbox/Kconfig | 7 + > drivers/mailbox/Makefile | 2 + > drivers/mailbox/mailbox-xilinx.c | 367 +++++++++++= ++++++++++ > 5 files changed, 427 insertions(+) > create mode 100644 Documentation/devicetree/bindings/mailbox/xilinx-= mailbox.txt > create mode 100644 drivers/mailbox/mailbox-xilinx.c > > -- > 2.4.3 > -- To unsubscribe from this list: send the line "unsubscribe devicetree" i= n the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html From mboxrd@z Thu Jan 1 00:00:00 1970 From: moritz.fischer@ettus.com (Moritz Fischer) Date: Tue, 28 Jul 2015 15:44:35 -0700 Subject: [PATCHv7 0/2] Adding driver for Xilinx LogiCORE IP mailbox. In-Reply-To: <1436922022-17564-1-git-send-email-moritz.fischer@ettus.com> References: <1436922022-17564-1-git-send-email-moritz.fischer@ettus.com> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Jassi, just a ping to see if you're waiting on me to fix things or if this is good to go from your point of view, and you're just waiting some time to give people more time to review. Let me know if you want me to resend with Soeren's Acked-By Cheers, Moritz On Tue, Jul 14, 2015 at 6:00 PM, Moritz Fischer wrote: > Hi all, > > Thanks for hanging in there, hopefully the last round of review ... > > This patchset adds mailbox framework integration for the Xilinx LogiCORE IP > mailbox. The Xilinx LogiCORE IP mailbox is a fpga softcore that allows > interprocessor communication between AXI4 stream / memory mapped > processors. > > Cheers, > > Moritz > > Changes from v6: > - As suggested by Jassi and S?ren use #mbox-cells = <1> > > Changes from v5: > - Fixed constness for ops struct > - Removed redundant checks > - Moved ops assignment > - Removed MODULE_ALIAS > > Changes from v4: > - Have separate mbox_ops structs for polling / irq mode > - Moved clk handling to startup / shutdown > - Embedded struct mbox_chan in struct xilinx_mbox > - Fixed mbox-cells in devicetree documentation > - Misc stylistic issues > > Changes from v3: > - Stylistic changes > - Changed reg size in dts to 0x100 > > Changes from v2: > - Fixed error condition for IRQ from >= 0 to > 0 > - Fixed clock enable > - Cleaned up docs > > Changes from v1: > - Added common clock framework support > - Deal with IRQs that happend before driver load, > since HW will not let us know about them when we enable IRQs > > Changes from v0: > - Several stylistic issues > - Dropped superfluous intr_mode member > - Really masking the IRQs on mailbox_shutdown > - No longer using polling by accident in non-IRQ mode > - Swapped doc and driver commits > > > Moritz Fischer (2): > dts: Adding docs for Xilinx LogiCORE IP mailbox driver. > mailbox: Adding driver for Xilinx LogiCORE IP mailbox. > > .../devicetree/bindings/mailbox/xilinx-mailbox.txt | 44 +++ > MAINTAINERS | 7 + > drivers/mailbox/Kconfig | 7 + > drivers/mailbox/Makefile | 2 + > drivers/mailbox/mailbox-xilinx.c | 367 +++++++++++++++++++++ > 5 files changed, 427 insertions(+) > create mode 100644 Documentation/devicetree/bindings/mailbox/xilinx-mailbox.txt > create mode 100644 drivers/mailbox/mailbox-xilinx.c > > -- > 2.4.3 >