Re: [PATCH 1/9] of: add of_parse_phandle() helper for parsing

Re: [PATCH 1/9] of: add of_parse_phandle() helper for parsing

[Grant Likely]

On Wed, Mar 18, 2009 at 11:37 PM, sunder ramani <sunder.svit-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
> To Summarize, should not we have a phy_type flag in the OF tree which
> will let the PHY layer do the various PHY tasks based on if it is a
> copper/fiber interface. I really dont like my fix in my kernel and am
> new to open source. Do you think it is a valid argument? If it so, I
> would like to plan to introduce those changes and post to the
> community.

Yes, we probably do need some form of flag here for describing the PHY
interface.  Feel free to post a suggested binding for this.

g.

-- 
Grant Likely, B.Sc., P.Eng.
Secret Lab Technologies Ltd.

Re: [PATCH 1/9] of: add of_parse_phandle() helper for parsing

[sunder ramani]

Hi Grant,

Thanx a lot for the comments. I will start activities to bind the phy
type to the phy layer and post someting useful. But, to be honest,
please correct me on spot wherever i go wrong. I am new the open
source and this is very exciting!

thanx!
Sundar

On Thu, Mar 19, 2009 at 11:31 AM, Grant Likely
<grant.likely@secretlab.ca> wrote:
> On Wed, Mar 18, 2009 at 11:37 PM, sunder ramani <sunder.svit@gmail.com> wrote:
>> To Summarize, should not we have a phy_type flag in the OF tree which
>> will let the PHY layer do the various PHY tasks based on if it is a
>> copper/fiber interface. I really dont like my fix in my kernel and am
>> new to open source. Do you think it is a valid argument? If it so, I
>> would like to plan to introduce those changes and post to the
>> community.
>
> Yes, we probably do need some form of flag here for describing the PHY
> interface.  Feel free to post a suggested binding for this.
>
> g.
>
> --
> Grant Likely, B.Sc., P.Eng.
> Secret Lab Technologies Ltd.
>