On Wed, Oct 22, 2014 at 04:01:26PM -0700, Stephen Boyd wrote: > >> "Currently a bunch of I2C/SPI MFD drivers are using IORESOURCE_IO for > >> register address ranges. Since this causes some confusion due to the > >> primary use of this resource type for PCI/ISA I/O ports create a new > >> resource type IORESOURCE_REG." > > Sorry, I mistook IORESOURCE_REG or IORESOURCE_IO. You're right, this > > isn't an issue. > > I'm still concerned about the implications of automatically populating > > platform_devices with this resource type. I'll talk to Mark about it > > face to fact at Connect this week. Hrm, missed that discussion. > Where did this end up? When we talked at Connect I think we settled on > exploring a driver core specific API like dev_get_localbus_address() > that calls of_get_localbus_address() for devices with an of_node and in ... > That's fine, but I still think we want to have the IORESOURCE_REG > resources given that we have drivers in-flight and some already merged > that are using the IORESOURCE_REG resource. Furthermore, ACPI is using Right, IORESOURCE_REG was the original solution to the overlapping use of IORESOURCE_IO (rather than having multiple IORESOURCE_IO trees since we do special magic for IORESOURCE_IO for legacy reasons which was causing issues but the idea of doing it for generic I/O make sense). > the platform bus for MFDs so it's not like we're going to be using a > different bus in the future for these pmic sub-device drivers if we > decide to do pmic devices in ACPI (looks like there is at least > precedence for that with Intel's i2c pmic). Supporting this on ACPI is > going to take the same effort if we plumb it into the resource table or > we plumb it into a new driver core API, so the bus agnostic angle isn't Even if we do these things in ACPI it's not at all clear to me that the idea of putting the internals of the device in ACPI would be at all tasteful there. Personally I'm not a big fan of always doing it in DT either but it's more tasteful there and definitely does make sense for some things.
Attachment:
signature.asc
Description: Digital signature
