Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-30 Thread Mark Brown
On Sat, May 28, 2016 at 10:57:33PM +0200, Christer Weinigel wrote: > On 05/27/2016 08:36 PM, Mark Brown wrote: > > It is reasonable to provide and document something here but when there's > > some fairly simple and obvious better things we could be doing it should > > be those rather than the

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-30 Thread Mark Brown
On Sat, May 28, 2016 at 10:57:33PM +0200, Christer Weinigel wrote: > On 05/27/2016 08:36 PM, Mark Brown wrote: > > It is reasonable to provide and document something here but when there's > > some fairly simple and obvious better things we could be doing it should > > be those rather than the

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-28 Thread Christer Weinigel
On 05/27/2016 08:36 PM, Mark Brown wrote: Personally the way I parse this situation is that the kernel is taking a look at what's in the DT and making an effort to present it usefully in the running systems. Fixing our current interpretation in stone as a supported thing when we don't have to

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-28 Thread Christer Weinigel
On 05/27/2016 08:36 PM, Mark Brown wrote: Personally the way I parse this situation is that the kernel is taking a look at what's in the DT and making an effort to present it usefully in the running systems. Fixing our current interpretation in stone as a supported thing when we don't have to

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-27 Thread Mark Brown
On Wed, May 25, 2016 at 11:46:35AM -0700, Frank Rowand wrote: > On 5/25/2016 10:48 AM, Mark Brown wrote: > > Sometimes the best thing to do is remove the behaviour, some of these > Yes. And I have not formed an opinion on whether the existing > behavior should be kept, deprecated, or removed.

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-27 Thread Mark Brown
On Wed, May 25, 2016 at 11:46:35AM -0700, Frank Rowand wrote: > On 5/25/2016 10:48 AM, Mark Brown wrote: > > Sometimes the best thing to do is remove the behaviour, some of these > Yes. And I have not formed an opinion on whether the existing > behavior should be kept, deprecated, or removed.

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-27 Thread Mark Brown
On Thu, May 26, 2016 at 11:04:18PM +0200, Christer Weinigel wrote: > My point is that I don't think it would be doable to get every > devicetree file out there into the mainline kernel; it's not even > desirable. devicetree files for custom platforms are a lot like > userspace applications, the

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-27 Thread Mark Brown
On Thu, May 26, 2016 at 11:04:18PM +0200, Christer Weinigel wrote: > My point is that I don't think it would be doable to get every > devicetree file out there into the mainline kernel; it's not even > desirable. devicetree files for custom platforms are a lot like > userspace applications, the

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-26 Thread Christer Weinigel
On 05/26/2016 08:47 PM, Mark Brown wrote: > On Thu, May 26, 2016 at 12:58:22PM +0200, Christer Weinigel wrote: > >> One of the main drivers behind devicetree was that Linus got fed >> up with the churn for all platform device changes in arch/arm. >> I faintly recall him writing that he would be

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-26 Thread Christer Weinigel
On 05/26/2016 08:47 PM, Mark Brown wrote: > On Thu, May 26, 2016 at 12:58:22PM +0200, Christer Weinigel wrote: > >> One of the main drivers behind devicetree was that Linus got fed >> up with the churn for all platform device changes in arch/arm. >> I faintly recall him writing that he would be

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-26 Thread Mark Brown
On Thu, May 26, 2016 at 12:58:22PM +0200, Christer Weinigel wrote: > One of the main drivers behind devicetree was that Linus got fed up > with the churn for all platform device changes in arch/arm. I faintly > recall him writing that he would be rather unhappy if that just got > replaced with

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-26 Thread Mark Brown
On Thu, May 26, 2016 at 12:58:22PM +0200, Christer Weinigel wrote: > One of the main drivers behind devicetree was that Linus got fed up > with the churn for all platform device changes in arch/arm. I faintly > recall him writing that he would be rather unhappy if that just got > replaced with

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-26 Thread Christer Weinigel
On 05/26/2016 12:07 PM, Mark Brown wrote: > I think Rob's referring to the fact that there are no in tree DTs > that use this feature - all the aliases for SPI controllers in > mainline are string based. One of the main drivers behind devicetree was that Linus got fed up with the churn for all

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-26 Thread Christer Weinigel
On 05/26/2016 12:07 PM, Mark Brown wrote: > I think Rob's referring to the fact that there are no in tree DTs > that use this feature - all the aliases for SPI controllers in > mainline are string based. One of the main drivers behind devicetree was that Linus got fed up with the churn for all

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-26 Thread Mark Brown
On Thu, May 26, 2016 at 03:56:11AM +0200, Christer Weinigel wrote: > On 05/26/2016 03:44 AM, Rob Herring wrote: > > Lovely. "Here's something that's simple and useful for users. Let's > > break it". What part of "we do not break userspace" do you not > > understand? Because that would be a user

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-26 Thread Mark Brown
On Thu, May 26, 2016 at 03:56:11AM +0200, Christer Weinigel wrote: > On 05/26/2016 03:44 AM, Rob Herring wrote: > > Lovely. "Here's something that's simple and useful for users. Let's > > break it". What part of "we do not break userspace" do you not > > understand? Because that would be a user

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-26 Thread Geert Uytterhoeven
On Wed, May 25, 2016 at 8:44 PM, Rob Herring wrote: > On Tue, May 24, 2016 at 06:41:41PM +0100, Mark Rutland wrote: >> On Tue, May 24, 2016 at 06:39:20PM +0200, Christer Weinigel wrote: >> > Document how to use devicetree aliases to assign a stable >> > bus number to a spi bus.

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-26 Thread Geert Uytterhoeven
On Wed, May 25, 2016 at 8:44 PM, Rob Herring wrote: > On Tue, May 24, 2016 at 06:41:41PM +0100, Mark Rutland wrote: >> On Tue, May 24, 2016 at 06:39:20PM +0200, Christer Weinigel wrote: >> > Document how to use devicetree aliases to assign a stable >> > bus number to a spi bus. >> > >> >

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Christer Weinigel
On 05/26/2016 03:44 AM, Rob Herring wrote: Lovely. "Here's something that's simple and useful for users. Let's break it". What part of "we do not break userspace" do you not understand? Because that would be a user visible change. The other saying is "if it is not upstream, it doesn't exist."

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Christer Weinigel
On 05/26/2016 03:44 AM, Rob Herring wrote: Lovely. "Here's something that's simple and useful for users. Let's break it". What part of "we do not break userspace" do you not understand? Because that would be a user visible change. The other saying is "if it is not upstream, it doesn't exist."

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Rob Herring
On Wed, May 25, 2016 at 8:10 PM, Christer Weinigel wrote: > On 05/25/2016 08:44 PM, Mark Brown wrote: >> On Wed, May 25, 2016 at 11:06:46AM -0700, Frank Rowand wrote: >>> On 5/25/2016 10:49 AM, Rob Herring wrote: >> Things get undocumented all the time when we deprecate

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Rob Herring
On Wed, May 25, 2016 at 8:10 PM, Christer Weinigel wrote: > On 05/25/2016 08:44 PM, Mark Brown wrote: >> On Wed, May 25, 2016 at 11:06:46AM -0700, Frank Rowand wrote: >>> On 5/25/2016 10:49 AM, Rob Herring wrote: >> Things get undocumented all the time when we deprecate them. >> >>> If it is

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Christer Weinigel
On 05/25/2016 08:44 PM, Mark Brown wrote: > On Wed, May 25, 2016 at 11:06:46AM -0700, Frank Rowand wrote: >> On 5/25/2016 10:49 AM, Rob Herring wrote: > >>> Things get undocumented all the time when we deprecate them. > >> If it is deprecated then it should be documented as deprecated >> so

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Christer Weinigel
On 05/25/2016 08:44 PM, Mark Brown wrote: > On Wed, May 25, 2016 at 11:06:46AM -0700, Frank Rowand wrote: >> On 5/25/2016 10:49 AM, Rob Herring wrote: > >>> Things get undocumented all the time when we deprecate them. > >> If it is deprecated then it should be documented as deprecated >> so

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Mark Brown
On Wed, May 25, 2016 at 01:44:21PM -0500, Rob Herring wrote: > For SPI, I think we should use "label" which reflects a name that is > defined by the h/w design and is meaningful to the user. Then perhaps > the device becomes "/dev/spi/by-name//spidev.0" or simply > "/dev/spidev-.0". I agree -

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Mark Brown
On Wed, May 25, 2016 at 01:44:21PM -0500, Rob Herring wrote: > For SPI, I think we should use "label" which reflects a name that is > defined by the h/w design and is meaningful to the user. Then perhaps > the device becomes "/dev/spi/by-name//spidev.0" or simply > "/dev/spidev-.0". I agree -

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Mark Brown
On Wed, May 25, 2016 at 11:06:46AM -0700, Frank Rowand wrote: > On 5/25/2016 10:49 AM, Rob Herring wrote: > > Things get undocumented all the time when we deprecate them. > If it is deprecated then it should be documented as deprecated so > people do not attempt to use it. Or we could just

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Mark Brown
On Wed, May 25, 2016 at 11:06:46AM -0700, Frank Rowand wrote: > On 5/25/2016 10:49 AM, Rob Herring wrote: > > Things get undocumented all the time when we deprecate them. > If it is deprecated then it should be documented as deprecated so > people do not attempt to use it. Or we could just

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Frank Rowand
On 5/25/2016 10:48 AM, Mark Brown wrote: > On Wed, May 25, 2016 at 08:32:51AM -0700, Frank Rowand wrote: >> On 5/25/2016 2:20 AM, Mark Rutland wrote: > >>> Linux for legacy reasons, documenting it as a binding is not necessarily >>> in anyone's best interest. If we want to document it, we may

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Frank Rowand
On 5/25/2016 10:48 AM, Mark Brown wrote: > On Wed, May 25, 2016 at 08:32:51AM -0700, Frank Rowand wrote: >> On 5/25/2016 2:20 AM, Mark Rutland wrote: > >>> Linux for legacy reasons, documenting it as a binding is not necessarily >>> in anyone's best interest. If we want to document it, we may

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Rob Herring
On Tue, May 24, 2016 at 06:41:41PM +0100, Mark Rutland wrote: > On Tue, May 24, 2016 at 06:39:20PM +0200, Christer Weinigel wrote: > > Document how to use devicetree aliases to assign a stable > > bus number to a spi bus. > > > > Signed-off-by: Christer Weinigel > > > >

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Rob Herring
On Tue, May 24, 2016 at 06:41:41PM +0100, Mark Rutland wrote: > On Tue, May 24, 2016 at 06:39:20PM +0200, Christer Weinigel wrote: > > Document how to use devicetree aliases to assign a stable > > bus number to a spi bus. > > > > Signed-off-by: Christer Weinigel > > > > --- > > > > Trivial

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Frank Rowand
On 5/25/2016 10:49 AM, Rob Herring wrote: > On Tue, May 24, 2016 at 04:34:50PM -0700, Frank Rowand wrote: >> On 5/24/2016 11:32 AM, Mark Brown wrote: >>> On Tue, May 24, 2016 at 08:03:48PM +0200, Christer Weinigel wrote: On 05/24/2016 07:20 PM, Mark Brown wrote: >>> > I'm not sure this is

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Frank Rowand
On 5/25/2016 10:49 AM, Rob Herring wrote: > On Tue, May 24, 2016 at 04:34:50PM -0700, Frank Rowand wrote: >> On 5/24/2016 11:32 AM, Mark Brown wrote: >>> On Tue, May 24, 2016 at 08:03:48PM +0200, Christer Weinigel wrote: On 05/24/2016 07:20 PM, Mark Brown wrote: >>> > I'm not sure this is

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Mark Brown
On Wed, May 25, 2016 at 12:49:32PM -0500, Rob Herring wrote: > On Tue, May 24, 2016 at 04:34:50PM -0700, Frank Rowand wrote: > > It is in the kernel, it is appropriate to document it. > Things get undocumented all the time when we deprecate them. There's also the X.org approach of breaking

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Mark Brown
On Wed, May 25, 2016 at 12:49:32PM -0500, Rob Herring wrote: > On Tue, May 24, 2016 at 04:34:50PM -0700, Frank Rowand wrote: > > It is in the kernel, it is appropriate to document it. > Things get undocumented all the time when we deprecate them. There's also the X.org approach of breaking

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Mark Brown
On Wed, May 25, 2016 at 04:59:50PM +0100, Mark Rutland wrote: > e.g. stating that this describes a well-defined system-specific bus > number as documented in a manual, with a note regarding Linux behaviour > is better simply describing the Linux behaviour. If it means anything it's really a

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Mark Brown
On Wed, May 25, 2016 at 04:59:50PM +0100, Mark Rutland wrote: > e.g. stating that this describes a well-defined system-specific bus > number as documented in a manual, with a note regarding Linux behaviour > is better simply describing the Linux behaviour. If it means anything it's really a

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Mark Brown
On Wed, May 25, 2016 at 08:32:51AM -0700, Frank Rowand wrote: > On 5/25/2016 2:20 AM, Mark Rutland wrote: > > Linux for legacy reasons, documenting it as a binding is not necessarily > > in anyone's best interest. If we want to document it, we may want to > > mark it as deprecated, with a pointer

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Mark Brown
On Wed, May 25, 2016 at 08:32:51AM -0700, Frank Rowand wrote: > On 5/25/2016 2:20 AM, Mark Rutland wrote: > > Linux for legacy reasons, documenting it as a binding is not necessarily > > in anyone's best interest. If we want to document it, we may want to > > mark it as deprecated, with a pointer

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Rob Herring
On Tue, May 24, 2016 at 04:34:50PM -0700, Frank Rowand wrote: > On 5/24/2016 11:32 AM, Mark Brown wrote: > > On Tue, May 24, 2016 at 08:03:48PM +0200, Christer Weinigel wrote: > >> On 05/24/2016 07:20 PM, Mark Brown wrote: > > > >>> I'm not sure this is something we want to support at all, I

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Rob Herring
On Tue, May 24, 2016 at 04:34:50PM -0700, Frank Rowand wrote: > On 5/24/2016 11:32 AM, Mark Brown wrote: > > On Tue, May 24, 2016 at 08:03:48PM +0200, Christer Weinigel wrote: > >> On 05/24/2016 07:20 PM, Mark Brown wrote: > > > >>> I'm not sure this is something we want to support at all, I

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Frank Rowand
On 5/25/2016 9:06 AM, Mark Rutland wrote: > On Wed, May 25, 2016 at 08:25:44AM -0700, Frank Rowand wrote: >> On 5/24/2016 10:41 AM, Mark Rutland wrote: >>> On Tue, May 24, 2016 at 06:39:20PM +0200, Christer Weinigel wrote: +Normally SPI buses are assigned dynamic bus numbers starting at 32766

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Frank Rowand
On 5/25/2016 9:06 AM, Mark Rutland wrote: > On Wed, May 25, 2016 at 08:25:44AM -0700, Frank Rowand wrote: >> On 5/24/2016 10:41 AM, Mark Rutland wrote: >>> On Tue, May 24, 2016 at 06:39:20PM +0200, Christer Weinigel wrote: +Normally SPI buses are assigned dynamic bus numbers starting at 32766

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Frank Rowand
On 5/25/2016 8:59 AM, Mark Rutland wrote: > On Wed, May 25, 2016 at 08:32:51AM -0700, Frank Rowand wrote: >> On 5/25/2016 2:20 AM, Mark Rutland wrote: >>> On Tue, May 24, 2016 at 01:41:26PM -0700, Frank Rowand wrote: On 5/24/2016 10:41 AM, Mark Rutland wrote: > On Tue, May 24, 2016 at

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Frank Rowand
On 5/25/2016 8:59 AM, Mark Rutland wrote: > On Wed, May 25, 2016 at 08:32:51AM -0700, Frank Rowand wrote: >> On 5/25/2016 2:20 AM, Mark Rutland wrote: >>> On Tue, May 24, 2016 at 01:41:26PM -0700, Frank Rowand wrote: On 5/24/2016 10:41 AM, Mark Rutland wrote: > On Tue, May 24, 2016 at

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Mark Rutland
On Wed, May 25, 2016 at 08:25:44AM -0700, Frank Rowand wrote: > On 5/24/2016 10:41 AM, Mark Rutland wrote: > > On Tue, May 24, 2016 at 06:39:20PM +0200, Christer Weinigel wrote: > >> +Normally SPI buses are assigned dynamic bus numbers starting at 32766 > >> +and counting downwards. It is

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Mark Rutland
On Wed, May 25, 2016 at 08:25:44AM -0700, Frank Rowand wrote: > On 5/24/2016 10:41 AM, Mark Rutland wrote: > > On Tue, May 24, 2016 at 06:39:20PM +0200, Christer Weinigel wrote: > >> +Normally SPI buses are assigned dynamic bus numbers starting at 32766 > >> +and counting downwards. It is

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Mark Rutland
On Wed, May 25, 2016 at 08:32:51AM -0700, Frank Rowand wrote: > On 5/25/2016 2:20 AM, Mark Rutland wrote: > > On Tue, May 24, 2016 at 01:41:26PM -0700, Frank Rowand wrote: > >> On 5/24/2016 10:41 AM, Mark Rutland wrote: > >>> On Tue, May 24, 2016 at 06:39:20PM +0200, Christer Weinigel wrote: >

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Mark Rutland
On Wed, May 25, 2016 at 08:32:51AM -0700, Frank Rowand wrote: > On 5/25/2016 2:20 AM, Mark Rutland wrote: > > On Tue, May 24, 2016 at 01:41:26PM -0700, Frank Rowand wrote: > >> On 5/24/2016 10:41 AM, Mark Rutland wrote: > >>> On Tue, May 24, 2016 at 06:39:20PM +0200, Christer Weinigel wrote: >

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Frank Rowand
On 5/25/2016 2:20 AM, Mark Rutland wrote: > On Tue, May 24, 2016 at 01:41:26PM -0700, Frank Rowand wrote: >> On 5/24/2016 10:41 AM, Mark Rutland wrote: >>> On Tue, May 24, 2016 at 06:39:20PM +0200, Christer Weinigel wrote: Document how to use devicetree aliases to assign a stable bus

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Frank Rowand
On 5/25/2016 2:20 AM, Mark Rutland wrote: > On Tue, May 24, 2016 at 01:41:26PM -0700, Frank Rowand wrote: >> On 5/24/2016 10:41 AM, Mark Rutland wrote: >>> On Tue, May 24, 2016 at 06:39:20PM +0200, Christer Weinigel wrote: Document how to use devicetree aliases to assign a stable bus

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Frank Rowand
On 5/24/2016 10:41 AM, Mark Rutland wrote: > On Tue, May 24, 2016 at 06:39:20PM +0200, Christer Weinigel wrote: >> Document how to use devicetree aliases to assign a stable >> bus number to a spi bus. >> >> Signed-off-by: Christer Weinigel >> >> --- >> >> Trivial

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Frank Rowand
On 5/24/2016 10:41 AM, Mark Rutland wrote: > On Tue, May 24, 2016 at 06:39:20PM +0200, Christer Weinigel wrote: >> Document how to use devicetree aliases to assign a stable >> bus number to a spi bus. >> >> Signed-off-by: Christer Weinigel >> >> --- >> >> Trivial documentation change. >> >> Not

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Mark Brown
On Wed, May 25, 2016 at 01:34:24PM +0100, Mark Rutland wrote: > On Wed, May 25, 2016 at 01:20:04PM +0200, Christer Weinigel wrote: > > Does everything have to be so damn difficult? > With the varied hardware that exists, and the constant expansion of the > set of things which exist, there is

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Mark Brown
On Wed, May 25, 2016 at 01:34:24PM +0100, Mark Rutland wrote: > On Wed, May 25, 2016 at 01:20:04PM +0200, Christer Weinigel wrote: > > Does everything have to be so damn difficult? > With the varied hardware that exists, and the constant expansion of the > set of things which exist, there is

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Mark Brown
On Wed, May 25, 2016 at 01:19:24PM +0100, Mark Rutland wrote: > On Tue, May 24, 2016 at 08:57:06PM +0200, Christer Weinigel wrote: > > It's bloody convenient. I'm working with a Zync board right now where > > we have multiple SPI ports. Being able to label the ports on the > > board spi1, spi2

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Mark Brown
On Wed, May 25, 2016 at 01:19:24PM +0100, Mark Rutland wrote: > On Tue, May 24, 2016 at 08:57:06PM +0200, Christer Weinigel wrote: > > It's bloody convenient. I'm working with a Zync board right now where > > we have multiple SPI ports. Being able to label the ports on the > > board spi1, spi2

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Mark Rutland
On Wed, May 25, 2016 at 01:20:04PM +0200, Christer Weinigel wrote: > On 05/25/2016 12:38 PM, Mark Brown wrote: > > On Wed, May 25, 2016 at 10:20:34AM +0100, Mark Rutland wrote: > >> On Tue, May 24, 2016 at 01:41:26PM -0700, Frank Rowand wrote: > > > >>> The code and behavior is in the Linux

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Mark Rutland
On Wed, May 25, 2016 at 01:20:04PM +0200, Christer Weinigel wrote: > On 05/25/2016 12:38 PM, Mark Brown wrote: > > On Wed, May 25, 2016 at 10:20:34AM +0100, Mark Rutland wrote: > >> On Tue, May 24, 2016 at 01:41:26PM -0700, Frank Rowand wrote: > > > >>> The code and behavior is in the Linux

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Mark Brown
On Tue, May 24, 2016 at 08:57:06PM +0200, Christer Weinigel wrote: > On 05/24/2016 08:32 PM, Mark Brown wrote: > > I'm not clear that we want to document this at all since I am not > > clear that there is a sensible use case for doing it. I did ask > > for one but you've not articulated one in

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Mark Brown
On Tue, May 24, 2016 at 08:57:06PM +0200, Christer Weinigel wrote: > On 05/24/2016 08:32 PM, Mark Brown wrote: > > I'm not clear that we want to document this at all since I am not > > clear that there is a sensible use case for doing it. I did ask > > for one but you've not articulated one in

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Mark Rutland
On Tue, May 24, 2016 at 08:57:06PM +0200, Christer Weinigel wrote: > On 05/24/2016 08:32 PM, Mark Brown wrote: > > On Tue, May 24, 2016 at 08:03:48PM +0200, Christer Weinigel wrote: > >> On 05/24/2016 07:20 PM, Mark Brown wrote: > > > >>> I'm not sure this is something we want to support at all,

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Mark Rutland
On Tue, May 24, 2016 at 08:57:06PM +0200, Christer Weinigel wrote: > On 05/24/2016 08:32 PM, Mark Brown wrote: > > On Tue, May 24, 2016 at 08:03:48PM +0200, Christer Weinigel wrote: > >> On 05/24/2016 07:20 PM, Mark Brown wrote: > > > >>> I'm not sure this is something we want to support at all,

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Christer Weinigel
On 05/25/2016 12:38 PM, Mark Brown wrote: > On Wed, May 25, 2016 at 10:20:34AM +0100, Mark Rutland wrote: >> On Tue, May 24, 2016 at 01:41:26PM -0700, Frank Rowand wrote: > >>> The code and behavior is in the Linux kernel. It should be >>> visible in the documentation instead of being a big

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Christer Weinigel
On 05/25/2016 12:38 PM, Mark Brown wrote: > On Wed, May 25, 2016 at 10:20:34AM +0100, Mark Rutland wrote: >> On Tue, May 24, 2016 at 01:41:26PM -0700, Frank Rowand wrote: > >>> The code and behavior is in the Linux kernel. It should be >>> visible in the documentation instead of being a big

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Mark Brown
On Wed, May 25, 2016 at 10:20:34AM +0100, Mark Rutland wrote: > On Tue, May 24, 2016 at 01:41:26PM -0700, Frank Rowand wrote: > > The code and behavior is in the Linux kernel. It should be visible in > > the documentation instead of being a big mystery of how it works. > As above, I don't

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Mark Brown
On Wed, May 25, 2016 at 10:20:34AM +0100, Mark Rutland wrote: > On Tue, May 24, 2016 at 01:41:26PM -0700, Frank Rowand wrote: > > The code and behavior is in the Linux kernel. It should be visible in > > the documentation instead of being a big mystery of how it works. > As above, I don't

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Mark Rutland
On Tue, May 24, 2016 at 01:41:26PM -0700, Frank Rowand wrote: > On 5/24/2016 10:41 AM, Mark Rutland wrote: > > On Tue, May 24, 2016 at 06:39:20PM +0200, Christer Weinigel wrote: > >> Document how to use devicetree aliases to assign a stable > >> bus number to a spi bus. > >> > >> Signed-off-by:

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Mark Rutland
On Tue, May 24, 2016 at 01:41:26PM -0700, Frank Rowand wrote: > On 5/24/2016 10:41 AM, Mark Rutland wrote: > > On Tue, May 24, 2016 at 06:39:20PM +0200, Christer Weinigel wrote: > >> Document how to use devicetree aliases to assign a stable > >> bus number to a spi bus. > >> > >> Signed-off-by:

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-24 Thread Frank Rowand
On 5/24/2016 4:34 PM, Frank Rowand wrote: > On 5/24/2016 11:32 AM, Mark Brown wrote: >> On Tue, May 24, 2016 at 08:03:48PM +0200, Christer Weinigel wrote: >>> On 05/24/2016 07:20 PM, Mark Brown wrote: >> I'm not sure this is something we want to support at all, I can't immediately see

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-24 Thread Frank Rowand
On 5/24/2016 4:34 PM, Frank Rowand wrote: > On 5/24/2016 11:32 AM, Mark Brown wrote: >> On Tue, May 24, 2016 at 08:03:48PM +0200, Christer Weinigel wrote: >>> On 05/24/2016 07:20 PM, Mark Brown wrote: >> I'm not sure this is something we want to support at all, I can't immediately see

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-24 Thread Frank Rowand
On 5/24/2016 11:32 AM, Mark Brown wrote: > On Tue, May 24, 2016 at 08:03:48PM +0200, Christer Weinigel wrote: >> On 05/24/2016 07:20 PM, Mark Brown wrote: > >>> I'm not sure this is something we want to support at all, I can't >>> immediately see anything that does this deliberately in the SPI

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-24 Thread Frank Rowand
On 5/24/2016 11:32 AM, Mark Brown wrote: > On Tue, May 24, 2016 at 08:03:48PM +0200, Christer Weinigel wrote: >> On 05/24/2016 07:20 PM, Mark Brown wrote: > >>> I'm not sure this is something we want to support at all, I can't >>> immediately see anything that does this deliberately in the SPI

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-24 Thread Frank Rowand
On 5/24/2016 10:41 AM, Mark Rutland wrote: > On Tue, May 24, 2016 at 06:39:20PM +0200, Christer Weinigel wrote: >> Document how to use devicetree aliases to assign a stable >> bus number to a spi bus. >> >> Signed-off-by: Christer Weinigel >> >> --- >> >> Trivial

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-24 Thread Frank Rowand
On 5/24/2016 10:41 AM, Mark Rutland wrote: > On Tue, May 24, 2016 at 06:39:20PM +0200, Christer Weinigel wrote: >> Document how to use devicetree aliases to assign a stable >> bus number to a spi bus. >> >> Signed-off-by: Christer Weinigel >> >> --- >> >> Trivial documentation change. >> >> Not

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-24 Thread Christer Weinigel
On 05/24/2016 08:32 PM, Mark Brown wrote: > On Tue, May 24, 2016 at 08:03:48PM +0200, Christer Weinigel wrote: >> On 05/24/2016 07:20 PM, Mark Brown wrote: > >>> I'm not sure this is something we want to support at all, I >>> can't immediately see anything that does this deliberately in >>> the

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-24 Thread Christer Weinigel
On 05/24/2016 08:32 PM, Mark Brown wrote: > On Tue, May 24, 2016 at 08:03:48PM +0200, Christer Weinigel wrote: >> On 05/24/2016 07:20 PM, Mark Brown wrote: > >>> I'm not sure this is something we want to support at all, I >>> can't immediately see anything that does this deliberately in >>> the

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-24 Thread Mark Brown
On Tue, May 24, 2016 at 08:03:48PM +0200, Christer Weinigel wrote: > On 05/24/2016 07:20 PM, Mark Brown wrote: > > I'm not sure this is something we want to support at all, I can't > > immediately see anything that does this deliberately in the SPI > > code and obviously the "bus number" is

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-24 Thread Mark Brown
On Tue, May 24, 2016 at 08:03:48PM +0200, Christer Weinigel wrote: > On 05/24/2016 07:20 PM, Mark Brown wrote: > > I'm not sure this is something we want to support at all, I can't > > immediately see anything that does this deliberately in the SPI > > code and obviously the "bus number" is

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-24 Thread Christer Weinigel
On 05/24/2016 07:20 PM, Mark Brown wrote: >> Not having used devicetree that much it was surprisingly hard to >> figure out how to assign a stable bus number to a spi bus. Add >> a simple example that shows how to do that. > > I'm not sure this is something we want to support at all, I can't

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-24 Thread Christer Weinigel
On 05/24/2016 07:20 PM, Mark Brown wrote: >> Not having used devicetree that much it was surprisingly hard to >> figure out how to assign a stable bus number to a spi bus. Add >> a simple example that shows how to do that. > > I'm not sure this is something we want to support at all, I can't

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-24 Thread Mark Rutland
On Tue, May 24, 2016 at 06:39:20PM +0200, Christer Weinigel wrote: > Document how to use devicetree aliases to assign a stable > bus number to a spi bus. > > Signed-off-by: Christer Weinigel > > --- > > Trivial documentation change. > > Not having used devicetree that

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-24 Thread Mark Rutland
On Tue, May 24, 2016 at 06:39:20PM +0200, Christer Weinigel wrote: > Document how to use devicetree aliases to assign a stable > bus number to a spi bus. > > Signed-off-by: Christer Weinigel > > --- > > Trivial documentation change. > > Not having used devicetree that much it was surprisingly

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-24 Thread Mark Brown
On Tue, May 24, 2016 at 06:39:20PM +0200, Christer Weinigel wrote: > Document how to use devicetree aliases to assign a stable > bus number to a spi bus. Please submit patches using subject lines reflecting the style for the subsystem. This makes it easier for people to identify relevant

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-24 Thread Mark Brown
On Tue, May 24, 2016 at 06:39:20PM +0200, Christer Weinigel wrote: > Document how to use devicetree aliases to assign a stable > bus number to a spi bus. Please submit patches using subject lines reflecting the style for the subsystem. This makes it easier for people to identify relevant

[PATCH] devicetree - document using aliases to set spi bus number.

2016-05-24 Thread Christer Weinigel
Document how to use devicetree aliases to assign a stable bus number to a spi bus. Signed-off-by: Christer Weinigel --- Trivial documentation change. Not having used devicetree that much it was surprisingly hard to figure out how to assign a stable bus number to a spi

[PATCH] devicetree - document using aliases to set spi bus number.

2016-05-24 Thread Christer Weinigel
Document how to use devicetree aliases to assign a stable bus number to a spi bus. Signed-off-by: Christer Weinigel --- Trivial documentation change. Not having used devicetree that much it was surprisingly hard to figure out how to assign a stable bus number to a spi bus. Add a simple