MIME-Version: 1.0
In-Reply-To: <20170627160045.GA9977@kroah.com>
References: <20170627144135.15599-1-laurentiu.tudor@nxp.com>
 <CAK8P3a2pD5+kd5NLbkJ_ppDb8jgNE7Cuxm=GLHf-ZgsLgoq9Sg@mail.gmail.com> <20170627160045.GA9977@kroah.com>
From: Arnd Bergmann <arnd@arndb.de>
Date: Tue, 27 Jun 2017 22:21:29 +0200
Message-ID: <CAK8P3a2A6-+ceo_gzYQ2Si0k2aUrzj9EupLnp2iiKpT2Dznk+w@mail.gmail.com>
Subject: Re: [PATCH v2 00/15] staging: fsl-mc: clean up header files
To: gregkh <gregkh@linuxfoundation.org>
Cc: laurentiu.tudor@nxp.com, devel@driverdev.osuosl.org,
        Stuart Yoder <stuyoder@gmail.com>, Marc Zyngier <marc.zyngier@arm.com>,
        Roy Pledge <roy.pledge@nxp.com>, Alexander Graf <agraf@suse.de>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        catalin.horghidan@nxp.com, ioana.ciornei@nxp.com,
        Leo Li <leoyang.li@nxp.com>, bharat.bhushan@nxp.com,
        Linux ARM <linux-arm-kernel@lists.infradead.org>
Content-Type: text/plain; charset="UTF-8"
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1422
Lines: 32

On Tue, Jun 27, 2017 at 6:00 PM, gregkh <gregkh@linuxfoundation.org> wrote:
> On Tue, Jun 27, 2017 at 05:33:21PM +0200, Arnd Bergmann wrote:
>> On Tue, Jun 27, 2017 at 4:41 PM,  <laurentiu.tudor@nxp.com> wrote:
>> > From: Laurentiu Tudor <laurentiu.tudor@nxp.com>
>> >
>> > This patch series contain mainly clean-ups of the mc-bus header files
>> > with the final goal of reorganizing them in just 2 files: a public
>> > and a private one, as per GregKH suggestion [1].
>> > Here's a summary of the header reorganizing:
>> >  - existing mc.h used as public header (contained most of the public API)
>> >  - existing fsl-mc-private.h used for private header
>> >  - mc-bus.h merged both in public and private header
>> >  - mc-sys.h and mc-cmd.h merged in public header
>> >  - dprc.h made private
>> >  - dpmng.h deleted
>> >
>> > [1] https://patchwork.kernel.org/patch/9775683/
>> >
>> > version 2 changes
>> >  - fix compilation issue
>> >  - reordered a comparison, be consistent when checking strcmp() return
>> >    (Joe Perches, first two patches)
>>
>> It looks like v1 was already merged into staging-next, so the changes would
>> have to be relative to what's already merged, rather than replacing them.
>
> No, I had to revert the whole v1 mess, so this is correct...

Ok, sorry for the confusion on my side then. Obviously you can ignore my fixup
patch too, as that is already part of this series.

       Arnd