Re: [PATCH v3 00/12] staging: tidspbridge: various cleanups

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Ionut,

On Sun, Nov 21, 2010 at 4:46 AM, Ionut Nicu <ionut.nicu@xxxxxxxxx> wrote:
> This set of patches replaces some of the redundant components of
> the tidspbridge driver, such as:
>
> * wrapper functions for kmalloc/kfree
> * custom bitmap implementation
> * custom linked list implementation (list_head wrapper)
>
> with the standard linux kernel interfaces.
>
> The patches also do some code reorganization for increasing readability.
> Most of the changes reduce the code indentation level and simplify the code.
> No functional changes were done.
>
> There are many places in this driver that need this kind of cleanup, but
> these patches only fix the functions that were touched while converting
> the code to use linux bitmap and list_head.

Thanks for your patches, I just fixed the style of some multiline
comments before pushing, since these are very minor fixes I avoided
the noise of resending the patches.

Please remember to follow the Coding Style for multi line comments
whenever you insert back those lines (even if the code was that way).

> Ionut Nicu (12):
>  staging: tidspbridge: remove gs memory allocator
>  staging: tidspbridge: remove utildefs
>  staging: tidspbridge: switch to linux bitmap API
>  staging: tidspbridge: remove gb bitmap implementation

Pushed

>  staging: tidspbridge: rmgr/node.c code cleanup

Changes requested

>  staging: tidspbridge: convert core to list_head

Fixed a multiline style comment

>  staging: tidspbridge: convert pmgr to list_head

Fixed 2 multiline style comment

>  staging: tidspbridge: convert rmgr to list_head
>  staging: tidspbridge: remove custom linked list

Pushed

>  staging: tidspbridge: core code cleanup

Fixed multiline style comments

>  staging: tidspbridge: pmgr code cleanup

Fixed multiline style comments

>  staging: tidspbridge: rmgr code cleanup

Doesn't apply because of patch "staging: tidspbridge: rmgr/node.c code
cleanup" was not pushed.

Regards,

Omar
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux