[PATCH 2/8] dal: remove some unused wrappers

Harry Wentland harry.wentland at amd.com
Mon Dec 19 15:27:05 UTC 2016


Hi Emil,

On 2016-12-19 07:46 AM, Emil Velikov wrote:
> Hi Harry,
>
> On 14 December 2016 at 16:26, Harry Wentland <harry.wentland at amd.com> wrote:
>> They are still used all over the place (e.g.
>> dc/dce110/dce110_resource.c:413).
>>
>> We should at least do an spatch to use kzalloc/krealloc/kfree across the
>> board if the wrappers are an issue.
>>
>> NAKed
>>
> [Ignoring all the heat around the DC/DAL topic for a moment]
>
> Pardon for reviving this old thread, yet I believe it's beneficial,
> esp. for the less experienced open-source/kernel devs.
>
> There is little-to-no merit of keeping dead code in upstream kernels.
> That is, unless there's a series in discussion which uses the code ;-)
>
> NACKs like this one are not cool, furthermore the tend to
> alienate/push away people trying to help. With DC/DAL in mind, I think
> you/others want all the help people can offer ;-)
>

Very good point. I think Dave set me straight pretty quickly there. :)

I'll make sure I'll take a much closer look at patches in the future 
before considering NAKs. My impression here was that this code was not 
dead, which was obviously a wrong one and a good indication that a lot 
of the DC codebase is still quite messy.

Thanks,
Harry

> Regards,
> Emil
>


More information about the amd-gfx mailing list