Hi David, On 10 September 2018 at 06:59, David Gibson <david@xxxxxxxxxxxxxxxxxxxxx> wrote: > > pylibfdt/setup.py currently adds include flags to the extension module > build to allow include files in the base dtc directory. But pylibfdt > doesn't rely on any headers there, only on headers in libfdt/ - it also > shouldn't rely on dtc headers at any future time. > > So, remove that from the include list, allowing some simplifications to > setup.py. > > Signed-off-by: David Gibson <david@xxxxxxxxxxxxxxxxxxxxx> > --- > pylibfdt/setup.py | 10 +++------- > 1 file changed, 3 insertions(+), 7 deletions(-) I can't recall why I added this, so I agree it's good to drop it. Reviewed-by: Simon Glass <sjg@xxxxxxxxxxxx>