On Fri, Mar 14, 2014 at 04:54:08PM +0100, Aaron Plattner wrote: > From: Thierry Reding <[email protected]> > > Most of the driver enumeration functions take an array and a maximum > number of entries that they are allowed to fill in. Upon success, they > return the number of entries filled in. This allows them to be easily > used to consecutively. > > One exception is the xf86MatchDriverFromFiles() function, which doesn't > return a value, so callers have to manually search the array for the > first empty entry. > > This commit modifies the xf86MatchDriverFromFiles() to behave the same > way as others, which makes it easier to deal with. > > Signed-off-by: Thierry Reding <[email protected]> > Reviewed-By: Aaron Plattner <[email protected]> > Tested-By: Aaron Plattner <[email protected]> > Signed-off-by: Aaron Plattner <[email protected]> > --- > hw/xfree86/common/xf86AutoConfig.c | 2 +- > hw/xfree86/common/xf86pciBus.c | 24 ++++++++++-------------- > hw/xfree86/common/xf86pciBus.h | 5 +++-- > hw/xfree86/common/xf86platformBus.c | 7 ++----- > 4 files changed, 16 insertions(+), 22 deletions(-)
Keith, Aaron indicated that you might have some comments or reservations about this series? Did you have time to look at it? Thierry ----------------------------------------------------------------------------------- This email message is for the sole use of the intended recipient(s) and may contain confidential information. Any unauthorized review, use, disclosure or distribution is prohibited. If you are not the intended recipient, please contact the sender by reply email and destroy all copies of the original message. -----------------------------------------------------------------------------------
pgpGh170vkmuG.pgp
Description: PGP signature
_______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
