On Tue, 2015-11-03 at 00:33 -0600, Dogra Raghav-B46184 wrote: > Hi Scott, > > I just checked on patchwork and noticed that the original patch has been > modified as somehow my original comment has become part of the patch: > + if (!lbc) > + goto out; > > ctrl->saved_regs = kmalloc(sizeof(struct fsl_lbc_regs), GFP_KERNEL); > if (!ctrl->saved_regs) > return -ENOMEM; > [Dogra Raghav-B46184] This is the existing upstream code. Are you > suggesting to modify the upstream code to take care of individual registers? > > _memcpy_fromio(ctrl->saved_regs, lbc, sizeof(struct fsl_lbc_regs)); > + > The patchwork is also showing two links for the patch. Please apply the > original patch present at: > https://patchwork.ozlabs.org/patch/538605/ > > I will look into how this happened to avoid such incidents in future.
Your reply looked like a new patch, so Patchwork treated it as such. If you fix your mailer to use proper quote markers (the > at the beginning of each line) then this won't happen. -Scott _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
