From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id E6389C38142 for ; Mon, 23 Jan 2023 16:56:27 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:Message-ID:Date:To:Cc:From:Subject: References:In-Reply-To:MIME-Version:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=+qZ9TV71QqvZ8DjU0RAOiOrGSSaBgBeq9iEcwIeMosc=; b=V1tpk+MkjvdHZT BaJhYhzIlXZrMT5754XHq8ZxW98Kf2R4fXx5+cBKjtvOFpVXC3YwSi1Dx+F7wT/4LYKE7HaZ7wDvj Reg6frPNcwiOe1HtPt1n+OQakaKWFAW/KEfQgKF5NxYuC+i7ZGFlvJJEnuM/kBOqPLHlovmvWimpc 437qjz01l/autswKV78mPuHmlR7TBo/41aJXuIwtReOjDjLRQIQVfR4s6l+eS0Q7zAmAOQ/sFQUmK Q7E+10Cny0Kv1zXVzwxZpn23huqW+r1ndC8ExQ/w2NXOAcsnq7IOp0R2O9LlHcj/rOfi7825YLsUb r5gADk/VG9DpCj9XY8zQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1pK060-000imZ-7U; Mon, 23 Jan 2023 16:55:00 +0000 Received: from perceval.ideasonboard.com ([213.167.242.64]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1pK05Y-000ia4-3S; Mon, 23 Jan 2023 16:54:33 +0000 Received: from pendragon.ideasonboard.com (cpc89244-aztw30-2-0-cust3082.18-1.cable.virginm.net [86.31.172.11]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id F371D2D9; Mon, 23 Jan 2023 17:54:26 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1674492867; bh=DEIsYAdzjE5R62Z2EIoklE1UuDpML7s7ZziMIR2i0Tk=; h=In-Reply-To:References:Subject:From:Cc:To:Date:From; b=spZCZjqgWsIrZZ8DCDJgVSoz+o5bHQbhdEJSHZdi17ndQeSv68aJl8jFYraY+0U7c XVqRgFVcSs5MHwsvLK+bYyj48Fa8GqvjJ0E6JOTmz/NGIBmUdIxkkSmjkeYuhgcmjF PqFDkWYDXv199SXuQapuKlzksaPPh5dMS/wBqxHs= MIME-Version: 1.0 In-Reply-To: References: <20230118115810.21979-1-umang.jain@ideasonboard.com> <167440481093.3533645.5465896340839457121@Monstersaurus> Subject: Re: [RFC PATCH 0/4] Drop custom logging From: Kieran Bingham Cc: Greg Kroah-Hartman , Florian Fainelli , Adrien Thierry , Dan Carpenter , Dave Stevenson , linux-arm-kernel@lists.infradead.org, Laurent Pinchart , linux-kernel@vger.kernel.org, linux-rpi-kernel@lists.infradead.org, linux-staging@lists.linux.dev, linux-media@vger.kernel.org, Phil Elwell To: Stefan Wahren , Umang Jain Date: Mon, 23 Jan 2023 16:54:24 +0000 Message-ID: <167449286433.135671.17893045578331831111@Monstersaurus> User-Agent: alot/0.10 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230123_085432_348129_D55A6722 X-CRM114-Status: GOOD ( 26.64 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi Stefan, Quoting Stefan Wahren (2023-01-22 18:07:03) > Hi Kieran, > > Am 22.01.23 um 17:26 schrieb Kieran Bingham: > > Hi Stefan, > > > > Quoting Stefan Wahren (2023-01-22 14:21:05) > >> Hi Umang, > >> > >> Am 18.01.23 um 12:58 schrieb Umang Jain: > >>> Drop custom logging from the vchiq interface. > >>> Mostly of them are replaced with dev_dbg and friends > >>> and/or pr_info and friends. > >>> > >>> The debugfs log levels (in 4/4) are mapped to kernel > >>> logs levels (coming from include/linux/kern_levels.h) > >>> Would like some thoughts on it as I am not sure (hence > >>> marking this is RFC) > >>> > >>> From drivers/staging/vc04_services/interface/TODO: > >>> > >>> """ > >>> * Cleanup logging mechanism > >>> > >>> The driver should probably be using the standard kernel logging mechanisms > >>> such as dev_info, dev_dbg, and friends. > >>> """ > >> at first i want to thank you for the work on vchiq so far. > >> > >> There is something which is not directly related to this series, but it > >> is also about debugging. The driver has a buffer which is accessed by > >> it's own DEBUG_* macros. The content of this debug buffer can be dumped > >> via the /dev/vchiq which is also used by ioctl. I would appreciate to > >> move this dump feature into a new debugfs entry. > > Do you have a full list of the tasks you'd like to see completed ? > > (including/or above drivers/staging/vc04_services/interface/TODO) > > i consider every point except of point 1 (importing new drivers) as > necessary to leave staging. Thanks > Additionally there is the additional point (i can add them to the TODO) > above. Unfortunately i don't have a complete insight, how vchiq should > be to be acceptable. Sorry, if i can't help you further with possible > resource planning. > > Are some points on the TODO list unclear? I believe the list is fine, but I was enquiring if there were anymore additional tasks above the scope listed in drivers/staging/vc04_services/interface/TODO which are required. I don't think you need to send a patch for the task above - unless there are a lot more tasks required, or it becomes too much to do now. The goal is to get the ISP upstream to support libcamera, and it would help to know how deep the rabbit hole really is ;-) -- Kieran _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel