Re: [PATCH] Revert "firmware: dmi_scan: Use lowercase letters for UUID"

2018-12-11 Thread Peter Korsgaard
> "Jean" == Jean Delvare writes: > On Tue, 2018-12-11 at 13:06 +0100, Peter Korsgaard wrote: >> > > > > > "Peter" == Peter Korsgaard writes: >> >> Hi Jean, >> >> >> Look, you can imagine that I was perfectly aware of what I was doing >> >> when I made that change, and that I

Re: [PATCH] Revert "firmware: dmi_scan: Use lowercase letters for UUID"

2018-12-11 Thread Jean Delvare
On Tue, 2018-12-11 at 13:06 +0100, Peter Korsgaard wrote: > > > > > > "Peter" == Peter Korsgaard writes: > > Hi Jean, > > >> Look, you can imagine that I was perfectly aware of what I was doing > >> when I made that change, and that I pondered the decision carefully at > >> that time. And my

Re: [PATCH] Revert "firmware: dmi_scan: Use lowercase letters for UUID"

2018-12-11 Thread Peter Korsgaard
> "Peter" == Peter Korsgaard writes: Hi Jean, >> Look, you can imagine that I was perfectly aware of what I was doing >> when I made that change, and that I pondered the decision carefully at >> that time. And my decision was that the change should be made. As far >> as I'm concerned,

Re: [PATCH] Revert "firmware: dmi_scan: Use lowercase letters for UUID"

2018-12-06 Thread Peter Korsgaard
> "Jean" == Jean Delvare writes: Hi, >> I get that - but it changes the content of sysfs entries, breaking real >> systems - E.G. a user space ABI regression. > I know it's very convenient to play the "user-space ABI regression" > card whenever you want a change reverted. Sorry, I am

Re: [PATCH] Revert "firmware: dmi_scan: Use lowercase letters for UUID"

2018-12-06 Thread Peter Korsgaard
> "Jean" == Jean Delvare writes: Hi, >> I get that - but it changes the content of sysfs entries, breaking real >> systems - E.G. a user space ABI regression. > I know it's very convenient to play the "user-space ABI regression" > card whenever you want a change reverted. Sorry, I am

Re: [PATCH] Revert "firmware: dmi_scan: Use lowercase letters for UUID"

2018-12-06 Thread Jean Delvare
On Thu, 2018-12-06 at 10:22 +0100, Peter Korsgaard wrote: > > > > > > "Jean" == Jean Delvare writes: > > > On Wed, 2018-12-05 at 22:13 +0100, Peter Korsgaard wrote: > >> This reverts commit 712ff25450bd01366301eef81c33e865d901e7b7. > >> > >> The output of dmi_save_uuid() is exposed to user

Re: [PATCH] Revert "firmware: dmi_scan: Use lowercase letters for UUID"

2018-12-06 Thread Jean Delvare
On Thu, 2018-12-06 at 10:22 +0100, Peter Korsgaard wrote: > > > > > > "Jean" == Jean Delvare writes: > > > On Wed, 2018-12-05 at 22:13 +0100, Peter Korsgaard wrote: > >> This reverts commit 712ff25450bd01366301eef81c33e865d901e7b7. > >> > >> The output of dmi_save_uuid() is exposed to user

Re: [PATCH] Revert "firmware: dmi_scan: Use lowercase letters for UUID"

2018-12-06 Thread Peter Korsgaard
> "Jean" == Jean Delvare writes: > On Wed, 2018-12-05 at 22:13 +0100, Peter Korsgaard wrote: >> This reverts commit 712ff25450bd01366301eef81c33e865d901e7b7. >> >> The output of dmi_save_uuid() is exposed to user space as >> /sys/devices/virtual/dmi/id/*_uuid, so this breaks backwards

Re: [PATCH] Revert "firmware: dmi_scan: Use lowercase letters for UUID"

2018-12-06 Thread Peter Korsgaard
> "Jean" == Jean Delvare writes: > On Wed, 2018-12-05 at 22:13 +0100, Peter Korsgaard wrote: >> This reverts commit 712ff25450bd01366301eef81c33e865d901e7b7. >> >> The output of dmi_save_uuid() is exposed to user space as >> /sys/devices/virtual/dmi/id/*_uuid, so this breaks backwards

Re: [PATCH] Revert "firmware: dmi_scan: Use lowercase letters for UUID"

2018-12-06 Thread Jean Delvare
On Wed, 2018-12-05 at 22:13 +0100, Peter Korsgaard wrote: > This reverts commit 712ff25450bd01366301eef81c33e865d901e7b7. > > The output of dmi_save_uuid() is exposed to user space as > /sys/devices/virtual/dmi/id/*_uuid, so this breaks backwards compatibility, > E.G. I have systems that include

Re: [PATCH] Revert "firmware: dmi_scan: Use lowercase letters for UUID"

2018-12-06 Thread Jean Delvare
On Wed, 2018-12-05 at 22:13 +0100, Peter Korsgaard wrote: > This reverts commit 712ff25450bd01366301eef81c33e865d901e7b7. > > The output of dmi_save_uuid() is exposed to user space as > /sys/devices/virtual/dmi/id/*_uuid, so this breaks backwards compatibility, > E.G. I have systems that include

Re: [PATCH] Revert "firmware: dmi_scan: Use lowercase letters for UUID"

2018-12-05 Thread Peter Korsgaard
> "Peter" == Peter Korsgaard writes: > This reverts commit 712ff25450bd01366301eef81c33e865d901e7b7. > The output of dmi_save_uuid() is exposed to user space as > /sys/devices/virtual/dmi/id/*_uuid, so this breaks backwards compatibility, > E.G. I have systems that include the content

Re: [PATCH] Revert "firmware: dmi_scan: Use lowercase letters for UUID"

2018-12-05 Thread Peter Korsgaard
> "Peter" == Peter Korsgaard writes: > This reverts commit 712ff25450bd01366301eef81c33e865d901e7b7. > The output of dmi_save_uuid() is exposed to user space as > /sys/devices/virtual/dmi/id/*_uuid, so this breaks backwards compatibility, > E.G. I have systems that include the content

[PATCH] Revert "firmware: dmi_scan: Use lowercase letters for UUID"

2018-12-05 Thread Peter Korsgaard
This reverts commit 712ff25450bd01366301eef81c33e865d901e7b7. The output of dmi_save_uuid() is exposed to user space as /sys/devices/virtual/dmi/id/*_uuid, so this breaks backwards compatibility, E.G. I have systems that include the content of dmi/id/product_uuid as part of the keyphrase for

[PATCH] Revert "firmware: dmi_scan: Use lowercase letters for UUID"

2018-12-05 Thread Peter Korsgaard
This reverts commit 712ff25450bd01366301eef81c33e865d901e7b7. The output of dmi_save_uuid() is exposed to user space as /sys/devices/virtual/dmi/id/*_uuid, so this breaks backwards compatibility, E.G. I have systems that include the content of dmi/id/product_uuid as part of the keyphrase for