Re: [PATCH v9] mmc: Export host capabilities to debugfs.

[Andy Shevchenko <andriy.shevchenko@linux.intel.com>]

On Mon, 2018-03-12 at 16:22 +0200, Andy Shevchenko wrote:
> On Mon, 2018-03-12 at 19:04 +0530, Harish Jenny K N wrote:
> > 
> > On Monday 12 March 2018 06:33 PM, Andy Shevchenko wrote:
> > > On Mon, 2018-03-12 at 18:00 +0530, Harish Jenny K N wrote:
> > > > > > > > > +	for_each_set_bit(bit, (const unsigned long
> > > > > > > > > *)&caps,
> > > > > > > > > BITS_PER_LONG)
> > > > > > > > > +	for_each_set_bit(bit, (const unsigned long
> > > > > > > > > *)&caps2,
> > > > > > > > > BITS_PER_LONG)
> > > > > > > > 
> > > > > > > > Explicit casting is not needed anymore in both cases.
> > Sorry for I was not clear that the compilation error is for the
> > following case

> > > u32 caps = host->caps;

This is wrong! I told you that and you seems changed at least in v9.
What are you doing locally?

> > > for_each_set_bit(bit, &caps, ARRAY_SIZE(mmc_host_capabilities))
> > 

> Are you sure you have caps defined as unsigned long?
> 
> Check your code, the explicit casting should gone. Otherwise you are
> doing something wrong.

See above.

-- 
Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Intel Finland Oy