David Hendricks posted comments on this change.

View Change

Patch set 1:Code-Review +2

I like the clean-up that this accomplishes, so LGTM.

Bikeshed: We may want to revisit the name or meaning of "internal" to not exclude ARM and other architectures. The naming is sort of confusing when considering linux_spi and (soon) linux_mtd which also control an internal programmer interface.

To view, visit change 22671. To unsubscribe, visit settings.