[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH RFC 17/19] Remove x86 specific defintions from efibind.h



On Wed, 2014-07-09 at 11:27 -0700, Roy Franz wrote:
> On Wed, Jul 2, 2014 at 5:03 AM, Ian Campbell <Ian.Campbell@xxxxxxxxxx> wrote:
> > On Sat, 2014-06-28 at 02:25 +0100, Roy Franz wrote:
> >> Clean up arm64 efibind.h to not include x86 specific definitions.
> >
> > For the majority of this series I really appreciate the refactor in one
> > patch and then move in another approach, but I think for this specific
> > case you could fold these together into a single "introduce the arm64
> > efi bindings" patch.
> >
> > Ian.
> >
> 
> I'll collapse these, and keep the arm64 efibind.h.

Thanks.

>   I try very hard to
> keep code movement isolated,
> as it's easy to miss small changes when mixed, but it does lead to
> longer series.

It's appreciated, much easier to review IMHO since you can trivially ack
half the patches if they are pure motion.

Ian.


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.