Re: [MeeGo-Dev][PATCH] Topcliff: Update PCH_PHUB driver to 2.6.35

From: Greg KH
Date: Tue Aug 24 2010 - 09:36:09 EST


On Tue, Aug 24, 2010 at 03:47:06PM +0900, Masayuki Ohtak wrote:
> Hi Greg,
>
> We have modified our phub driver for your indication.
> Please check below.
>
> Best Regards, Ohtake(OKISEMI).
>
> ---
> Packet hub driver of Topcliff PCH
>
> Topcliff PCH is the platform controller hub that is going to be used in
> Intel's upcoming general embedded platform. All IO peripherals in
> Topcliff PCH are actually devices sitting on AMBA bus. Packet hub is
> a special converter device in Topcliff PCH that translate AMBA transactions
> to PCI Express transactions and vice versa. Thus packet hub helps present
> all IO peripherals in Topcliff PCH as PCIE devices to IA system.
> Topcliff PCH has MAC address and Option ROM data.
> These data are in SROM which is connected to PCIE bus.
> Packet hub driver of Topcliff PCH can access MAC address and Option ROM data in
> SROM via sysfs interface.
>
> Signed-off-by: Masayuki Ohtake <masa-korg@xxxxxxxxxxxxxxx>
> ---
> Documentation/ABI/testing/sysfs-module | 14 +
> drivers/misc/Kconfig | 9 +
> drivers/misc/Makefile | 1 +
> drivers/misc/pch_phub.c | 719 ++++++++++++++++++++++++++++++++
> 4 files changed, 743 insertions(+), 0 deletions(-)
> create mode 100644 Documentation/ABI/testing/sysfs-module
> create mode 100755 drivers/misc/pch_phub.c
>
> diff --git a/Documentation/ABI/testing/sysfs-module b/Documentation/ABI/testing/sysfs-module
> new file mode 100644
> index 0000000..fe07ffa
> --- /dev/null
> +++ b/Documentation/ABI/testing/sysfs-module
> @@ -0,0 +1,14 @@
> +What: /sys/module/pch_phub/drivers/pci:pch_phub/<dev>/pch_mac

Please don't put a ':' in your driver name.

> +Date: August 2010
> +KernelVersion: 2.6.35
> +Contact: masa-korg@xxxxxxxxxxxxxxx
> +Description: Write/read GbE MAC address.
> +Users: masa-korg@xxxxxxxxxxxxxxx

You are really going to be the only user of this file?

> +
> +What: /sys/module/pch_phub/drivers/pci:pch_phub/<dev>/pch_firmware
> +Date: August 2010
> +KernelVersion: 2.6.35
> +Contact: masa-korg@xxxxxxxxxxxxxxx
> +Description: Write/read Option ROM data.
> +Users: masa-korg@xxxxxxxxxxxxxxx
> +
> diff --git a/drivers/misc/Kconfig b/drivers/misc/Kconfig
> index 26386a9..756f832 100644
> --- a/drivers/misc/Kconfig
> +++ b/drivers/misc/Kconfig
> @@ -353,6 +353,15 @@ config VMWARE_BALLOON
> To compile this driver as a module, choose M here: the
> module will be called vmware_balloon.
>
> +config PCH_PHUB
> + tristate "PCH Packet Hub of Intel Topcliff"
> + depends on PCI
> + help
> + This driver is for PCH(Platform controller Hub) PHUB(Packet Hub) of
> + Intel Topcliff which is an IOH(Input/Output Hub) for x86 embedded
> + processor. The Topcliff has MAC address and Option ROM data in SROM.
> + This driver can access MAC address and Option ROM data in SROM.
> +

Please add the wording "To compile this driver..." here as well.

Your driver code looks much better, nice job.

thanks,

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/