* [PATCH] Revert "iio: add modifiers for A and B ultraviolet light"
@ 2024-01-23 10:43 Paul Cercueil
2024-01-23 10:51 ` Javier Carrasco
0 siblings, 1 reply; 7+ messages in thread
From: Paul Cercueil @ 2024-01-23 10:43 UTC (permalink / raw)
To: Jonathan Cameron, Javier Carrasco, Lars-Peter Clausen
Cc: Nuno Sa, linux-iio, linux-kernel, Paul Cercueil
This reverts
b89710bd215e ("iio: add modifiers for A and B ultraviolet light")
Enum iio_modifer is *ABI*, you can't just decide to change all the
values from one version to another, otherwise you break userspace.
The new entries should have been added to the end of the enum.
Signed-off-by: Paul Cercueil <paul@crapouillou.net>
---
Documentation/ABI/testing/sysfs-bus-iio | 7 ++-----
drivers/iio/industrialio-core.c | 2 --
include/uapi/linux/iio/types.h | 2 --
tools/iio/iio_event_monitor.c | 2 --
4 files changed, 2 insertions(+), 11 deletions(-)
diff --git a/Documentation/ABI/testing/sysfs-bus-iio b/Documentation/ABI/testing/sysfs-bus-iio
index 2e6d5ebfd3c7..7937bb4a4a68 100644
--- a/Documentation/ABI/testing/sysfs-bus-iio
+++ b/Documentation/ABI/testing/sysfs-bus-iio
@@ -1587,8 +1587,6 @@ What: /sys/.../iio:deviceX/in_intensityY_raw
What: /sys/.../iio:deviceX/in_intensityY_ir_raw
What: /sys/.../iio:deviceX/in_intensityY_both_raw
What: /sys/.../iio:deviceX/in_intensityY_uv_raw
-What: /sys/.../iio:deviceX/in_intensityY_uva_raw
-What: /sys/.../iio:deviceX/in_intensityY_uvb_raw
What: /sys/.../iio:deviceX/in_intensityY_duv_raw
KernelVersion: 3.4
Contact: linux-iio@vger.kernel.org
@@ -1597,9 +1595,8 @@ Description:
that measurements contain visible and infrared light
components or just infrared light, respectively. Modifier
uv indicates that measurements contain ultraviolet light
- components. Modifiers uva, uvb and duv indicate that
- measurements contain A, B or deep (C) ultraviolet light
- components respectively.
+ components. Modifier duv indicates that measurements
+ contain deep ultraviolet light components.
What: /sys/.../iio:deviceX/in_uvindex_input
KernelVersion: 4.6
diff --git a/drivers/iio/industrialio-core.c b/drivers/iio/industrialio-core.c
index 9a85752124dd..bce09d325142 100644
--- a/drivers/iio/industrialio-core.c
+++ b/drivers/iio/industrialio-core.c
@@ -117,8 +117,6 @@ static const char * const iio_modifier_names[] = {
[IIO_MOD_LIGHT_GREEN] = "green",
[IIO_MOD_LIGHT_BLUE] = "blue",
[IIO_MOD_LIGHT_UV] = "uv",
- [IIO_MOD_LIGHT_UVA] = "uva",
- [IIO_MOD_LIGHT_UVB] = "uvb",
[IIO_MOD_LIGHT_DUV] = "duv",
[IIO_MOD_QUATERNION] = "quaternion",
[IIO_MOD_TEMP_AMBIENT] = "ambient",
diff --git a/include/uapi/linux/iio/types.h b/include/uapi/linux/iio/types.h
index 5060963707b1..9c2ffdcd6623 100644
--- a/include/uapi/linux/iio/types.h
+++ b/include/uapi/linux/iio/types.h
@@ -91,8 +91,6 @@ enum iio_modifier {
IIO_MOD_CO2,
IIO_MOD_VOC,
IIO_MOD_LIGHT_UV,
- IIO_MOD_LIGHT_UVA,
- IIO_MOD_LIGHT_UVB,
IIO_MOD_LIGHT_DUV,
IIO_MOD_PM1,
IIO_MOD_PM2P5,
diff --git a/tools/iio/iio_event_monitor.c b/tools/iio/iio_event_monitor.c
index 8073c9e4fe46..2eaaa7123b04 100644
--- a/tools/iio/iio_event_monitor.c
+++ b/tools/iio/iio_event_monitor.c
@@ -105,8 +105,6 @@ static const char * const iio_modifier_names[] = {
[IIO_MOD_LIGHT_GREEN] = "green",
[IIO_MOD_LIGHT_BLUE] = "blue",
[IIO_MOD_LIGHT_UV] = "uv",
- [IIO_MOD_LIGHT_UVA] = "uva",
- [IIO_MOD_LIGHT_UVB] = "uvb",
[IIO_MOD_LIGHT_DUV] = "duv",
[IIO_MOD_QUATERNION] = "quaternion",
[IIO_MOD_TEMP_AMBIENT] = "ambient",
--
2.43.0
^ permalink raw reply related [flat|nested] 7+ messages in thread* Re: [PATCH] Revert "iio: add modifiers for A and B ultraviolet light"
2024-01-23 10:43 [PATCH] Revert "iio: add modifiers for A and B ultraviolet light" Paul Cercueil
@ 2024-01-23 10:51 ` Javier Carrasco
2024-01-23 10:55 ` Paul Cercueil
0 siblings, 1 reply; 7+ messages in thread
From: Javier Carrasco @ 2024-01-23 10:51 UTC (permalink / raw)
To: Paul Cercueil, Jonathan Cameron, Lars-Peter Clausen
Cc: Nuno Sa, linux-iio, linux-kernel
On 23.01.24 11:43, Paul Cercueil wrote:
> This reverts
> b89710bd215e ("iio: add modifiers for A and B ultraviolet light")
>
> Enum iio_modifer is *ABI*, you can't just decide to change all the
> values from one version to another, otherwise you break userspace.
> The new entries should have been added to the end of the enum.
>
> Signed-off-by: Paul Cercueil <paul@crapouillou.net>
> ---
> Documentation/ABI/testing/sysfs-bus-iio | 7 ++-----
> drivers/iio/industrialio-core.c | 2 --
> include/uapi/linux/iio/types.h | 2 --
> tools/iio/iio_event_monitor.c | 2 --
> 4 files changed, 2 insertions(+), 11 deletions(-)
>
> diff --git a/Documentation/ABI/testing/sysfs-bus-iio b/Documentation/ABI/testing/sysfs-bus-iio
> index 2e6d5ebfd3c7..7937bb4a4a68 100644
> --- a/Documentation/ABI/testing/sysfs-bus-iio
> +++ b/Documentation/ABI/testing/sysfs-bus-iio
> @@ -1587,8 +1587,6 @@ What: /sys/.../iio:deviceX/in_intensityY_raw
> What: /sys/.../iio:deviceX/in_intensityY_ir_raw
> What: /sys/.../iio:deviceX/in_intensityY_both_raw
> What: /sys/.../iio:deviceX/in_intensityY_uv_raw
> -What: /sys/.../iio:deviceX/in_intensityY_uva_raw
> -What: /sys/.../iio:deviceX/in_intensityY_uvb_raw
> What: /sys/.../iio:deviceX/in_intensityY_duv_raw
> KernelVersion: 3.4
> Contact: linux-iio@vger.kernel.org
> @@ -1597,9 +1595,8 @@ Description:
> that measurements contain visible and infrared light
> components or just infrared light, respectively. Modifier
> uv indicates that measurements contain ultraviolet light
> - components. Modifiers uva, uvb and duv indicate that
> - measurements contain A, B or deep (C) ultraviolet light
> - components respectively.
> + components. Modifier duv indicates that measurements
> + contain deep ultraviolet light components.
>
> What: /sys/.../iio:deviceX/in_uvindex_input
> KernelVersion: 4.6
> diff --git a/drivers/iio/industrialio-core.c b/drivers/iio/industrialio-core.c
> index 9a85752124dd..bce09d325142 100644
> --- a/drivers/iio/industrialio-core.c
> +++ b/drivers/iio/industrialio-core.c
> @@ -117,8 +117,6 @@ static const char * const iio_modifier_names[] = {
> [IIO_MOD_LIGHT_GREEN] = "green",
> [IIO_MOD_LIGHT_BLUE] = "blue",
> [IIO_MOD_LIGHT_UV] = "uv",
> - [IIO_MOD_LIGHT_UVA] = "uva",
> - [IIO_MOD_LIGHT_UVB] = "uvb",
> [IIO_MOD_LIGHT_DUV] = "duv",
> [IIO_MOD_QUATERNION] = "quaternion",
> [IIO_MOD_TEMP_AMBIENT] = "ambient",
> diff --git a/include/uapi/linux/iio/types.h b/include/uapi/linux/iio/types.h
> index 5060963707b1..9c2ffdcd6623 100644
> --- a/include/uapi/linux/iio/types.h
> +++ b/include/uapi/linux/iio/types.h
> @@ -91,8 +91,6 @@ enum iio_modifier {
> IIO_MOD_CO2,
> IIO_MOD_VOC,
> IIO_MOD_LIGHT_UV,
> - IIO_MOD_LIGHT_UVA,
> - IIO_MOD_LIGHT_UVB,
> IIO_MOD_LIGHT_DUV,
> IIO_MOD_PM1,
> IIO_MOD_PM2P5,
> diff --git a/tools/iio/iio_event_monitor.c b/tools/iio/iio_event_monitor.c
> index 8073c9e4fe46..2eaaa7123b04 100644
> --- a/tools/iio/iio_event_monitor.c
> +++ b/tools/iio/iio_event_monitor.c
> @@ -105,8 +105,6 @@ static const char * const iio_modifier_names[] = {
> [IIO_MOD_LIGHT_GREEN] = "green",
> [IIO_MOD_LIGHT_BLUE] = "blue",
> [IIO_MOD_LIGHT_UV] = "uv",
> - [IIO_MOD_LIGHT_UVA] = "uva",
> - [IIO_MOD_LIGHT_UVB] = "uvb",
> [IIO_MOD_LIGHT_DUV] = "duv",
> [IIO_MOD_QUATERNION] = "quaternion",
> [IIO_MOD_TEMP_AMBIENT] = "ambient",
Oops, sorry about that. You are right, I will send a new patch with the
new elements at the end of the enum. This patch should be applied to get
things right again, though.
Thanks and best regards,
Javier Carrasco
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH] Revert "iio: add modifiers for A and B ultraviolet light"
2024-01-23 10:51 ` Javier Carrasco
@ 2024-01-23 10:55 ` Paul Cercueil
2024-01-23 10:56 ` Javier Carrasco
0 siblings, 1 reply; 7+ messages in thread
From: Paul Cercueil @ 2024-01-23 10:55 UTC (permalink / raw)
To: Javier Carrasco, Jonathan Cameron, Lars-Peter Clausen
Cc: Nuno Sa, linux-iio, linux-kernel
Hi Javier,
Le mardi 23 janvier 2024 à 11:51 +0100, Javier Carrasco a écrit :
> On 23.01.24 11:43, Paul Cercueil wrote:
> > This reverts
> > b89710bd215e ("iio: add modifiers for A and B ultraviolet light")
> >
> > Enum iio_modifer is *ABI*, you can't just decide to change all the
> > values from one version to another, otherwise you break userspace.
> > The new entries should have been added to the end of the enum.
> >
> > Signed-off-by: Paul Cercueil <paul@crapouillou.net>
> > ---
> > Documentation/ABI/testing/sysfs-bus-iio | 7 ++-----
> > drivers/iio/industrialio-core.c | 2 --
> > include/uapi/linux/iio/types.h | 2 --
> > tools/iio/iio_event_monitor.c | 2 --
> > 4 files changed, 2 insertions(+), 11 deletions(-)
> >
> > diff --git a/Documentation/ABI/testing/sysfs-bus-iio
> > b/Documentation/ABI/testing/sysfs-bus-iio
> > index 2e6d5ebfd3c7..7937bb4a4a68 100644
> > --- a/Documentation/ABI/testing/sysfs-bus-iio
> > +++ b/Documentation/ABI/testing/sysfs-bus-iio
> > @@ -1587,8 +1587,6 @@
> > What: /sys/.../iio:deviceX/in_intensityY_raw
> > What: /sys/.../iio:deviceX/in_intensityY_ir_raw
> > What: /sys/.../iio:deviceX/in_intensityY_both_raw
> > What: /sys/.../iio:deviceX/in_intensityY_uv_raw
> > -What: /sys/.../iio:deviceX/in_intensityY_uva_raw
> > -What: /sys/.../iio:deviceX/in_intensityY_uvb_raw
> > What: /sys/.../iio:deviceX/in_intensityY_duv_raw
> > KernelVersion: 3.4
> > Contact: linux-iio@vger.kernel.org
> > @@ -1597,9 +1595,8 @@ Description:
> > that measurements contain visible and infrared
> > light
> > components or just infrared light, respectively.
> > Modifier
> > uv indicates that measurements contain ultraviolet
> > light
> > - components. Modifiers uva, uvb and duv indicate
> > that
> > - measurements contain A, B or deep (C) ultraviolet
> > light
> > - components respectively.
> > + components. Modifier duv indicates that
> > measurements
> > + contain deep ultraviolet light components.
> >
> > What: /sys/.../iio:deviceX/in_uvindex_input
> > KernelVersion: 4.6
> > diff --git a/drivers/iio/industrialio-core.c
> > b/drivers/iio/industrialio-core.c
> > index 9a85752124dd..bce09d325142 100644
> > --- a/drivers/iio/industrialio-core.c
> > +++ b/drivers/iio/industrialio-core.c
> > @@ -117,8 +117,6 @@ static const char * const iio_modifier_names[]
> > = {
> > [IIO_MOD_LIGHT_GREEN] = "green",
> > [IIO_MOD_LIGHT_BLUE] = "blue",
> > [IIO_MOD_LIGHT_UV] = "uv",
> > - [IIO_MOD_LIGHT_UVA] = "uva",
> > - [IIO_MOD_LIGHT_UVB] = "uvb",
> > [IIO_MOD_LIGHT_DUV] = "duv",
> > [IIO_MOD_QUATERNION] = "quaternion",
> > [IIO_MOD_TEMP_AMBIENT] = "ambient",
> > diff --git a/include/uapi/linux/iio/types.h
> > b/include/uapi/linux/iio/types.h
> > index 5060963707b1..9c2ffdcd6623 100644
> > --- a/include/uapi/linux/iio/types.h
> > +++ b/include/uapi/linux/iio/types.h
> > @@ -91,8 +91,6 @@ enum iio_modifier {
> > IIO_MOD_CO2,
> > IIO_MOD_VOC,
> > IIO_MOD_LIGHT_UV,
> > - IIO_MOD_LIGHT_UVA,
> > - IIO_MOD_LIGHT_UVB,
> > IIO_MOD_LIGHT_DUV,
> > IIO_MOD_PM1,
> > IIO_MOD_PM2P5,
> > diff --git a/tools/iio/iio_event_monitor.c
> > b/tools/iio/iio_event_monitor.c
> > index 8073c9e4fe46..2eaaa7123b04 100644
> > --- a/tools/iio/iio_event_monitor.c
> > +++ b/tools/iio/iio_event_monitor.c
> > @@ -105,8 +105,6 @@ static const char * const iio_modifier_names[]
> > = {
> > [IIO_MOD_LIGHT_GREEN] = "green",
> > [IIO_MOD_LIGHT_BLUE] = "blue",
> > [IIO_MOD_LIGHT_UV] = "uv",
> > - [IIO_MOD_LIGHT_UVA] = "uva",
> > - [IIO_MOD_LIGHT_UVB] = "uvb",
> > [IIO_MOD_LIGHT_DUV] = "duv",
> > [IIO_MOD_QUATERNION] = "quaternion",
> > [IIO_MOD_TEMP_AMBIENT] = "ambient",
> Oops, sorry about that. You are right, I will send a new patch with
> the
> new elements at the end of the enum. This patch should be applied to
> get
> things right again, though.
Np.
I notice now that we can't really apply the revert as the veml6075
driver is already using these enum values - so applying it would result
in build errors.
Can you just move these entries to the end of the enum in your new
patch?
>
> Thanks and best regards,
> Javier Carrasco
Cheers,
-Paul
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH] Revert "iio: add modifiers for A and B ultraviolet light"
2024-01-23 10:55 ` Paul Cercueil
@ 2024-01-23 10:56 ` Javier Carrasco
2024-01-27 15:52 ` Jonathan Cameron
0 siblings, 1 reply; 7+ messages in thread
From: Javier Carrasco @ 2024-01-23 10:56 UTC (permalink / raw)
To: Paul Cercueil, Jonathan Cameron, Lars-Peter Clausen
Cc: Nuno Sa, linux-iio, linux-kernel
On 23.01.24 11:55, Paul Cercueil wrote:
> Hi Javier,
>
> Le mardi 23 janvier 2024 à 11:51 +0100, Javier Carrasco a écrit :
>> On 23.01.24 11:43, Paul Cercueil wrote:
>>> This reverts
>>> b89710bd215e ("iio: add modifiers for A and B ultraviolet light")
>>>
>>> Enum iio_modifer is *ABI*, you can't just decide to change all the
>>> values from one version to another, otherwise you break userspace.
>>> The new entries should have been added to the end of the enum.
>>>
>>> Signed-off-by: Paul Cercueil <paul@crapouillou.net>
>>> ---
>>> Documentation/ABI/testing/sysfs-bus-iio | 7 ++-----
>>> drivers/iio/industrialio-core.c | 2 --
>>> include/uapi/linux/iio/types.h | 2 --
>>> tools/iio/iio_event_monitor.c | 2 --
>>> 4 files changed, 2 insertions(+), 11 deletions(-)
>>>
>>> diff --git a/Documentation/ABI/testing/sysfs-bus-iio
>>> b/Documentation/ABI/testing/sysfs-bus-iio
>>> index 2e6d5ebfd3c7..7937bb4a4a68 100644
>>> --- a/Documentation/ABI/testing/sysfs-bus-iio
>>> +++ b/Documentation/ABI/testing/sysfs-bus-iio
>>> @@ -1587,8 +1587,6 @@
>>> What: /sys/.../iio:deviceX/in_intensityY_raw
>>> What: /sys/.../iio:deviceX/in_intensityY_ir_raw
>>> What: /sys/.../iio:deviceX/in_intensityY_both_raw
>>> What: /sys/.../iio:deviceX/in_intensityY_uv_raw
>>> -What: /sys/.../iio:deviceX/in_intensityY_uva_raw
>>> -What: /sys/.../iio:deviceX/in_intensityY_uvb_raw
>>> What: /sys/.../iio:deviceX/in_intensityY_duv_raw
>>> KernelVersion: 3.4
>>> Contact: linux-iio@vger.kernel.org
>>> @@ -1597,9 +1595,8 @@ Description:
>>> that measurements contain visible and infrared
>>> light
>>> components or just infrared light, respectively.
>>> Modifier
>>> uv indicates that measurements contain ultraviolet
>>> light
>>> - components. Modifiers uva, uvb and duv indicate
>>> that
>>> - measurements contain A, B or deep (C) ultraviolet
>>> light
>>> - components respectively.
>>> + components. Modifier duv indicates that
>>> measurements
>>> + contain deep ultraviolet light components.
>>>
>>> What: /sys/.../iio:deviceX/in_uvindex_input
>>> KernelVersion: 4.6
>>> diff --git a/drivers/iio/industrialio-core.c
>>> b/drivers/iio/industrialio-core.c
>>> index 9a85752124dd..bce09d325142 100644
>>> --- a/drivers/iio/industrialio-core.c
>>> +++ b/drivers/iio/industrialio-core.c
>>> @@ -117,8 +117,6 @@ static const char * const iio_modifier_names[]
>>> = {
>>> [IIO_MOD_LIGHT_GREEN] = "green",
>>> [IIO_MOD_LIGHT_BLUE] = "blue",
>>> [IIO_MOD_LIGHT_UV] = "uv",
>>> - [IIO_MOD_LIGHT_UVA] = "uva",
>>> - [IIO_MOD_LIGHT_UVB] = "uvb",
>>> [IIO_MOD_LIGHT_DUV] = "duv",
>>> [IIO_MOD_QUATERNION] = "quaternion",
>>> [IIO_MOD_TEMP_AMBIENT] = "ambient",
>>> diff --git a/include/uapi/linux/iio/types.h
>>> b/include/uapi/linux/iio/types.h
>>> index 5060963707b1..9c2ffdcd6623 100644
>>> --- a/include/uapi/linux/iio/types.h
>>> +++ b/include/uapi/linux/iio/types.h
>>> @@ -91,8 +91,6 @@ enum iio_modifier {
>>> IIO_MOD_CO2,
>>> IIO_MOD_VOC,
>>> IIO_MOD_LIGHT_UV,
>>> - IIO_MOD_LIGHT_UVA,
>>> - IIO_MOD_LIGHT_UVB,
>>> IIO_MOD_LIGHT_DUV,
>>> IIO_MOD_PM1,
>>> IIO_MOD_PM2P5,
>>> diff --git a/tools/iio/iio_event_monitor.c
>>> b/tools/iio/iio_event_monitor.c
>>> index 8073c9e4fe46..2eaaa7123b04 100644
>>> --- a/tools/iio/iio_event_monitor.c
>>> +++ b/tools/iio/iio_event_monitor.c
>>> @@ -105,8 +105,6 @@ static const char * const iio_modifier_names[]
>>> = {
>>> [IIO_MOD_LIGHT_GREEN] = "green",
>>> [IIO_MOD_LIGHT_BLUE] = "blue",
>>> [IIO_MOD_LIGHT_UV] = "uv",
>>> - [IIO_MOD_LIGHT_UVA] = "uva",
>>> - [IIO_MOD_LIGHT_UVB] = "uvb",
>>> [IIO_MOD_LIGHT_DUV] = "duv",
>>> [IIO_MOD_QUATERNION] = "quaternion",
>>> [IIO_MOD_TEMP_AMBIENT] = "ambient",
>> Oops, sorry about that. You are right, I will send a new patch with
>> the
>> new elements at the end of the enum. This patch should be applied to
>> get
>> things right again, though.
>
> Np.
>
> I notice now that we can't really apply the revert as the veml6075
> driver is already using these enum values - so applying it would result
> in build errors.
>
> Can you just move these entries to the end of the enum in your new
> patch?
>
>>
>> Thanks and best regards,
>> Javier Carrasco
>
> Cheers,
> -Paul
Sure, I will do it right now.
Thank you again and best regards,
Javier Carrasco
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH] Revert "iio: add modifiers for A and B ultraviolet light"
2024-01-23 10:56 ` Javier Carrasco
@ 2024-01-27 15:52 ` Jonathan Cameron
2024-01-27 16:08 ` Javier Carrasco
0 siblings, 1 reply; 7+ messages in thread
From: Jonathan Cameron @ 2024-01-27 15:52 UTC (permalink / raw)
To: Javier Carrasco
Cc: Paul Cercueil, Lars-Peter Clausen, Nuno Sa, linux-iio,
linux-kernel
...
> > Can you just move these entries to the end of the enum in your new
> > patch?
> >
> >>
> >> Thanks and best regards,
> >> Javier Carrasco
> >
> > Cheers,
> > -Paul
> Sure, I will do it right now.
>
> Thank you again and best regards,
> Javier Carrasco
Hi Javier,
If you sent a v2, it didn't reach me or the list.
Jonathan
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] Revert "iio: add modifiers for A and B ultraviolet light"
2024-01-27 15:52 ` Jonathan Cameron
@ 2024-01-27 16:08 ` Javier Carrasco
2024-01-27 16:53 ` Jonathan Cameron
0 siblings, 1 reply; 7+ messages in thread
From: Javier Carrasco @ 2024-01-27 16:08 UTC (permalink / raw)
To: Jonathan Cameron
Cc: Paul Cercueil, Lars-Peter Clausen, Nuno Sa, linux-iio,
linux-kernel
On 27.01.24 16:52, Jonathan Cameron wrote:
> ...
>>> Can you just move these entries to the end of the enum in your new
>>> patch?
>>>
>>>>
>>>> Thanks and best regards,
>>>> Javier Carrasco
>>>
>>> Cheers,
>>> -Paul
>> Sure, I will do it right now.
>>
>> Thank you again and best regards,
>> Javier Carrasco
>
> Hi Javier,
>
> If you sent a v2, it didn't reach me or the list.
>
> Jonathan
>
>
Hi Jonathan,
I retrieved the To and CC with b4 and it seems that you were one of
them, but not the iio mailing list, maybe because the modification only
applies to an include. The linux-kernel mailing list was added, though:
https://lore.kernel.org/all/20240123-uva_uvb_fix-v1-1-5b9c25d50d90@gmail.com/T/
Could you apply that patch like that or do you need a RESEND to the
iio-mailing list?
Best regards,
Javier Carrasco
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] Revert "iio: add modifiers for A and B ultraviolet light"
2024-01-27 16:08 ` Javier Carrasco
@ 2024-01-27 16:53 ` Jonathan Cameron
0 siblings, 0 replies; 7+ messages in thread
From: Jonathan Cameron @ 2024-01-27 16:53 UTC (permalink / raw)
To: Javier Carrasco
Cc: Paul Cercueil, Lars-Peter Clausen, Nuno Sa, linux-iio,
linux-kernel
On Sat, 27 Jan 2024 17:08:28 +0100
Javier Carrasco <javier.carrasco.cruz@gmail.com> wrote:
> On 27.01.24 16:52, Jonathan Cameron wrote:
> > ...
> >>> Can you just move these entries to the end of the enum in your new
> >>> patch?
> >>>
> >>>>
> >>>> Thanks and best regards,
> >>>> Javier Carrasco
> >>>
> >>> Cheers,
> >>> -Paul
> >> Sure, I will do it right now.
> >>
> >> Thank you again and best regards,
> >> Javier Carrasco
> >
> > Hi Javier,
> >
> > If you sent a v2, it didn't reach me or the list.
> >
> > Jonathan
> >
> >
> Hi Jonathan,
>
> I retrieved the To and CC with b4 and it seems that you were one of
> them, but not the iio mailing list, maybe because the modification only
> applies to an include. The linux-kernel mailing list was added, though:
>
> https://lore.kernel.org/all/20240123-uva_uvb_fix-v1-1-5b9c25d50d90@gmail.com/T/
>
> Could you apply that patch like that or do you need a RESEND to the
> iio-mailing list?
Please resend. IIO patches are all tracked via patchwork which monitors the
linux-iio@vger.kernel.org list.
I also use several mail accounts, so only easy way to ensure it reaches
which ever machine I'm on today is to send it to the list.
Explains why I thought I'd seen it during the week but couldn't find it today though!
Thanks,
Jonathan
>
> Best regards,
> Javier Carrasco
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2024-01-27 16:53 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-01-23 10:43 [PATCH] Revert "iio: add modifiers for A and B ultraviolet light" Paul Cercueil
2024-01-23 10:51 ` Javier Carrasco
2024-01-23 10:55 ` Paul Cercueil
2024-01-23 10:56 ` Javier Carrasco
2024-01-27 15:52 ` Jonathan Cameron
2024-01-27 16:08 ` Javier Carrasco
2024-01-27 16:53 ` Jonathan Cameron
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox