* [PATCH 0/2] DT: net: davinci_emac: couple more properties actually optional
@ 2014-01-27 23:45 Sergei Shtylyov
2014-01-29 7:43 ` David Miller
0 siblings, 1 reply; 3+ messages in thread
From: Sergei Shtylyov @ 2014-01-27 23:45 UTC (permalink / raw)
To: netdev, robh+dt, pawel.moll, mark.rutland, ijc+devicetree, galak,
rob, devicetree
Cc: linux-doc, davinci-linux-open-source
Hello.
Though described as required, couple more properties in the DaVinci EMAC
binding are actually optional, as the driver will happily function without them.
The patchset is against DaveM's 'net.git' tree this time.
[1/2] DT: net: davinci_emac: "ti,davinci-rmii-en" property is actually optional
[2/2] DT: net: davinci_emac: "ti,davinci-no-bd-ram" property is actually optional
WBR, Sergei
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH 0/2] DT: net: davinci_emac: couple more properties actually optional
2014-01-27 23:45 [PATCH 0/2] DT: net: davinci_emac: couple more properties actually optional Sergei Shtylyov
@ 2014-01-29 7:43 ` David Miller
2014-01-29 22:42 ` Sergei Shtylyov
0 siblings, 1 reply; 3+ messages in thread
From: David Miller @ 2014-01-29 7:43 UTC (permalink / raw)
To: sergei.shtylyov
Cc: netdev, robh+dt, pawel.moll, mark.rutland, ijc+devicetree, galak,
rob, devicetree, linux-doc, davinci-linux-open-source
From: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
Date: Tue, 28 Jan 2014 02:45:34 +0300
> Though described as required, couple more properties in the DaVinci EMAC
> binding are actually optional, as the driver will happily function without them.
> The patchset is against DaveM's 'net.git' tree this time.
>
> [1/2] DT: net: davinci_emac: "ti,davinci-rmii-en" property is actually optional
> [2/2] DT: net: davinci_emac: "ti,davinci-no-bd-ram" property is actually optional
Series applied with the "has/have" thing fixed.
Thanks.
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH 0/2] DT: net: davinci_emac: couple more properties actually optional
2014-01-29 7:43 ` David Miller
@ 2014-01-29 22:42 ` Sergei Shtylyov
0 siblings, 0 replies; 3+ messages in thread
From: Sergei Shtylyov @ 2014-01-29 22:42 UTC (permalink / raw)
To: David Miller
Cc: netdev, robh+dt, pawel.moll, mark.rutland, ijc+devicetree, galak,
rob, devicetree, linux-doc, davinci-linux-open-source
Hello.
On 01/29/2014 10:43 AM, David Miller wrote:
>> Though described as required, couple more properties in the DaVinci EMAC
>> binding are actually optional, as the driver will happily function without them.
>> The patchset is against DaveM's 'net.git' tree this time.
>> [1/2] DT: net: davinci_emac: "ti,davinci-rmii-en" property is actually optional
>> [2/2] DT: net: davinci_emac: "ti,davinci-no-bd-ram" property is actually optional
> Series applied with the "has/have" thing fixed.
> Thanks.
Thank you!
Unfortunately, this driver presents a bad example of DT bindings overall
(caused in its turn by a misuse of the platform data for the EMAC type
differing instead of the platform device IDs).
WBR, Sergei
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2014-01-29 22:42 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-01-27 23:45 [PATCH 0/2] DT: net: davinci_emac: couple more properties actually optional Sergei Shtylyov
2014-01-29 7:43 ` David Miller
2014-01-29 22:42 ` Sergei Shtylyov
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).