Re: Exposing ROM's though sysfs

From: Christoph Hellwig
Date: Fri Jul 30 2004 - 12:26:12 EST


On Fri, Jul 30, 2004 at 10:10:29AM -0700, Jesse Barnes wrote:
> + unsigned long start = dev->resource[PCI_ROM_RESOURCE].start;
> + int size = dev->resource[PCI_ROM_RESOURCE].end -
> + dev->resource[PCI_ROM_RESOURCE].start;

pci_resource_start and pci_resource_len please.

> + .name = "rom",
> + .mode = S_IRUGO | S_IWUSR,

do we really want it world readable if a read messes with pci config
space?

> + if (pdev->resource[PCI_ROM_RESOURCE].start) {
> + pci_rom_attr.size = pdev->resource[PCI_ROM_RESOURCE].end -
> + pdev->resource[PCI_ROM_RESOURCE].start;

as above.

-
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/