[coreboot] Patch set updated for coreboot: 897a07d Refactor src/southbridge/intel/i82801ex/
Paul Menzel
paulepanter at users.sourceforge.net
Tue Nov 8 11:05:39 CET 2011
Dear Idwer,
Am Dienstag, den 08.11.2011, 03:23 +0100 schrieb Idwer Vollering:
> Idwer Vollering (vidwer at gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/415
>
> -gerrit
>
> commit 897a07d55541bec5336b67789cfc155c31053d41
> Author: Idwer Vollering <vidwer at gmail.com>
> Date: Mon Nov 7 17:48:33 2011 +0100
>
> Refactor src/southbridge/intel/i82801ex/
>
> Change-Id: I8be22f0292e322562b117f1e8bf493ed25eb4bec
> Signed-off-by: Idwer Vollering <vidwer at gmail.com>
> ---
> src/southbridge/intel/i82801ex/chip.h | 1 -
> src/southbridge/intel/i82801ex/early_smbus.c | 32 ++++++++------
> src/southbridge/intel/i82801ex/ehci.c | 8 ++-
> src/southbridge/intel/i82801ex/i82801ex.c | 12 +++---
> src/southbridge/intel/i82801ex/i82801ex.h | 26 +++++++++---
> src/southbridge/intel/i82801ex/ide.c | 15 +++----
> src/southbridge/intel/i82801ex/lpc.c | 59 +++++++++++++-------------
> src/southbridge/intel/i82801ex/pci.c | 20 +++++----
> src/southbridge/intel/i82801ex/sata.c | 35 +++++++++-------
> src/southbridge/intel/i82801ex/smbus.c | 3 +-
> src/southbridge/intel/i82801ex/smbus.h | 6 ++-
> src/southbridge/intel/i82801ex/uhci.c | 1 -
> src/southbridge/intel/i82801ex/watchdog.c | 10 +++--
> src/superio/winbond/w83627thg/early_serial.c | 11 +++++
> src/superio/winbond/w83627thg/w83627thg.h | 2 +
> 15 files changed, 141 insertions(+), 100 deletions(-)
that new diff looks rather large. Especially in comparison with the
former patch iteration. The w83627thg changes could also go in as a
separate patch?
What is the objective of the refactoring? To more conform with other
targets?
[…]
Thanks,
Paul
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20111108/015e524c/attachment.sig>
More information about the coreboot
mailing list