Martin, On 9/28/16 12:54, Martin K. Petersen wrote: >>>>>> "Damien" == Damien Le Moal <[email protected]> writes: > > Purely cosmetic: Looks like whitespace is a bit funky (spaces instead of > tabs several places). Also a few typos. > > + * blkdev_report_zones - Get zones information > + * @bdev: Target block device > + * @sector: Sector from which to report zones > + * @zones: Array of zone structures where to return the zones > information > + * @nr_zones: Number of zone structures in the zone array > + * @gfp_mask: Memory allocation flags (for bio_alloc) > > Tabs and spaces: Together at last! ^^^^^ > > + /* > + * Process the report resukt: skip the header and go through the > > result > > + /* Check alignement (handle eventual smaller last zone) */ > > alignment
Thanks for all the comments. Should I send a fixed-up series or just send correction patches later ? Best regards. -- Damien Le Moal, Ph.D. Sr. Manager, System Software Group, HGST Research, HGST, a Western Digital brand [email protected] (+81) 0466-98-3593 (ext. 513593) 1 kirihara-cho, Fujisawa, Kanagawa, 252-0888 Japan www.hgst.com -- To unsubscribe from this list: send the line "unsubscribe linux-block" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
