Thread (39 messages) flat view 39 messages, 6 authors, 2012-07-02
STALE5193d

[PATCH v2 2/2] net: flexcan: add transceiver switch gpios support

From: Shawn Guo <hidden>
Date: 2012-06-28 11:52:26
Also in: linux-can

On Thu, Jun 28, 2012 at 07:41:10PM +0800, Shawn Guo wrote:
On Thu, Jun 28, 2012 at 01:29:19PM +0200, Marc Kleine-Budde wrote:
quoted
I mean which name is more precise, do these gpio enable/standy a "phy"
or a "transceiver". For example:
http://www.nxp.com/documents/application_note/AN00094.pdf, this document
says: TJA1041/1041A high speed CAN transceiver.
Isn't term "phy" (physical interface) generally meant to be the same
thing as "transceiver"?  I just happened to like the shorter one as
what Hui did in his patch.

But it does not really matter to me, will change the name since you
care about it.
Do you care the variable name also?  If so, we will get:

        int transceiver_en_gpio;
        int transceiver_stby_gpio;
        bool transceiver_en_high;
        bool transceiver_stby_high;

So everything becomes long :)

-- 
Regards,
Shawn
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help