On Thu, Sep 12, 2013 at 05:55:59PM +0200, Tomasz Figa wrote: > On Thursday 12 of September 2013 14:52:33 Mark Brown wrote: > > Please delete unneeded context from your mails, it makes it much easier > > to find the new content. > I always snip unnecessary parts of quoted message if its really long, but > if its length is in acceptable range then I try to keep the whole message > just for reference. I tend to say that when I find myself paging through entire screens of quote and can't see the relevance to any content I eventually find. > i2c@12CA0000 does look like a normal hardware I2C controller, which needs > some extra properties like samsung,i2c-sda-delay or samsung,i2c-max-bus- Shouldn't these have sane defaults? > freq. However they are specified in cros5250-common.dtsi, so I'd say that > this might be better place for the status override as well. Can we rename that file to something like exynos5250-cros-common.dtsi so that it's obvious it's relevant to exynos? I had completely failed to notice that it was anything to do with Exynos when I was searching for affected DTS files. Especially given that it's included in exactly one DTS at the minute...
Attachment:
signature.asc
Description: Digital signature
