linux-kernel - Re: [PATCH] staging: octeon: Add Lanai board USB clock type

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]

Message-ID: <20101012155426.GA15238@kroah.com>
Date:	Tue, 12 Oct 2010 08:54:26 -0700
From:	Greg KH <greg@...ah.com>
To:	David Daney <ddaney@...iumnetworks.com>
Cc:	Bernhard Walle <walle@...science.de>, devel@...verdev.osuosl.org,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH] staging: octeon: Add Lanai board USB clock type

On Mon, Oct 11, 2010 at 10:13:37AM -0700, David Daney wrote:
> On 10/11/2010 10:05 AM, Bernhard Walle wrote:
>> Am 11.10.2010 18:53, schrieb David Daney:
>>> On 10/11/2010 03:09 AM, Bernhard Walle wrote:
>>>> Add the LANAI2 board to __cvmx_helper_board_usb_get_clock_type(). It
>>>> also uses the USB_CLOCK_TYPE_CRYSTAL_12 clock.
>>>>
>>>> Signed-off-by: Bernhard Walle<walle@...science.de>
>>>> ---
>>>>    drivers/staging/octeon/cvmx-helper-board.c |    1 +
>>>>    1 files changed, 1 insertions(+), 0 deletions(-)
>>>
>>> NAK.
>>>
>>> We don't currently support USB, so churning up this file doesn't help
>>> anything.
>>
>> Well, agreed, but why not removing that code entirely then?
>>
>
> That is the eventual plan.

Why not do that now?

Or add USB support to the driver, but don't leave it in the in-kernel
driver and expect users to know that this is something they shouldn't be
doing.

Also, why aren't you going to support the USB devices?  What's wrong
with that?

thanks,

greg k-h
--
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/