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.txtb/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 Sandquoted
arc is also a cpu arch. While not a vendor, it could be confusing. Howabout "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
RobOlimpiu