* RE: [PATCH] ARM: dts: Disable Exynos5250 I2S controllers by default [not found] ` <20130910174113.GB4708@sirena.org.uk> @ 2013-09-12 10:42 ` Kukjin Kim 2013-09-12 11:15 ` Tomasz Figa 0 siblings, 1 reply; 2+ messages in thread From: Kukjin Kim @ 2013-09-12 10:42 UTC (permalink / raw) To: 'Mark Brown', 'Mark Rutland' Cc: rob.herring, 'Pawel Moll', 'Stephen Warren', linux-samsung-soc, linaro-kernel, 'devicetree' Mark Brown wrote: > > On Tue, Sep 10, 2013 at 06:35:48PM +0100, Mark Rutland wrote: > > > It seems far more sensible to me to mark devices disabled by default in > > shared dtsi files and then okay them as needed in particular dts files. > > I'd be happy with more of this. > > Yeah, me too - though only for devices that have an external impact, for > things that are internal only (eg, a crypto engine) it makes sense to > enable them by default since they should normally be usable regardless > of the system configuration. (+ DT ML) Makes sense but I need to get the opinions from DT guys... - Kukjin ^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH] ARM: dts: Disable Exynos5250 I2S controllers by default 2013-09-12 10:42 ` [PATCH] ARM: dts: Disable Exynos5250 I2S controllers by default Kukjin Kim @ 2013-09-12 11:15 ` Tomasz Figa 0 siblings, 0 replies; 2+ messages in thread From: Tomasz Figa @ 2013-09-12 11:15 UTC (permalink / raw) To: Kukjin Kim Cc: 'Mark Brown', 'Mark Rutland', rob.herring, 'Pawel Moll', 'Stephen Warren', linux-samsung-soc, linaro-kernel, 'devicetree' Hi Kukjin, On Thursday 12 of September 2013 19:42:57 Kukjin Kim wrote: > Mark Brown wrote: > > On Tue, Sep 10, 2013 at 06:35:48PM +0100, Mark Rutland wrote: > > > It seems far more sensible to me to mark devices disabled by default > > > in > > > shared dtsi files and then okay them as needed in particular dts > > > files. > > > I'd be happy with more of this. > > > > Yeah, me too - though only for devices that have an external impact, > > for > > things that are internal only (eg, a crypto engine) it makes sense to > > enable them by default since they should normally be usable regardless > > of the system configuration. > > (+ DT ML) > > Makes sense but I need to get the opinions from DT guys... Yes, this patch should have been posted to the DT ML indeed, but it has been already acked by Mark Rutland, one of DT binding maintainers. Best regards, Tomasz ^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2013-09-12 11:15 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1378491435-15624-1-git-send-email-broonie@kernel.org>
[not found] ` <20130910173548.GA18878@e106331-lin.cambridge.arm.com>
[not found] ` <20130910174113.GB4708@sirena.org.uk>
2013-09-12 10:42 ` [PATCH] ARM: dts: Disable Exynos5250 I2S controllers by default Kukjin Kim
2013-09-12 11:15 ` Tomasz Figa
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).