On 02.12.19 14:37, Ardelean, Alexandru wrote:
> On Mon, 2019-12-02 at 14:01 +0100, Lars Möllendorf wrote:
>>
>
> Hey Lars,
>
> Thank you for the patch.
>
> Could you add a description of the problem in the commit description?
> You did a really great job on describing it via email earlier, and it would
> be great to have it in the git history as well.
Is the description in my latest patch ok?
> Also, this patch is marked 1/3 ; curios: are there 2 more patches in a set?
> Sometimes, some patches get lost via email clients/servers.
No, there is only one patch. I just did not use `git format-patch`
correctly in my first attempt.
> Maybe Jonathan [or someone else] has some more points to this.
Anything else I can do to improve the patch? It is the first time I am
trying to submit a patch to the kernel. Would be nice to know if it is
accepted and if not, why. So I can learn from my mistakes.
>
> Thanks
> Alex
>
>> Signed-off-by: Lars Möllendorf <lars.moellendorf@xxxxxxxxxx>
>> ---
>> drivers/iio/industrialio-buffer.c | 6 +++++-
>> 1 file changed, 5 insertions(+), 1 deletion(-)
>>
>> diff --git a/drivers/iio/industrialio-buffer.c
>> b/drivers/iio/industrialio-buffer.c
>> index 5d05c38c4ba9..2f037cd59d53 100644
>> --- a/drivers/iio/industrialio-buffer.c
>> +++ b/drivers/iio/industrialio-buffer.c
>> @@ -546,7 +546,7 @@ static int iio_compute_scan_bytes(struct iio_dev
>> *indio_dev,
>> const unsigned long *mask, bool timestamp)
>> {
>> unsigned bytes = 0;
>> - int length, i;
>> + int length, i, largest = 0;
>>
>> /* How much space will the demuxed element take? */
>> for_each_set_bit(i, mask,
>> @@ -554,13 +554,17 @@ static int iio_compute_scan_bytes(struct iio_dev
>> *indio_dev,
>> length = iio_storage_bytes_for_si(indio_dev, i);
>> bytes = ALIGN(bytes, length);
>> bytes += length;
>> + largest = max(largest, length);
>> }
>>
>> if (timestamp) {
>> length = iio_storage_bytes_for_timestamp(indio_dev);
>> bytes = ALIGN(bytes, length);
>> bytes += length;
>> + largest = max(largest, length);
>> }
>> +
>> + bytes = ALIGN(bytes, largest);
>> return bytes;
>> }
>>
>> --
>> 2.23.0
>>
--
Lars Möllendorf, B. Eng.
Tel.: +49 (0) 7641 93500-425
Fax: +49 (0) 7641 93500-999
E-Mail: lars.moellendorf@xxxxxxxxxx <mailto:lars.moellendorf@xxxxxxxxxx>
Website: www.plating.de <http://www.plating.de>
--------------------------------
plating electronic GmbH - Amtsgericht Freiburg - HRB Nr. 260 592 /
Geschäftsführer Karl Rieder / Rheinstraße 4 – 79350 Sexau – Tel.:+49 (0)
7641 – 93500-0
--------------------------------
Der Inhalt dieser E-Mail ist vertraulich und ausschließlich für den
bezeichneten Adressaten bestimmt. Wenn Sie nicht der vorgesehene
Adressat dieser E-Mail oder dessen Vertreter sein sollten, so beachten
Sie bitte, dass jede Form der Kenntnisnahme, Veröffentlichung,
Vervielfältigung oder Weitergabe des Inhalts dieser E-Mail unzulässig
ist. Wir bitten Sie, sich in diesem Fall mit dem Absender der E-Mail in
Verbindung zu setzen. Aussagen gegenüber dem Adressaten unterliegen den
Regelungen des zugrundeliegenden Angebotes bzw. Auftrags, insbesondere
den Allgemeinen Geschäftsbedingungen und der individuellen
Haftungsvereinbarung. Der Inhalt der E-Mail ist nur rechtsverbindlich,
wenn er unsererseits durch einen Brief oder ein Telefax entsprechend
bestaetigt wird.
The information contained in this email is confidential. It is intended
solely for the addressee. Access to this email by anyone else is
unauthorized. If you are not the intended recipient, any form of
disclosure, reproduction, distribution or any action taken or refrained
from in reliance on it, is prohibited and may be unlawful. Please notify
the sender immediately. All statements of opinion or advice directed via
this email to our clients are subject to the terms and conditions
expressed in the governing client engagement letter. The content of this
email is not legally binding unless confirmed by letter or fax.
Although plating electronic GmbH attempts to sweep e-mail and
attachments for viruses, it does not guarantee that either are
virus-free and accepts no liability for any damage sustained as a result
of viruses.
Attachment:
pEpkey.asc
Description: application/pgp-keys
