Re: [PATCH 0/3] iio: add labels with accel-location to bmc150 and kxcjk-1013 drivers

[Jonathan Cameron <jic23@kernel.org>]

On Fri, 12 Feb 2021 19:39:11 +0100
Hans de Goede <hdegoede@redhat.com> wrote:

> Hi,
> 
> On 2/12/21 7:33 PM, Jonathan Cameron wrote:
> > On Sun,  7 Feb 2021 17:08:58 +0100
> > Hans de Goede <hdegoede@redhat.com> wrote:
> >   
> >> Hi All,
> >>
> >> Here is a patch-set implementing the standardized "accel-display"
> >> and "accel-base" label sysfs-attributes defined in my earlier
> >> "[PATCH 1/2] iio: documentation: Document proximity sensor label use"
> >> "[PATCH 2/2] iio: documentation: Document accelerometer label use"
> >> series.
> >>
> >> This patch sets adds these labels to the bmc150 and kxcjk-1013 accel
> >> drivers.  
> > Series applied to the togreg branch of iio.git and pushed out as testing
> > for the autobuilders to poke at it.  
> 
> Thanks, much appreciated.
> 
> But this was sorta a follow-up to:
> 
> [PATCH 1/2] iio: documentation: Document proximity sensor label use
> [PATCH 2/2] iio: documentation: Document accelerometer label use
> 
> Where 2/2 defines the standardized label values which this series uses
> and AFAIK those have not been merged yet?
> 
> I guess we can always fix the labels if discussion surrounding those
> leads to standardizing on different label contents for this. But if
> the discussion surrounding those results in the conclusion to not
> use labels for this at all, while we already have this merged, then
> we probably need to revert 2/3 and 3/3 of this series...
> 
> But maybe I just missed the 2 documentation patches getting merged ?

Understood.   The changes for the docs patch that I was looking for
are just editorial, so I wasn't that fussed if it goes in slightly out of
order.

Jonathan

> 
> > Note we are almost certainly too late for coming merge window so this
> > will be next time around.  
> 
> That is fine.
> 
> Regards,
> 
> Hans
>