Re: [PATCH 3/4] block: replace rssd_disk_name_format() todisk_name_format()

From: Asai Thambi Samymuthu Pattrayasamy (asamymuthupa) [CONT - Type 2]
Date: Fri Mar 30 2012 - 19:55:05 EST


On 3/30/2012 2:53 AM, Ren Mingxin wrote:

> Currently, block core has been supplied "disk_name_format()", so
> we should remove duplicate function "rssd_disk_name_format()"
> and use the new function to format rssd disk names.
>
> Signed-off-by: Ren Mingxin <renmx@xxxxxxxxxxxxxx>
> ---
> mtip32xx.c | 33 +--------------------------------
> 1 file changed, 1 insertion(+), 32 deletions(-)


Looks fine.

Should the subject be "mtip32xx: ..." instead of "block: ..."? I
understand "block:" as relating to block core. I am fairly new here. If
"block:" can be used for block drivers too, that is fine too.

--
Regards,
Asai Thambi
èº{.nÇ+‰·Ÿ®‰­†+%ŠËlzwm…ébëæìr¸›zX§»®w¥Š{ayºÊÚë,j­¢f£¢·hš‹àz¹®w¥¢¸ ¢·¦j:+v‰¨ŠwèjØm¶Ÿÿ¾«‘êçzZ+ƒùšŽŠÝj"ú!¶iO•æ¬z·švØ^¶m§ÿðà nÆàþY&—