From: "Jingoo Han" <jingoohan1@gmail.com>
To: "'Michal Simek'" <michal.simek@xilinx.com>,
        "'Moritz Fischer'" <moritz.fischer@ettus.com>,
        <jassisinghbrar@gmail.com>
Cc: <linux-kernel@vger.kernel.org>, <robh+dt@kernel.org>, <pawel.moll@arm.com>,
        <mark.rutland@arm.com>, <ijc+devicetree@hellion.org.uk>,
        <galak@codeaurora.org>, <soren.brinkmann@xilinx.com>,
        <akpm@linux-foundation.org>, <gregkh@linuxfoundation.org>,
        <mchehab@osg.samsung.com>, <arnd@arndb.de>, <joe@perches.com>,
        <devicetree@vger.kernel.org>, <linux-arm-kernel@lists.infradead.org>
References: <1432317791-26851-1-git-send-email-moritz.fischer@ettus.com> <5562BB92.60808@xilinx.com>
In-Reply-To: <5562BB92.60808@xilinx.com>
Subject: Re: [PATCHv1 0/2] Adding driver for Xilinx LogiCORE IP mailbox.
Date: Mon, 25 May 2015 22:56:14 +0900
Message-ID: <000101d096f2$97457cf0$c5d076d0$@com>
MIME-Version: 1.0
Content-Type: text/plain;
	charset="us-ascii"
Content-Transfer-Encoding: 7bit
Thread-Index: AdCWsMqHkpsb6OB2T6Wtpd1TeOyLcQAQNZSA
Content-Language: ko
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1220
Lines: 35

On Monday, May 25, 2015 3:05 PM, Michal Simek wrote:
> On 05/22/2015 08:03 PM, Moritz Fischer wrote:
> > 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.
> >
> > 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
> 
> Changes update should be in every particular patch.
> Also v1 is used to the first version that's why this one should be v2.

Right, I agree with Michal Simek's opinion.
If there are 'Changes update' in each patch, it makes people review
the patches more easily. In addition, normally, 'v0' is not used.

Best regards,
Jingoo Han

> 
> Thanks,
> Michal

--
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/