[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 5/7] nios2: Add periodic timer emulation
From: |
Marek Vasut |
Subject: |
Re: [Qemu-devel] [PATCH 5/7] nios2: Add periodic timer emulation |
Date: |
Wed, 17 Aug 2016 22:54:08 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Icedove/45.2.0 |
On 08/17/2016 10:26 PM, Peter Maydell wrote:
> On 17 August 2016 at 21:19, Marek Vasut <address@hidden> wrote:
>> On 08/16/2016 11:38 PM, Dmitry Osipenko wrote:
>>> If you would prefer to keep error'ing out, then I can suggest to add some
>>> verbose message instead of the assertion, like:
>>>
>>> if (!t->freq_hz) {
>>> error_setg(errp, "altera_timer: \"clock-frequency\" property " \
>
> You don't need the '\' at the end of the line here.
>
>>> "must be provided");
>>> return;
>>> }
>>
>> That's better, thanks.
>>
>> btw breaking strings is frowned upon in linux/u-boot as it makes it
>> impossible to git grep for error message. Does the same apply to qemu?
>
> We tend to prefer to keep the lines below our line length limit,
> even if that means splitting them sometimes.
Is there any reason for that except for "we always did it that way" ?
Linux did it that way before, but then added this exception to make it
possible to grep for those strings.
Thanks
> thanks
> -- PMM
>
--
Best regards,
Marek Vasut
- Re: [Qemu-devel] [PATCH 5/7] nios2: Add periodic timer emulation, (continued)
- Re: [Qemu-devel] [PATCH 5/7] nios2: Add periodic timer emulation, Dmitry Osipenko, 2016/08/10
- Re: [Qemu-devel] [PATCH 5/7] nios2: Add periodic timer emulation, Marek Vasut, 2016/08/12
- Re: [Qemu-devel] [PATCH 5/7] nios2: Add periodic timer emulation, Dmitry Osipenko, 2016/08/15
- Re: [Qemu-devel] [PATCH 5/7] nios2: Add periodic timer emulation, Marek Vasut, 2016/08/16
- Re: [Qemu-devel] [PATCH 5/7] nios2: Add periodic timer emulation, Dmitry Osipenko, 2016/08/16
- Re: [Qemu-devel] [PATCH 5/7] nios2: Add periodic timer emulation, Marek Vasut, 2016/08/16
- Re: [Qemu-devel] [PATCH 5/7] nios2: Add periodic timer emulation, Dmitry Osipenko, 2016/08/16
- Re: [Qemu-devel] [PATCH 5/7] nios2: Add periodic timer emulation, Marek Vasut, 2016/08/17
- Re: [Qemu-devel] [PATCH 5/7] nios2: Add periodic timer emulation, Peter Maydell, 2016/08/17
- Re: [Qemu-devel] [PATCH 5/7] nios2: Add periodic timer emulation,
Marek Vasut <=
- Re: [Qemu-devel] [PATCH 5/7] nios2: Add periodic timer emulation, Dmitry Osipenko, 2016/08/18
- Re: [Qemu-devel] [PATCH 5/7] nios2: Add periodic timer emulation, Marek Vasut, 2016/08/18
- Re: [Qemu-devel] [PATCH 5/7] nios2: Add periodic timer emulation, Dmitry Osipenko, 2016/08/19
- Re: [Qemu-devel] [PATCH 5/7] nios2: Add periodic timer emulation, Marek Vasut, 2016/08/19