[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <afd4c1a8-4230-4757-a373-027812ec0ca6@www.fastmail.com>
Date: Mon, 04 Oct 2021 13:22:18 +0200
From: "Sven Peter" <sven@...npeter.dev>
To: "Arnd Bergmann" <arnd@...db.de>, "Wolfram Sang" <wsa@...nel.org>,
"Christian Zigotzky" <chzigotzky@...osoft.de>,
"Michael Ellerman" <mpe@...erman.id.au>,
"Benjamin Herrenschmidt" <benh@...nel.crashing.org>,
"Paul Mackerras" <paulus@...ba.org>,
"Olof Johansson" <olof@...om.net>,
"Hector Martin" <marcan@...can.st>,
"Mohamed Mediouni" <mohamed.mediouni@...amail.com>,
"Stan Skowronek" <stan@...ellium.com>,
"Mark Kettenis" <mark.kettenis@...all.nl>,
"Linux ARM" <linux-arm-kernel@...ts.infradead.org>,
"Alyssa Rosenzweig" <alyssa@...enzweig.io>,
linuxppc-dev <linuxppc-dev@...ts.ozlabs.org>,
"Linux I2C" <linux-i2c@...r.kernel.org>,
"Linux Kernel Mailing List" <linux-kernel@...r.kernel.org>,
"R.T.Dickinson" <rtd2@...a.co.nz>,
"Darren Stevens" <darren@...vens-zone.net>,
"Matthew Leaman" <matthew@...on.biz>,
"R.T.Dickinson" <rtd@...on.com>
Subject: Re: Add Apple M1 support to PASemi i2c driver
On Mon, Oct 4, 2021, at 13:20, Arnd Bergmann wrote:
> On Mon, Oct 4, 2021 at 11:55 AM Wolfram Sang <wsa@...nel.org> wrote:
>>
>>
>> > i2c-8 i2c PA Semi SMBus adapter at 0x(____ptrval____) I2C adapter
>> > i2c-9 i2c PA Semi SMBus adapter at 0x(____ptrval____) I2C adapter
>> > i2c-10 i2c PA Semi SMBus adapter at 0x(____ptrval____) I2C adapter
>>
>> As Sven correctly switched from %lx to %p, this is intended behaviour.
>> Run 'i2cdetect' as root to see the values again.
>
> I think the address could just get removed here, as this is clearly not helpful.
> port number, which is somewhat useful for identifying the device, now
> it's either the pointless string, or the virtual address that the
> device is mapped
> to, which is not helpful either and potentially leaks information about kernel
> internal structures.
Yeah, now that I'm looking at it again it doesn't make much sense to
include it there. Maybe just dev_name(smbus->dev) instead of the address?
Sven
Powered by blists - more mailing lists