am33xx: default to status = "disabled"?

Hiremath, Vaibhav
2012-08-03T09:31:50Z

On Fri, Aug 03, 2012 at 04:58:16, Arnd Bergmann wrote:
> On Thursday 02 August 2012, Daniel Mack wrote:
> > currently, all devices in arch/arm/boot/dts/am33xx.dtsi are enabled by
> > default. However, depending on the actual board dts, only some of the
> > devices should actually be initialized, given that they only make sense
> > if their pins are actually wired on the board.
> >
> > On other platform, such devices are marked with status = "disabled", and
> > the board files re-enable those they really use by overriding that
> > status again. That approach seems to make sense - shouldn't the same be
> > done in am33xx.dtsi or am I missing something?
>
> I agree, they should be disabled, at least the serial ports, and probably
> also the i2c controllers.
>

You are right, it needs to be disabled. I will create a patch and submit it.

Thanks,
Vaibhav