linux-kernel - Re: [PATCH v3.1 1/2] device property: Add device_get_bd_address() and fwnode_get_bd

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20180927113829.GQ11965@kuha.fi.intel.com>
Date:   Thu, 27 Sep 2018 14:38:29 +0300
From:   Heikki Krogerus <heikki.krogerus@...ux.intel.com>
To:     Marcel Holtmann <marcel@...tmann.org>
Cc:     Matthias Kaehlcke <mka@...omium.org>,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        "Rafael J . Wysocki" <rafael@...nel.org>,
        Sakari Ailus <sakari.ailus@...ux.intel.com>,
        Marcin Wojtas <mw@...ihalf.com>,
        Andy Shevchenko Andy Shevchenko <andy.shevchenko@...il.com>,
        Sinan Kaya <okaya@...eaurora.org>,
        Johan Hedberg <johan.hedberg@...il.com>,
        Andrew Morton <akpm@...ux-foundation.org>,
        Pekka Enberg <penberg@...nel.org>,
        Masahiro Yamada <yamada.masahiro@...ionext.com>,
        Alexey Dobriyan <adobriyan@...il.com>,
        linux-kernel@...r.kernel.org, linux-bluetooth@...r.kernel.org,
        Balakrishna Godavarthi <bgodavar@...eaurora.org>,
        Loic Poulain <loic.poulain@...aro.org>,
        Brian Norris <briannorris@...omium.org>
Subject: Re: [PATCH v3.1 1/2] device property: Add device_get_bd_address()
 and fwnode_get_bd_address()

Hi Marcel,

On Thu, Sep 27, 2018 at 12:24:33PM +0200, Marcel Holtmann wrote:
> > Let's not fill property.c with framework specific helper functions any
> > more!
> > 
> > Those functions are completely bluetooth specific, so they do not
> > belong here. The fact that some other framework already managed to
> > slip their helpers in does not justify others to do the same.
> 
> so? The firmware guys decided to put MAC addresses and BD addresses into the
> firmware. So you have to deal with that.

I think you have misunderstood the point.

> Moving this into the Bluetooth subsystem is as pointless. I rather keep the
> accessor function to firmware specific data in one place and not spread around
> the whole tree. Especially once this is also provided via ACPI or some other
> means. I assumed that is what the whole device_get part was suppose to
> abstract.

Unified device property API defines a _generic_ API that can be used
by any type of device to access the device properties regardless of
the way the hardware is described.

Any device can use device_property_read_u8/u16/u32/u64/string()
functions, but only bluetooth devices can use device_get_bd_address().
Therefore that function does not belong to drivers/base/properties.c.


Thanks,

-- 
heikki