Re: [Buildroot] [PATCH v2 1/1] package: Add iotools package

[Andy Shevchenko <andriy.shevchenko@linux.intel.com>]

On Sat, Oct 26, 2024 at 02:47:48PM +0200, Thomas Petazzoni via buildroot wrote:
> On Fri, 27 Sep 2024 21:34:59 +0300
> Andy Shevchenko <andriy.shevchenko@linux.intel.com> wrote:
> 
> > The iotools package provides a set of simple command line tools which
> > allow access to hardware device registers. Supported register interfaces
> > include PCI, IO, memory mapped IO, SMBus, CPUID, and MSR. Also included
> > are some utilities which allow for simple arithmetic, logical, and other
> > operations.
> > 
> > If you ever have to debug hardware, you could probably use these tools.
> 
> Applied to master after doing some tweaks:

Thank you!

> - Used STATIC=1/STATIC=0 to achieve the proper behavior with regard to
>   BR2_STATIC_LIBS
> 
> - Use the canonical $(INSTALL) incantation for the installation step
> 
> One thing that isn't super nice is your handling of the DEBUG variable.
> Ideally in Buildroot we would want neither DEBUG=0 nor DEBUG=1, but
> "don't do anything and let Buildroot pass the CFLAGS it wants".

Any suggestion how it may be addressed?

-- 
With Best Regards,
Andy Shevchenko


_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot