svn commit: r229032 - head/sys/modules/cfi

[Bjoern A. Zeeb]

On 31. Dec 2011, at 16:48 , Marius Strobl wrote:

> On Sat, Dec 31, 2011 at 04:26:04PM +0000, Bjoern A. Zeeb wrote:
>> 
>> On 31. Dec 2011, at 15:59 , Marius Strobl wrote:
>> 
>>> On Sat, Dec 31, 2011 at 02:05:40PM +0000, Bjoern A. Zeeb wrote:
>>>> 
>>>> On 30. Dec 2011, at 21:22 , Marius Strobl wrote:
>>>> 
>>>>> Author: marius
>>>>> Date: Fri Dec 30 21:22:10 2011
>>>>> New Revision: 229032
>>>>> URL: http://svn.freebsd.org/changeset/base/229032
>>>>> 
>>>>> Log:
>>>>> Add header required by cfi_bus_fdt.c.
>>>>> 
>>>>> Modified:
>>>>> head/sys/modules/cfi/Makefile
>>>>> 
>>>> 
>>>> This (most likely) broke at least the arm KB920X config:
>>>> 
>>>> In file included from @/arm/xscale/ixp425/ixp425var.h:47,
>>>>                from /scratch/tmp/bz/head.universe/sys/modules/cfi/../../dev/cfi/cfi_bus_ixp4xx.c:45:
>>>> @/dev/pci/pcivar.h:219:20: error: pci_if.h: No such file or directory
>>>> mkdep: compile failed
>>>> 
>>> 
>>> It didn't, the original r228981 just reveals a problem with ixp425var.h.
>>> The funny thing is that while it's pretty much clear from your above
>>> snippet what's going on I can't reproduce this, also not part of an
>>> universe build and apparently also not the tinderbox.
>> 
>> The tinderbox does not build it if you look at the details;-)
> 
> I was referring to the absence of any new breakage report by the
> tinderbox after r229032, which IMO should have occurred by now.

Oh, it's been arm not mips; yeah it should have built that and failed.
Not sure why not.

make universe is happy now at least as well.

Thanks a lot!

-- 
Bjoern A. Zeeb                                 You have to have visions!
   It does not matter how good you are. It matters what good you do!