Re: [PATCH 5/6] ARM: DTS: da850-evm: Add node for tlv320aic3106 codec

From: Sergei Shtylyov
Date: Fri Aug 01 2014 - 09:14:06 EST


Hello.

On 08/01/2014 05:02 PM, Peter Ujfalusi wrote:

I do. We should follow the standard consistently. Why not call the node
"sound-codec"?

Well, there is _zero_ cases when the audio codec node is named as
"sound-codec" in linux-next but we have wmXXXX, tlvXXXX, twlXXXX, maxXXXX etc.

Which only means people don't read the standard (which is referred to on http://www.devicetree.org/Device_Tree_Usage, that says the same).

Yeah, there are few DTS files which have codec as node name.
So, no, I'm not going to change the node name from tlv320aic3106.

So you prefer following the bad examples to following the standard? Well, "the Moor has done his duty, the Moor can go"...

WBR, Sergei

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/