Messages in this thread | | | From | Doug Anderson <> | Date | Wed, 31 Aug 2016 14:45:40 -0700 | Subject | Re: [RESEND PATCH 3/4] arm64: dts: rockchip: support gmac for rk3399 |
| |
Hi,
On Wed, Aug 31, 2016 at 2:29 PM, Heiko Stübner <heiko@sntech.de> wrote: >> IMHO it would be nice if this were broken into two patches. >> >> 1. First patch would be the power domain patch and that could land any >> time. You wouldn't actually be able to use the gmac but at least >> you'd be able to turn off its power. This would be a handy patch to >> be able to backport if you happened to not need Ethernet support but >> wanted to save power. >> >> 2. Second patch would actually add the gmac. > > according to my talk with Caesar in the real v1, the gmac even with power- > domains should work just nicely even without the dts patches, as the driver > core takes care of powering up the pd before probe. > > But I may miss some peculiarity of the dwmac?
Nothing that I'm terribly aware of. I was just being selfish because:
1. I'm on a board where I don't need Ethernet.
2. I'm running a semi old kernel (4.4)
3. I don't want to pick back the various fixes that might be needed to make gmac work on rk3399 to that old kernel.
4. I want it very obvious that gmac isn't really supported on this old kernel on rk3399 (and having stmmac not in the device tree would make it very obvious)
5. I do want the power savings of turning the power domains off for the gmac.
If this patch is broken in two then I can pick back just the power domain patch. :-P
-Doug
| |