Re: [PATCH RFT 2/2] macb: kill PHY reset code

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 04/11/2016 09:51 PM, Andrew Lunn wrote:

The code you are deleting would of ignored the flags in the gpio
property, i.e. active low.

    Hm, you're right -- I forgot about that... :-/

The new code in the previous patch does
however take the flags into account. Did you check if there are any
device trees which have flags, which were never used, but are now
going to be used and thus break...

    Checked this now and found out arch/arm/boot/dts/ar91-vinco.dts.
Looks like it needs to be fixed indeed...

And this is where it gets tricky. You are breaking backwards
compatibility by now respecting the flag. An old DT blob is not going
to work.

    Do we care that much about the DT blobs that are just *wrong*?

Wrong, but currently works.

Note that it's not only using GPIO_ACTIVE_HIGH but does that against what the MACB binding documents.

You potentially need to add a new property and deprecate the old one.

    I would like to avoid that...

You will need the agreement from the at91-vinco maintainer.

  I'll try submitting a formal DT patch...

     Andrew

MBR, Sergei

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux