Thread (14 messages) 14 messages, 7 authors, 2017-04-28

Re: [PATCH v8 1/3] backlight arcxcnn add arc to vendor prefix

From: "Jingoo Han" <jingoohan1@gmail.com>
Date: 2017-04-25 16:36:56
Also in: linux-devicetree, lkml

On Monday, April 24, 2017 1:56 PM, Olimpiu Dejeu wrote:
On Mon, April 24, 2017 11:10 AM, Rob Herring < robh@kernel.org> wrote:
quoted
On Wed, Mar 15, 2017 at 2:45 PM, Olimpiu Dejeu [off-list ref]
wrote:
quoted
quoted
backlight: Add arc to vendor prefixes
Signed-off-by: Olimpiu Dejeu <redacted>
---
v8:
- Version to match other patches in set

 Documentation/devicetree/bindings/vendor-prefixes.txt | 1 +
 1 file changed, 1 insertion(+)
diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt
b/Documentation/devicetree/bindings/vendor-prefixes.txt
index 16d3b5e..6f33a4b 100644
--- a/Documentation/devicetree/bindings/vendor-prefixes.txt
+++ b/Documentation/devicetree/bindings/vendor-prefixes.txt
@@ -28,6 +28,7 @@ andestech     Andes Technology Corporation
 apm    Applied Micro Circuits Corporation (APM)
 aptina Aptina Imaging
 arasan Arasan Chip Systems
+arc    Arctic Sand
quoted
arc is also a cpu arch. While not a vendor, it could be confusing. How
about "arctic" >instead?

Rob, will do, i.e. I will change it to "arctic"
Hi Olimpiu,

Oh, "arc" and "arctic" is totally different.
In my opinion, one of the purposes of DT is to describe hardware stuffs.
So, please use more detailed words.
quoted
BTW, some reason your patches are not going to the DT list.

I'm emailing to devicetree@vger.kernel.org, I think this is the correct
list. Please advise.
DT is some kind of ABI. It means that changing DT names such as property names
is not easy, after DT patches were merged. So, if someone want to add new DT stuff
into the vanilla kernel, that patch should be reviewed more thoroughly
than normal patches about driver stuffs.

Thanks,
Jingoo Han
quoted
Rob
Olimpiu
  
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help