Re: [PATCH 1/5] cpufreq: Define cpufreq_set_drv_attr_files() to addper CPU sysfs attributes

From: Lukasz Majewski
Date: Thu Jun 06 2013 - 05:16:24 EST


Hi Viresh,

> On 6 June 2013 14:28, Lukasz Majewski <l.majewski@xxxxxxxxxxx> wrote:
> > I'm confused here.
> >
> > So you want to check dev_attr for NULL just after:
> > drv_attr = cpufreq_driver->attr;
> > if (!drv_attr)
> > goto error;
> >
> > and skip the check at the while loop:
> > while ((drv_attr) && (*drv_attr))
> >
> > to
> >
> > while ((*drv_attr))
> >
> > Am I correct?
>
> Bingo!!

Ok, no problem :-)

--
Best regards,

Lukasz Majewski

Samsung R&D Institute Poland (SRPOL) | Linux Platform Group
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/