Messages in this thread | | | Date | Mon, 14 Jul 2014 09:04:05 +0100 | Subject | Re: [PATCHv8 2/2] mailbox: Introduce framework for mailbox | From | Ashwin Chaugule <> |
| |
Hello,
On 14 July 2014 05:17, Jassi Brar <jaswinder.singh@linaro.org> wrote: > On 11 July 2014 17:16, Ashwin Chaugule <ashwin.chaugule@linaro.org> wrote: >> Hi Jassi, >> >> Other than a few nits, this looks good to me. >> > Thanks for the nits. I will club them together with other feedback on > the patchset. > >> >> Hopefully you've run this through checkpatch as well? Also, were you >> able to sort out the process of getting this stuff hosted on Linaro's >> git servers? or elsewhere? It would really help maintainers to pick >> this up and for others to rebase on top of your changes. >> > I didn't think it was a good idea to keep a 'dev-branch' for a new > subsystem before having even a single Ack or Reviewed-by. Users > should track patch revisions on mailing lists and later maintainers > will be sent a tree:branch to pick Acked patches from.
Your private dev branch doesn't need to have any Acks/Reviewed-bys. You can add my signature after addressing the feedback.
Reviewed-by: Ashwin Chaugule <ashwin.chaugule@linaro.org>
Cheers, Ashwin
| |