On Mon, Jul 27, 2015 at 04:54:09PM +0530, Kishon Vijay Abraham I wrote: > vsel_reg and enable_reg of the pbias regulator descriptor should actually > have the offset from syscon. However after the pbias device tree node I'm having a hard time understanding this statement, sorry. What makes you say that they "shouild actually have the offset from syscon"? What is the problem that this is supposed to fix? > is moved as a child node of syscon, vsel_reg and enable_reg has the > absolute address because of the address translation that happens while > creating device from device tree node. > So avoid using platform_get_resource and use of_get_address in order to > get only the offset (untranslated address) and populate these in > vsel_reg and enable_reg. This sounds like we're going in the wrong direction, we're moving from a more generic API to a firmware specific one. Why is this a good fix?
Attachment:
signature.asc
Description: Digital signature