Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1421667 > unrolled thread

RE: [RESEND PATCH v2 1/2] device property: Add function to search for named child of device

Started by"Opensource [Adam Thomson]" <Adam.Thomson.Opensource@diasemi.com>
First post2016-06-14 10:40 +0200
Last post2016-06-14 10:40 +0200
Articles 1 — 1 participant

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  RE: [RESEND PATCH v2 1/2] device property: Add function to search  for named child of device "Opensource [Adam Thomson]" <Adam.Thomson.Opensource@diasemi.com> - 2016-06-14 10:40 +0200

#1421667 — RE: [RESEND PATCH v2 1/2] device property: Add function to search for named child of device

From"Opensource [Adam Thomson]" <Adam.Thomson.Opensource@diasemi.com>
Date2016-06-14 10:40 +0200
SubjectRE: [RESEND PATCH v2 1/2] device property: Add function to search for named child of device
Message-ID<rJS38-2kn-13@gated-at.bofh.it>
On 13 June 2016 20:33, Frank Rowand wrote:

> > DT node names are case insensitive. The of.h header does provide a helper macro
> > which is equivalent to this, but that macro is part of the '#ifdef CONFIG_OF'
> > block. If I were to use it then it would cause non-DT builds to fail. I opted
> > for strcasecmp() directly as I didn't think for just this one scenario it made
> > sense to reorganise the of.h header with regards to the helper macros. Of course
> > if there are other opinions on this then am happy to listen.
>
> DT node names are not always case insensitive.  Please us of_node_cmp().
>
> -Frank

Ok, fair enough. I'll have to move those definitions in the of.h header out of
the CONFIG_OF block then.

[toc] | [standalone]


Back to top | Article view | linux.kernel


csiph-web