Scott --
On 04/30/2013 06:42 PM, Scott Wood wrote:
I just meant that, for whatever boards you would have put this in the
device tree, put it in platform code instead (if the platform file
supports more than one board type, then check the compatible at the
top of the device tree).
I think I understand what you're suggesting.
Instead of a new property name, I would instead check for my specific
board type (let's call it a foo-8315) in the top-level compatible list?
So I'd change my devtree to have this top-level compatible:
/ {
compatible = "example,foo-8315", "fsl,mpc8315erdb";
If I saw that, I would then twiddle the bits as needed?
MIght work, although having it in the sata block of the device tree has
the advantage of providing me exactly the OF node that I need (in
ofdev->dev.of_node). I'd have to figure out how to traverse to the dev
tree root and then back down one to the root compat entry. Probably not
impossible, but I was aiming for a fairly minimal patch.
It would also be nice if we could unravel exactly why that
CONFIG_8315_DS ever showed up in the first place. (The other "minimal"
aspect of my patch was to try to make changes only around that one area,
so that others could see that it was a simple change.)
Or do you mean that you would not set this on any board's device tree
by default, and instead have users set it if they encounter problems?
No, I would expect to set it on all the boards, so using the
compatibility hack above would work.
Is this a custom board you're seeing it on?
Not ours, but our vendor isn't very active on upstreaming things. (And
yes, had I know that 5 years ago, I could possibly have changed vendors,
or made upstreaming a part of the contract. But at this point, we're
stuck with this vendor, and they're not going to fix it; so I'm trying
to fix it, and I'm trying to do the best job of upstreaming those fixes
that I can.)
git send-email can connect directly to an SMTP server.
Ok, I'll play around with that.
Thanks again,
Anthony
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev
