[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH V1]sm501: dead code removal
From: |
Aishwarya Kadlag |
Subject: |
Re: [Qemu-devel] [PATCH V1]sm501: dead code removal |
Date: |
Mon, 12 Mar 2018 23:28:54 +0530 |
Really glad to get a feedback so soon on my first attempt.
Yes Thomas sir, I will apply these changes in my next patch. Now that this
patch is already been implemented by Gerd sir in 2014 and it was not
applied, should I consider finding some other task or should I improve this
patch to get it merged or applied? Also as this was my first attempt, I
selected the task mentioned in the wiki page
https://wiki.qemu.org/BiteSizedTasks#Dead_code_removal
As Paolo sir said, if this is too easy then can someone please suggest me
any other task to work on so that I can create something useful to qemu??
Thanks,
Aishwarya
On Mon, Mar 12, 2018 at 6:33 PM, Peter Maydell <address@hidden>
wrote:
> On 12 March 2018 at 07:05, Gerd Hoffmann <address@hidden> wrote:
> > Might be it actually is that easy.
>
> In fact looking through the list archives I see you actually
> sent a patchset to do this for the sm501 device back in March
> last year:
> http://lists.gnu.org/archive/html/qemu-devel/2017-03/msg01172.html
>
> I think it didn't get applied mostly because it was on list
> at the same time as a significant revamp of the sm501 device
> code. So it probably won't apply cleanly to the current code,
> but it looks like a good starting point for identifying what
> the changes needed here are.
>
> thanks
> -- PMM
>