[Qemu-devel] [PATCH for 2.10 3/4] fdt: probe for v1.4.2 using fdt_setprop_inplace_namelen_partial()

Philippe Mathieu-Daudé posted 4 patches 8 years, 6 months ago
[Qemu-devel] [PATCH for 2.10 3/4] fdt: probe for v1.4.2 using fdt_setprop_inplace_namelen_partial()
Posted by Philippe Mathieu-Daudé 8 years, 6 months ago
instead of fdt_first_subnode() which is v1.4.0

Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
---
 configure | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/configure b/configure
index 0d5bdb3ae9..2d803d6a77 100755
--- a/configure
+++ b/configure
@@ -3565,7 +3565,10 @@ if test "$fdt" != "no" ; then
   cat > $TMPC << EOF
 #include <libfdt.h>
 #include <libfdt_env.h>
-int main(void) { fdt_first_subnode(0, 0); return 0; }
+int main(void) {
+    fdt_setprop_inplace_namelen_partial(0, 0, 0, 0, 0, 0, 0);
+    return 0;
+}
 EOF
   if compile_prog "" "$fdt_libs" ; then
     # system DTC is good - use it
-- 
2.13.3


Re: [Qemu-devel] [PATCH for 2.10 3/4] fdt: probe for v1.4.2 using fdt_setprop_inplace_namelen_partial()
Posted by Michael Roth 8 years, 6 months ago
Quoting Philippe Mathieu-Daudé (2017-07-26 16:40:09)
> instead of fdt_first_subnode() which is v1.4.0
> 
> Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>

Signed-off-by: Michael Roth <mdroth@linux.vnet.ibm.com>

> ---
>  configure | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/configure b/configure
> index 0d5bdb3ae9..2d803d6a77 100755
> --- a/configure
> +++ b/configure
> @@ -3565,7 +3565,10 @@ if test "$fdt" != "no" ; then
>    cat > $TMPC << EOF
>  #include <libfdt.h>
>  #include <libfdt_env.h>
> -int main(void) { fdt_first_subnode(0, 0); return 0; }
> +int main(void) {
> +    fdt_setprop_inplace_namelen_partial(0, 0, 0, 0, 0, 0, 0);
> +    return 0;
> +}
>  EOF
>    if compile_prog "" "$fdt_libs" ; then
>      # system DTC is good - use it
> -- 
> 2.13.3
> 
> 


Re: [Qemu-devel] [PATCH for 2.10 3/4] fdt: probe for v1.4.2 using fdt_setprop_inplace_namelen_partial()
Posted by Michael Roth 8 years, 6 months ago
Quoting Michael Roth (2017-07-27 16:43:36)
> Quoting Philippe Mathieu-Daudé (2017-07-26 16:40:09)
> > instead of fdt_first_subnode() which is v1.4.0
> > 
> > Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
> 
> Signed-off-by: Michael Roth <mdroth@linux.vnet.ibm.com>

Doh, please ignore that. I meant:

Reviewed-by: Michael Roth <mdroth@linux.vnet.ibm.com>

> 
> > ---
> >  configure | 5 ++++-
> >  1 file changed, 4 insertions(+), 1 deletion(-)
> > 
> > diff --git a/configure b/configure
> > index 0d5bdb3ae9..2d803d6a77 100755
> > --- a/configure
> > +++ b/configure
> > @@ -3565,7 +3565,10 @@ if test "$fdt" != "no" ; then
> >    cat > $TMPC << EOF
> >  #include <libfdt.h>
> >  #include <libfdt_env.h>
> > -int main(void) { fdt_first_subnode(0, 0); return 0; }
> > +int main(void) {
> > +    fdt_setprop_inplace_namelen_partial(0, 0, 0, 0, 0, 0, 0);
> > +    return 0;
> > +}
> >  EOF
> >    if compile_prog "" "$fdt_libs" ; then
> >      # system DTC is good - use it
> > -- 
> > 2.13.3
> > 
> > 
> 
>