[PATCH] drm/i915/pcode: Fix the coding style
Jani Nikula
jani.nikula at linux.intel.com
Wed Feb 12 09:53:27 UTC 2025
On Wed, 12 Feb 2025, "Gote, Nitin R" <nitin.r.gote at intel.com> wrote:
> Hi,
>
>> -----Original Message-----
>> From: Jani Nikula <jani.nikula at linux.intel.com>
>> Sent: Tuesday, February 11, 2025 6:56 PM
>> To: Andi Shyti <andi.shyti at linux.intel.com>; Gote, Nitin R
>> <nitin.r.gote at intel.com>
>> Cc: intel-gfx at lists.freedesktop.org
>> Subject: Re: [PATCH] drm/i915/pcode: Fix the coding style
>>
>> On Mon, 10 Feb 2025, Andi Shyti <andi.shyti at linux.intel.com> wrote:
>> > Hi Nitin,
>> >
>> > On Mon, Feb 10, 2025 at 07:39:24PM +0530, Nitin Gote wrote:
>> >> Prefer binary operator at the end of the previous line instead of
>> >> putting operator at the start of the next line as per coding style.
>> >
>> > I'm not finding any documentation for this change, where did you take
>> > it?
>>
>> If not documented, it's the prevalent style, anyway.
>
> Yeah, My bad I thought it is a standard coding style but it is not so.
> But still, I feel the more correct way is to use binary "|" operator in continuation of first line
> instead of second line.
Yes, that's what I said, that's the prevalent style.
>
> Thanks,
> Nitin
>>
>> BR,
>> Jani.
>>
>>
>> --
>> Jani Nikula, Intel
--
Jani Nikula, Intel
More information about the Intel-gfx
mailing list