On Fri, Apr 3, 2020 at 1:27 AM Matthew Wilcox <willy@infradead.org> wrote:

> On Thu, Apr 02, 2020 at 04:22:37PM -0700, Aslan Bakirov wrote:
> > +static inline int __init cma_declare_contiguous(phys_addr_t base,
> > +                     phys_addr_t size, phys_addr_t limit,
> > +                     phys_addr_t alignment, unsigned int order_per_bit,
> > +                     bool fixed, const char *name, struct cma **res_cma)
> > +                     {
>
> This { should be in the first column.


> > -                     addr = memblock_phys_alloc_range(size, alignment,
> > -                                                      highmem_start,
> limit);
> > +                     addr = memblock_alloc_range_nid(size, alignment,
> > +                              highmem_start, limit, nid, false);
>
> Two extra tabs, not one.
>
> Thanks for the comments. Addressed them.