linux-kernel - Re: [PATCH 3/6] hwmon: jc42: use bitops' sign

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]

Date:	Sun, 11 Jan 2015 08:52:57 -0800
From:	Guenter Roeck <linux@...ck-us.net>
To:	Martin Kepplinger <martink@...teo.de>
CC:	peterz@...radead.org, mingo@...nel.org, tytso@....edu,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH 3/6] hwmon: jc42: use bitops' sign_extend16

On 01/11/2015 02:34 AM, Martin Kepplinger wrote:
> Am 2014-12-16 um 08:17 schrieb Martin Kepplinger:
>> Am 2014-12-15 um 22:29 schrieb Guenter Roeck:
>>> On Mon, Dec 15, 2014 at 05:18:34PM +0100, Martin Kepplinger wrote:
>>>> ---
>>>
>>> Some description would be nice. Also, please consider adding
>>> relevant subsystem mailing lists and maintainers to your patches.
>>>
>>
>> I shouldn't have added the Signed-off-by line to some of them. Sorry.
>>
>> The driver-patches are meant to be examples of what can be changed if
>> the sign_extend functions are added. I don't know if they are taken and
>> planned to post the driver patches (probably more) thereafter, and of
>> course to the relevant people.
>
> Is this sign_extendXX() set of functions considered to be added to
> bitops.h ?
>
> Just checking if I can prepare some driver-patches (the ones I posted
> are just examples, meant to be re-sent to relevant maintainers when it's
> time), or not.
>

You should probably ask the question as response to patch 1/6.

In general, it might be useful to send example patches like this one as RFC.
Sending it as real patch and then saying "it is just an example" may result
in the entire series being ignored. This is not a matter of Signed-off or not
(originally I didn't even understand what you wanted to say with your reply),
but a matter of a non-misleading headline.

Thanks,
Guenter

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/