From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?UTF-8?B?QW5kcmVhcyBGw6RyYmVy?= Subject: Re: [PATCH v4 4/4] ARM: dts: Add exynos5250-spring device tree Date: Sat, 02 Aug 2014 09:49:53 +0200 Message-ID: <53DC9821.3010108@suse.de> References: <1406822910-6255-1-git-send-email-afaerber@suse.de> <1406822910-6255-5-git-send-email-afaerber@suse.de> <53DA936E.9060004@gmail.com> <53DA9709.10602@suse.de> <53DA9BA9.60808@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Received: from cantor2.suse.de ([195.135.220.15]:36074 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751099AbaHBHuB (ORCPT ); Sat, 2 Aug 2014 03:50:01 -0400 In-Reply-To: Sender: linux-samsung-soc-owner@vger.kernel.org List-Id: linux-samsung-soc@vger.kernel.org To: Doug Anderson , Tomasz Figa Cc: linux-samsung-soc , Vincent Palatin , "linux-arm-kernel@lists.infradead.org" , "devicetree@vger.kernel.org" , Stephan van Schaik , Javier Martinez Canillas , Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Russell King , Ben Dooks , Kukjin Kim , LKML Am 02.08.2014 07:15, schrieb Doug Anderson: > On Thu, Jul 31, 2014 at 12:40 PM, Tomasz Figa = wrote: >> On 31.07.2014 21:20, Andreas F=C3=A4rber wrote: >>> Am 31.07.2014 21:05, schrieb Tomasz Figa: >>>> On 31.07.2014 18:08, Andreas F=C3=A4rber wrote: >>>>> +&sd1_clk { >>>>> + samsung,pin-drv =3D <0>; >>>>> +}; >>>>> + >>>>> +&sd1_cmd { >>>>> + samsung,pin-pud =3D <3>; >>>>> + samsung,pin-drv =3D <0>; >>>>> +}; >>>>> + >>>>> +&sd1_cd { >>>>> + samsung,pin-drv =3D <0>; >>>>> +}; >>>>> + >>>>> +&sd1_bus4 { >>>>> + samsung,pin-drv =3D <0>; >>>>> +}; >>>> >>>> Here generic settings are being overridden, so it might be a good = idea >>>> to explain why, like with i2c pull-up above. >>> >>> Snow does not have an explanation either, so please suggest what co= mment >>> you'd like to see. Consider me just a user with no specs. :) >> >> Doug, Vincent, someone else? >=20 > The comment is just in a different place--it's in the dw_mmc node. > Probably belongs here, though: >=20 > /* > * Wifi is a SiP, so can keep drive strengths low > * to reduce EMI. > */ >=20 I did copy such a comment for the MMC node in v5, still present in v6. Andreas > I guess the cmd line isn't documented. There is no external pull on > the command line and on most boards there is one. Our hardware guys > thought we didn't need it and apparently we don't... --=20 SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 N=C3=BCrnberg, Germany GF: Jeff Hawn, Jennifer Guild, Felix Imend=C3=B6rffer; HRB 16746 AG N=C3= =BCrnberg From mboxrd@z Thu Jan 1 00:00:00 1970 From: afaerber@suse.de (=?UTF-8?B?QW5kcmVhcyBGw6RyYmVy?=) Date: Sat, 02 Aug 2014 09:49:53 +0200 Subject: [PATCH v4 4/4] ARM: dts: Add exynos5250-spring device tree In-Reply-To: References: <1406822910-6255-1-git-send-email-afaerber@suse.de> <1406822910-6255-5-git-send-email-afaerber@suse.de> <53DA936E.9060004@gmail.com> <53DA9709.10602@suse.de> <53DA9BA9.60808@gmail.com> Message-ID: <53DC9821.3010108@suse.de> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Am 02.08.2014 07:15, schrieb Doug Anderson: > On Thu, Jul 31, 2014 at 12:40 PM, Tomasz Figa wrote: >> On 31.07.2014 21:20, Andreas F?rber wrote: >>> Am 31.07.2014 21:05, schrieb Tomasz Figa: >>>> On 31.07.2014 18:08, Andreas F?rber wrote: >>>>> +&sd1_clk { >>>>> + samsung,pin-drv = <0>; >>>>> +}; >>>>> + >>>>> +&sd1_cmd { >>>>> + samsung,pin-pud = <3>; >>>>> + samsung,pin-drv = <0>; >>>>> +}; >>>>> + >>>>> +&sd1_cd { >>>>> + samsung,pin-drv = <0>; >>>>> +}; >>>>> + >>>>> +&sd1_bus4 { >>>>> + samsung,pin-drv = <0>; >>>>> +}; >>>> >>>> Here generic settings are being overridden, so it might be a good idea >>>> to explain why, like with i2c pull-up above. >>> >>> Snow does not have an explanation either, so please suggest what comment >>> you'd like to see. Consider me just a user with no specs. :) >> >> Doug, Vincent, someone else? > > The comment is just in a different place--it's in the dw_mmc node. > Probably belongs here, though: > > /* > * Wifi is a SiP, so can keep drive strengths low > * to reduce EMI. > */ > I did copy such a comment for the MMC node in v5, still present in v6. Andreas > I guess the cmd line isn't documented. There is no external pull on > the command line and on most boards there is one. Our hardware guys > thought we didn't need it and apparently we don't... -- SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 N?rnberg, Germany GF: Jeff Hawn, Jennifer Guild, Felix Imend?rffer; HRB 16746 AG N?rnberg