2005-03-22 05:41:36

by Lee Nicks

[permalink] [raw]
Subject: [PATCH] 2.6.12-rc1, ./drivers/base/platform.c

The latest prepatch, 2.6.12-rc1, introduced the following change.

--- a/drivers/base/platform.c 2005-03-17 17:35:04 -08:00
+++ b/drivers/base/platform.c 2005-03-17 17:35:04 -08:00
@@ -131,7 +131,7 @@
pdev->dev.bus = &platform_bus_type;

if (pdev->id != -1)
- snprintf(pdev->dev.bus_id, BUS_ID_SIZE, "%s%u",
pdev->name, pdev->id);
+ snprintf(pdev->dev.bus_id, BUS_ID_SIZE, "%s.%u",
pdev->name, pdev->id);
else
strlcpy(pdev->dev.bus_id, pdev->name, BUS_ID_SIZE);

It causes problem, as most platform files, for example,
arch/ppc/platforms/katana.c, still use the old name without ".". I do
not understand why bus_id "mpsc.0" is better than "mpsc0".
Please explain what is the benefit of introducing such a change,
before I can submit a patch for all those platform files to work with
this change.
Please CC me, as I am currently not in the list.

Thanks,

Lee


2005-03-22 06:10:49

by Dmitry Torokhov

[permalink] [raw]
Subject: Re: [PATCH] 2.6.12-rc1, ./drivers/base/platform.c

On Tuesday 22 March 2005 00:32, All Linux wrote:
> The latest prepatch, 2.6.12-rc1, introduced the following change.
>
> --- a/drivers/base/platform.c 2005-03-17 17:35:04 -08:00
> +++ b/drivers/base/platform.c 2005-03-17 17:35:04 -08:00
> @@ -131,7 +131,7 @@
> pdev->dev.bus = &platform_bus_type;
>
> if (pdev->id != -1)
> - snprintf(pdev->dev.bus_id, BUS_ID_SIZE, "%s%u",
> pdev->name, pdev->id);
> + snprintf(pdev->dev.bus_id, BUS_ID_SIZE, "%s.%u",
> pdev->name, pdev->id);
> else
> strlcpy(pdev->dev.bus_id, pdev->name, BUS_ID_SIZE);
>
> It causes problem, as most platform files, for example,
> arch/ppc/platforms/katana.c, still use the old name without ".". I do
> not understand why bus_id "mpsc.0" is better than "mpsc0".
> Please explain what is the benefit of introducing such a change,
> before I can submit a patch for all those platform files to work with
> this change.
> Please CC me, as I am currently not in the list.
>

Devices/drivers ending with a digit, such as i8250, produce "wierd"
names - i82500, i82501, etc.

--
Dmitry

2005-03-22 18:45:13

by Brian Waite

[permalink] [raw]
Subject: Re: [PATCH] 2.6.12-rc1, ./drivers/base/platform.c

On Tuesday 22 March 2005 00:32, All Linux wrote:
> It causes problem, as most platform files, for example,
> arch/ppc/platforms/katana.c, still use the old name without ".". I do
Mark Greer recently produced a patch for the katana board among other PPC platforms
to fix this breakage. I'll look for the announcement mail but I recall seeing it a day or two ago.

Thanks
Brian


Attachments:
(No filename) (377.00 B)
(No filename) (189.00 B)
Download all attachments

2005-03-22 20:23:17

by Lee Nicks

[permalink] [raw]
Subject: Re: [PATCH] 2.6.12-rc1, ./drivers/base/platform.c

On Tue, 22 Mar 2005 13:32:09 -0500, Brian Waite <[email protected]> wrote:
> On Tuesday 22 March 2005 00:32, All Linux wrote:
> > It causes problem, as most platform files, for example,
> > arch/ppc/platforms/katana.c, still use the old name without ".". I do
> Mark Greer recently produced a patch for the katana board among other PPC platforms
> to fix this breakage. I'll look for the announcement mail but I recall seeing it a day or two ago.
>
> Thanks
> Brian
>
>
>
Ok, I will wait for Mark Greer's patch. Thanks.