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


Groups > linux.kernel > #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>
Newsgroups linux.kernel
Subject RE: [RESEND PATCH v2 1/2] device property: Add function to search for named child of device
Date 2016-06-14 10:40 +0200
Message-ID <rJS38-2kn-13@gated-at.bofh.it> (permalink)
References <rIhoZ-4MY-5@gated-at.bofh.it> <rIhoZ-4MY-7@gated-at.bofh.it> <rIhoZ-4MY-3@gated-at.bofh.it> <rIrol-2W0-11@gated-at.bofh.it> <rJS38-2kn-15@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


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.

Back to linux.kernel | Previous | Next | Find similar | Unroll thread


Thread

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

csiph-web