Re: [PATCH v3 1/6] dt-bindings: mux-j7200-wiz: Add lane function defines

2020-09-17 Thread Nishanth Menon
On 14:53-20200917, Peter Rosin wrote: [..] > I do not have a tree and dt-patches should normally not go *through* me. yep. > But I'd still like to see what's happening. > I did not realize this was going to cause a *lot* of churn in the dt > files. How bad can it be when the file is new in this c

Re: [PATCH v3 1/6] dt-bindings: mux-j7200-wiz: Add lane function defines

2020-09-17 Thread Peter Rosin
On 2020-09-17 14:27, Nishanth Menon wrote: > On 11:45-20200917, Peter Rosin wrote: > [...] >> Should not the defines start with J7200_WIZ? SERDES0 seems like a too generic prefix, at least to me. >>> >>> Thanks, good point. I am not sure if WIZ should even be used.. It is >>> a TI inte

Re: [PATCH v3 1/6] dt-bindings: mux-j7200-wiz: Add lane function defines

2020-09-17 Thread Nishanth Menon
On 14:37-20200917, Peter Rosin wrote: [...] > >>> Should not the defines start with J7200_WIZ? SERDES0 seems like a too > >>> generic prefix, at least to me. > >> > >> Thanks, good point. I am not sure if WIZ should even be used.. It is > >> a TI internal prefix for various serdes solutions, but I

Re: [PATCH v3 1/6] dt-bindings: mux-j7200-wiz: Add lane function defines

2020-09-17 Thread Peter Rosin
Hi! On 2020-09-17 14:00, Roger Quadros wrote: > Hi Peter & Nishanth, > > On 16/09/2020 18:45, Nishanth Menon wrote: >> On 06:52-20200916, Peter Rosin wrote: >>> Hi, >>> >>> Sorry for the delay. >>> >>> On 2020-09-15 13:20, Roger Quadros wrote: Each SERDES lane mux can select upto 4 different

Re: [PATCH v3 1/6] dt-bindings: mux-j7200-wiz: Add lane function defines

2020-09-17 Thread Nishanth Menon
On 15:47-20200917, Kishon Vijay Abraham I wrote: [..] > > Thanks, good point. I am not sure if WIZ should even be used.. It is > > a TI internal prefix for various serdes solutions, but I agree that > > SERDES0 is too generic a terminology. That said, we should cleanup > > include/dt-bindings/mux/

Re: [PATCH v3 1/6] dt-bindings: mux-j7200-wiz: Add lane function defines

2020-09-17 Thread Nishanth Menon
On 11:45-20200917, Peter Rosin wrote: [...] > > >> Should not the defines start with J7200_WIZ? SERDES0 seems like a too > >> generic prefix, at least to me. > > > > Thanks, good point. I am not sure if WIZ should even be used.. It is > > a TI internal prefix for various serdes solutions, but I a

Re: [PATCH v3 1/6] dt-bindings: mux-j7200-wiz: Add lane function defines

2020-09-17 Thread Roger Quadros
Hi Peter & Nishanth, On 16/09/2020 18:45, Nishanth Menon wrote: On 06:52-20200916, Peter Rosin wrote: Hi, Sorry for the delay. On 2020-09-15 13:20, Roger Quadros wrote: Each SERDES lane mux can select upto 4 different IPs. There are 4 lanes in each J7200 SERDES. Define all the possible funct

Re: [PATCH v3 1/6] dt-bindings: mux-j7200-wiz: Add lane function defines

2020-09-17 Thread Nishanth Menon
On 15:00-20200917, Roger Quadros wrote: > Hi Peter & Nishanth, > > On 16/09/2020 18:45, Nishanth Menon wrote: > > On 06:52-20200916, Peter Rosin wrote: > > > Hi, > > > > > > Sorry for the delay. > > > > > > On 2020-09-15 13:20, Roger Quadros wrote: > > > > Each SERDES lane mux can select upto 4

Re: [PATCH v3 1/6] dt-bindings: mux-j7200-wiz: Add lane function defines

2020-09-17 Thread Kishon Vijay Abraham I
Nishanth, On 16/09/20 9:15 pm, Nishanth Menon wrote: > On 06:52-20200916, Peter Rosin wrote: >> Hi, >> >> Sorry for the delay. >> >> On 2020-09-15 13:20, Roger Quadros wrote: >>> Each SERDES lane mux can select upto 4 different IPs. >>> There are 4 lanes in each J7200 SERDES. Define all >>> the po

Re: [PATCH v3 1/6] dt-bindings: mux-j7200-wiz: Add lane function defines

2020-09-17 Thread Peter Rosin
Hi! On 2020-09-16 17:45, Nishanth Menon wrote: > On 06:52-20200916, Peter Rosin wrote: >> Hi, >> >> Sorry for the delay. >> >> On 2020-09-15 13:20, Roger Quadros wrote: >>> Each SERDES lane mux can select upto 4 different IPs. >>> There are 4 lanes in each J7200 SERDES. Define all >>> the possible

Re: [PATCH v3 1/6] dt-bindings: mux-j7200-wiz: Add lane function defines

2020-09-16 Thread Nishanth Menon
On 06:52-20200916, Peter Rosin wrote: > Hi, > > Sorry for the delay. > > On 2020-09-15 13:20, Roger Quadros wrote: > > Each SERDES lane mux can select upto 4 different IPs. > > There are 4 lanes in each J7200 SERDES. Define all > > the possible functions in this file. > > > > Cc: Peter Rosin >

Re: [PATCH v3 1/6] dt-bindings: mux-j7200-wiz: Add lane function defines

2020-09-15 Thread Peter Rosin
Hi, Sorry for the delay. On 2020-09-15 13:20, Roger Quadros wrote: > Each SERDES lane mux can select upto 4 different IPs. > There are 4 lanes in each J7200 SERDES. Define all > the possible functions in this file. > > Cc: Peter Rosin > Signed-off-by: Roger Quadros > --- > include/dt-bindings

[PATCH v3 1/6] dt-bindings: mux-j7200-wiz: Add lane function defines

2020-09-15 Thread Roger Quadros
Each SERDES lane mux can select upto 4 different IPs. There are 4 lanes in each J7200 SERDES. Define all the possible functions in this file. Cc: Peter Rosin Signed-off-by: Roger Quadros --- include/dt-bindings/mux/mux-j7200-wiz.h | 29 + 1 file changed, 29 insertions(+)