Re: [PATCH 1/5] usb: host: xhci: use ffs() in xhci_mem_init()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: "Linyu Yuan (QUIC)" <quic_linyyuan@xxxxxxxxxxx>, Mathias Nyman <mathias.nyman@xxxxxxxxx>, Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
Subject: Re: [PATCH 1/5] usb: host: xhci: use ffs() in xhci_mem_init()
From: Sergei Shtylyov <sergei.shtylyov@xxxxxxxxx>
Date: Wed, 16 Feb 2022 12:53:08 +0300
Cc: "linux-usb@xxxxxxxxxxxxxxx" <linux-usb@xxxxxxxxxxxxxxx>, "Jack Pham (QUIC)" <quic_jackp@xxxxxxxxxxx>
In-reply-to: <DM8PR02MB819861F71713D5539EF66D12E3359@DM8PR02MB8198.namprd02.prod.outlook.com>
References: <1644994755-12975-1-git-send-email-quic_linyyuan@quicinc.com> <1644994755-12975-2-git-send-email-quic_linyyuan@quicinc.com> <4688f5bb-c0fd-bbce-de1f-a554d543ed03@gmail.com> <DM8PR02MB819861F71713D5539EF66D12E3359@DM8PR02MB8198.namprd02.prod.outlook.com>
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.10.1

On 2/16/22 12:47 PM, Linyu Yuan (QUIC) wrote:

> that's correct,  from my view, one line is good,
> 
> What's your suggestion ? two lines ?

   Yes, and it is not just my suggestion -- it's the kernel coding style.

[...]

MBR, Sergey

Follow-Ups:
- RE: [PATCH 1/5] usb: host: xhci: use ffs() in xhci_mem_init()
  - From: Linyu Yuan (QUIC)

References:
- [PATCH 0/5] usb: xhci: fix of some code/comment
  - From: Linyu Yuan
- [PATCH 1/5] usb: host: xhci: use ffs() in xhci_mem_init()
  - From: Linyu Yuan
- Re: [PATCH 1/5] usb: host: xhci: use ffs() in xhci_mem_init()
  - From: Sergei Shtylyov
- RE: [PATCH 1/5] usb: host: xhci: use ffs() in xhci_mem_init()
  - From: Linyu Yuan (QUIC)

Prev by Date: Re: [PATCH v10 6/6] usb: dwc3: qcom: Enable the interrupts during probe
Next by Date: RE: [PATCH 1/5] usb: host: xhci: use ffs() in xhci_mem_init()
Previous by thread: RE: [PATCH 1/5] usb: host: xhci: use ffs() in xhci_mem_init()
Next by thread: RE: [PATCH 1/5] usb: host: xhci: use ffs() in xhci_mem_init()
Index(es):
- Date
- Thread

[Index of Archives] [Linux Media] [Linux Input] [Linux Audio Users] [Yosemite News] [Linux Kernel] [Linux SCSI] [Old Linux USB Devel Archive]