On 17:18-20121106, Tony Lindgren wrote:
> * Nishanth Menon <n...@ti.com> [121106 13:50]:
> > On 10:49-20121106, Tony Lindgren wrote:
> > >
> > > Looks like there are other things there too that's not platform data:
> > >
> > > struct voltagedomain *voltdm_lookup(const char *name);
> > > int voltdm_scale(struct voltagedomain *voltdm, unsigned long target_volt);
> > > unsigned long voltdm_get_voltage(struct voltagedomain *voltdm);
> > > struct omap_volt_data *omap_voltage_get_voltdata(struct voltagedomain
> > > *voltdm,
> > > unsigned long volt);
> > >
> > > Can you please add a patch fixing that ASAP?
> >
> > Agreed include/linux/platform_data/voltage-omap.h has more functions as
> > well.
> > Considering it did:
> > rename arch/arm/plat-omap/include/plat/voltage.h =>
> > include/linux/platform_data/voltage-omap.h
> >
> > Where do we move these functions to?
> >
> > drivers/power/avs/smartreflex.c needs:
> > omap_voltage_get_voltdata
> > and
> > drivers/power/avs/smartreflex-class3.c
> > will need voltdm_reset and voltdm_get_voltage
>
> How about something local drivers/power/avs/smartreflex.h?
These APIs are exposed by voltage layer, not smartreflex :(
stuff like voltdm_scale will have to be used by regulator logic
eventually, so moving to AVS driver header is probably not right.
--
Regards,
Nishanth Menon
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
