Re: [PATCH 1/2] arm/mx5: parse iomuxc pad configuratoin from device tree

2011-08-07 Thread Sascha Hauer
On Thu, Aug 04, 2011 at 06:07:15PM -0500, Matt Sealey wrote: If you think the solution is not so great due to the complexity of describing the IOMUX settings, including the pad definitions as binary blobs or so such that Linux can read them out, please feel free to take the hint and go nag

Re: [PATCH 1/2] arm/mx5: parse iomuxc pad configuratoin from device tree

2011-08-07 Thread Russell King - ARM Linux
On Fri, Aug 05, 2011 at 01:36:56PM -0500, Matt Sealey wrote: On Fri, Aug 5, 2011 at 2:07 AM, David Brown dav...@codeaurora.org wrote: On Thu, Aug 04, 2011 at 06:07:15PM -0500, Matt Sealey wrote: Hi Grant, Shawn, On Mon, Jul 25, 2011 at 3:46 PM, Grant Likely grant.lik...@secretlab.ca

Re: [PATCH 1/2] arm/mx5: parse iomuxc pad configuratoin from device tree

2011-08-05 Thread David Brown
On Thu, Aug 04, 2011 at 06:07:15PM -0500, Matt Sealey wrote: Hi Grant, Shawn, On Mon, Jul 25, 2011 at 3:46 PM, Grant Likely grant.lik...@secretlab.ca wrote: This could get really verbose in a really big hurry.  Fortunately the dtb format is sophisticated enough to only store each unique

Re: [PATCH 1/2] arm/mx5: parse iomuxc pad configuratoin from device tree

2011-08-05 Thread Scott Wood
On 08/05/2011 01:36 PM, Matt Sealey wrote: On Fri, Aug 5, 2011 at 2:07 AM, David Brown dav...@codeaurora.org wrote: On Thu, Aug 04, 2011 at 06:07:15PM -0500, Matt Sealey wrote: Hi Grant, Shawn, On Mon, Jul 25, 2011 at 3:46 PM, Grant Likely grant.lik...@secretlab.ca wrote: This could get

Re: [PATCH 1/2] arm/mx5: parse iomuxc pad configuratoin from device tree

2011-08-05 Thread David Brown
On Fri, Aug 05, 2011 at 03:26:29PM -0500, Scott Wood wrote: Yes, it puts the onus of the work on the firmware guys, but they're the ones writing the device trees for their hardware anyway. Sometimes. I'm not even sure that is going to be common. At least our setup is going to have the

Re: [PATCH 1/2] arm/mx5: parse iomuxc pad configuratoin from device tree

2011-08-05 Thread Matt Sealey
On Fri, Aug 5, 2011 at 3:36 PM, David Brown dav...@codeaurora.org wrote: On Fri, Aug 05, 2011 at 03:26:29PM -0500, Scott Wood wrote: Yes, it puts the onus of the work on the firmware guys, but they're the ones writing the device trees for their hardware anyway. Sometimes. I'm not even

Re: [PATCH 1/2] arm/mx5: parse iomuxc pad configuratoin from device tree

2011-08-05 Thread Scott Wood
On 08/05/2011 04:29 PM, Matt Sealey wrote: On Fri, Aug 5, 2011 at 3:36 PM, David Brown dav...@codeaurora.org wrote: On Fri, Aug 05, 2011 at 03:26:29PM -0500, Scott Wood wrote: Yes, it puts the onus of the work on the firmware guys, but they're the ones writing the device trees for their

Re: [PATCH 1/2] arm/mx5: parse iomuxc pad configuratoin from device tree

2011-08-05 Thread Grant Likely
On Thu, Aug 04, 2011 at 06:07:15PM -0500, Matt Sealey wrote: Hi Grant, Shawn, On Mon, Jul 25, 2011 at 3:46 PM, Grant Likely grant.lik...@secretlab.ca wrote: This could get really verbose in a really big hurry.  Fortunately the dtb format is sophisticated enough to only store each

Re: [PATCH 1/2] arm/mx5: parse iomuxc pad configuratoin from device tree

2011-08-05 Thread Mitch Bradley
On 8/5/2011 12:58 PM, Grant Likely wrote: On Thu, Aug 04, 2011 at 06:07:15PM -0500, Matt Sealey wrote: Hi Grant, Shawn, On Mon, Jul 25, 2011 at 3:46 PM, Grant Likelygrant.lik...@secretlab.ca wrote: This could get really verbose in a really big hurry. Fortunately the dtb format is

Re: [PATCH 1/2] arm/mx5: parse iomuxc pad configuratoin from device tree

2011-08-05 Thread Mark Brown
On Fri, Aug 05, 2011 at 01:31:10PM -1000, Mitch Bradley wrote: On 8/5/2011 12:58 PM, Grant Likely wrote: We've had this argument before. There are many use cases where the firmware simply cannot be relied upon to do the right thing. I am a firmware engineer who tries very hard to deliver

Re: [PATCH 1/2] arm/mx5: parse iomuxc pad configuratoin from device tree

2011-08-04 Thread Matt Sealey
Hi Grant, Shawn, On Mon, Jul 25, 2011 at 3:46 PM, Grant Likely grant.lik...@secretlab.ca wrote: This could get really verbose in a really big hurry.  Fortunately the dtb format is sophisticated enough to only store each unique property name once, so the data shouldn't be huge, but it is

Re: [PATCH 1/2] arm/mx5: parse iomuxc pad configuratoin from device tree

2011-07-31 Thread Grant Likely
On Tue, Jul 26, 2011 at 10:43:55AM +0800, Shawn Guo wrote: On Mon, Jul 25, 2011 at 02:46:30PM -0600, Grant Likely wrote: On Mon, Jul 25, 2011 at 11:07:46PM +0800, Shawn Guo wrote: The current linux code has each pin config simply a u64. Now, an engineer certainly wouldn't be asked to write

Re: [PATCH 1/2] arm/mx5: parse iomuxc pad configuratoin from device tree

2011-07-26 Thread Sascha Hauer
On Tue, Jul 26, 2011 at 10:43:55AM +0800, Shawn Guo wrote: On Mon, Jul 25, 2011 at 02:46:30PM -0600, Grant Likely wrote: On Mon, Jul 25, 2011 at 11:07:46PM +0800, Shawn Guo wrote: It adds function mxc_iomuxc_dt_init() to parse iomuxc pad configuration Specifying string in dts, and then

Re: [PATCH 1/2] arm/mx5: parse iomuxc pad configuratoin from device tree

2011-07-26 Thread Sascha Hauer
On Mon, Jul 25, 2011 at 11:07:46PM +0800, Shawn Guo wrote: It adds function mxc_iomuxc_dt_init() to parse iomuxc pad configuration from device tree. Signed-off-by: Shawn Guo shawn@linaro.org Cc: Grant Likely grant.lik...@secretlab.ca Cc: Sascha Hauer s.ha...@pengutronix.de ---

Re: [PATCH 1/2] arm/mx5: parse iomuxc pad configuratoin from device tree

2011-07-26 Thread Sascha Hauer
On Mon, Jul 25, 2011 at 11:07:46PM +0800, Shawn Guo wrote: It adds function mxc_iomuxc_dt_init() to parse iomuxc pad configuration from device tree. Signed-off-by: Shawn Guo shawn@linaro.org Cc: Grant Likely grant.lik...@secretlab.ca Cc: Sascha Hauer s.ha...@pengutronix.de ---

Re: [PATCH 1/2] arm/mx5: parse iomuxc pad configuratoin from device tree

2011-07-26 Thread Eric Miao
On Tue, Jul 26, 2011 at 4:46 AM, Grant Likely grant.lik...@secretlab.ca wrote: On Mon, Jul 25, 2011 at 11:07:46PM +0800, Shawn Guo wrote: It adds function mxc_iomuxc_dt_init() to parse iomuxc pad configuration from device tree. Signed-off-by: Shawn Guo shawn@linaro.org Cc: Grant Likely

Re: [PATCH 1/2] arm/mx5: parse iomuxc pad configuratoin from device tree

2011-07-26 Thread Shawn Guo
On Tue, Jul 26, 2011 at 08:29:32AM +0200, Sascha Hauer wrote: On Tue, Jul 26, 2011 at 10:43:55AM +0800, Shawn Guo wrote: On Mon, Jul 25, 2011 at 02:46:30PM -0600, Grant Likely wrote: On Mon, Jul 25, 2011 at 11:07:46PM +0800, Shawn Guo wrote: It adds function mxc_iomuxc_dt_init() to parse

Re: [PATCH 1/2] arm/mx5: parse iomuxc pad configuratoin from device tree

2011-07-26 Thread Shawn Guo
On Tue, Jul 26, 2011 at 08:31:44AM +0200, Sascha Hauer wrote: On Mon, Jul 25, 2011 at 11:07:46PM +0800, Shawn Guo wrote: It adds function mxc_iomuxc_dt_init() to parse iomuxc pad configuration from device tree. Signed-off-by: Shawn Guo shawn@linaro.org Cc: Grant Likely

Re: [PATCH 1/2] arm/mx5: parse iomuxc pad configuratoin from device tree

2011-07-26 Thread Shawn Guo
On Tue, Jul 26, 2011 at 08:39:14AM +0200, Sascha Hauer wrote: On Mon, Jul 25, 2011 at 11:07:46PM +0800, Shawn Guo wrote: It adds function mxc_iomuxc_dt_init() to parse iomuxc pad configuration from device tree. Signed-off-by: Shawn Guo shawn@linaro.org Cc: Grant Likely

[PATCH 1/2] arm/mx5: parse iomuxc pad configuratoin from device tree

2011-07-25 Thread Shawn Guo
It adds function mxc_iomuxc_dt_init() to parse iomuxc pad configuration from device tree. Signed-off-by: Shawn Guo shawn@linaro.org Cc: Grant Likely grant.lik...@secretlab.ca Cc: Sascha Hauer s.ha...@pengutronix.de --- .../devicetree/bindings/arm/fsl/iomuxc.txt | 47 +

Re: [PATCH 1/2] arm/mx5: parse iomuxc pad configuratoin from device tree

2011-07-25 Thread Grant Likely
On Mon, Jul 25, 2011 at 11:07:46PM +0800, Shawn Guo wrote: It adds function mxc_iomuxc_dt_init() to parse iomuxc pad configuration from device tree. Signed-off-by: Shawn Guo shawn@linaro.org Cc: Grant Likely grant.lik...@secretlab.ca Cc: Sascha Hauer s.ha...@pengutronix.de ---

Re: [PATCH 1/2] arm/mx5: parse iomuxc pad configuratoin from device tree

2011-07-25 Thread Shawn Guo
On Mon, Jul 25, 2011 at 02:46:30PM -0600, Grant Likely wrote: On Mon, Jul 25, 2011 at 11:07:46PM +0800, Shawn Guo wrote: It adds function mxc_iomuxc_dt_init() to parse iomuxc pad configuration from device tree. Signed-off-by: Shawn Guo shawn@linaro.org Cc: Grant Likely