Hi Alan, generally ok with the change. On Wed, Sep 28, 2016 at 11:21 AM, Alan Tull <atull@xxxxxxxxxxxxxxxxxxxxx> wrote: > -int fpga_mgr_buf_load(struct fpga_manager *mgr, u32 flags, const char *buf, > - size_t count) > +int fpga_mgr_buf_load(struct fpga_manager *mgr, struct fpga_image_info *info, > + const char *buf, size_t count) Doesn't this break the both socfpga and zynq if [6/12] and [7/12] are not part of this commit? i.e shouldn't 5,6 and 7 be a single commit? Cheers, Moritz -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html