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 3D6ADC4167B for ; Thu, 7 Dec 2023 22:29:38 +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-Type: Content-Transfer-Encoding:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:From:References:Cc:To:Subject: MIME-Version:Date:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=CkX6qqF/ZbEaiP/Bd8o8IMabum4siDa8/kRyaKIrk+g=; b=r6akUcuBHWbrUV AkzlOMykU1pD6Kld4/cWsPG1W89hLTv6oNhU414iEkAhdTr5SYp4ASvwacimSFfuew08M3sIccqRn LeLsCL4bt9oSGu0mSPRu+OkOooEBJ+TGGcV+vP6BBBbe/Zg17Ug2EO4+Meej2UwWQODJTKvEazW35 y0ugRJoVc8N1bwKqRDR5C7ygSE85xFQpvaxW2rnh+B+Jl42OSoBNbf2arDZjOd/ojEYXFLLBIlXsC ILlxH5VaN+WY39mIYn1dHm5z3Fve8B+lkrMmIYr+Fqy357DzDx/zMJW+Y8nltJWrHNbi1SRHVxgA1 iMpbxxjFmm1+JkZKd7wQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1rBMro-00E872-2Y; Thu, 07 Dec 2023 22:29:12 +0000 Received: from mout.gmx.net ([212.227.17.21]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1rBMrk-00E84i-1y; Thu, 07 Dec 2023 22:29:10 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=gmx.net; s=s31663417; t=1701988130; x=1702592930; i=wahrenst@gmx.net; bh=yIkkOrIVK5rLyaPwPCK2hV1Fb4ExE8ngwDBFJOAF8Fk=; h=X-UI-Sender-Class:Date:Subject:To:Cc:References:From: In-Reply-To; b=T6KFEE4HPidHR016I4ycwemiTJygcGWNZefoqFFGcVAMopEfgcW5jqmJ9dxbJULW biFlVGVJ7QsCOIen+ikg5QjCidRDtA6R1joI5sn2L6GyfikPiPjuX4rcjWQwEElsn u4L5Bem3SCAJftjpIrOHz9RFmyyBRTzW7Eh5G+6bhGPIA9IhOuXNEhs1AzC/Zm7qR 6UKKOpR5k67e2zLCXrSVI90dn97e1PtHKX6wSiVuJ89A6yPa/WJRvXcEHN/fAMAj3 xlspy5N1bWjqfNGmaQXkR8pZmFIn981o0xINe8muLKMUS1F4GOp9ozzpiRL1NtoSl 1li6PiUy9A/Ut049Aw== X-UI-Sender-Class: 724b4f7f-cbec-4199-ad4e-598c01a50d3a Received: from [192.168.1.130] ([37.4.248.43]) by mail.gmx.net (mrgmx105 [212.227.17.168]) with ESMTPSA (Nemesis) id 1N2E1G-1rN1UC12Wk-013g3B; Thu, 07 Dec 2023 23:28:50 +0100 Message-ID: <12236d9d-fe18-439e-b622-0e64ef112400@gmx.net> Date: Thu, 7 Dec 2023 23:28:49 +0100 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v3 1/4] staging: vc04_services: Drop vchiq_log_error() in favour of dev_err Content-Language: en-US To: Umang Jain , linux-staging@lists.linux.dev, linux-rpi-kernel@lists.infradead.org, linux-media@vger.kernel.org, linux-arm-kernel@lists.infradead.org Cc: Stefan Wahren , Greg Kroah-Hartman , Dan Carpenter , Kieran Bingham , "Ricardo B . Marliere" References: <20231205084157.73819-1-umang.jain@ideasonboard.com> <20231205084157.73819-2-umang.jain@ideasonboard.com> <6a8566c0-bd26-4ec1-9f7d-d58bbed8411c@ideasonboard.com> <3920385c-3c10-471a-ba82-8e48ba34f6a5@gmx.net> <855cd4ee-86db-ca46-ce91-17303941ff82@ideasonboard.com> From: Stefan Wahren In-Reply-To: <855cd4ee-86db-ca46-ce91-17303941ff82@ideasonboard.com> X-Provags-ID: V03:K1:44rfj2Q0ulLso+QvKlBaVECfWyp/ydLX+DM+5YVRt8qT6jqLXOt W4ri/nGXtjDxw16G7DKpbfCb27/TeF345Lp1ZcKo9UrK2oqsUrDWE2nCofk+Jnt59CZr4Rn LaVZJnFA6U/IGR4Kgf6NePqoIzKt7NqR0ymyBdkNtLOG2a4X7ADZGYqj977C+scgoxlLUuf c3sdWJb1cu5t3z+DYQuqw== UI-OutboundReport: notjunk:1;M01:P0:VOC5bf0eE48=;r55t3dOV2K0SDP/4QuEGK+UhjQb NOWaxIYPLjK1gxgQ6IS7KY5wUWt/Ch4p8JIAteTPSt6vMeQINUlLAU/NcAO8Bt1S2DgwMSNTb D0ZpznihwyXz+zHah60d0FUrbz+MjCo1HiJGmbqGfbRZjF/D25DpkRuqXVwsm5tvlNRNu9O/H FZS5K/5s2nGQmwUYSsS58b8dzzJ1Nw/M2TY4/Dvo4l+K78G+j3+U/krUr/ioqiUSsMcusqB0T TWYGmmCS+yBvSHH9s9Qv7vYH+C5jWALyGHWVHn5qa+naozHoriKGNnJkDyRNI/YAcKHfa3v6U ztoxiF0ooddDoynnBeYIKPPvYxhG34HB/jU8bpeFe8ENtCOfUgTFSac0miPg+80MsFGAxJh0K +Pl3dxLYAnbm56jgoIkafgymqK4mhsDiuBX+rx+RXsnfdEW9JIXKQHrobLyHmTKSQF2MhXPuv EITjH1uEZwAVLx8Ks03/oB6iP6UYqT726X9WuH9eomsaXfcR0qiz3nUirnbwrOJ+17Qu/qkMM Vs1YFhddFe9iO/qoerdedCFvDbkW6MENhf2RfcwtOj8Uw8I6of7gcPMqyCKmnZFlwNo1OBXg6 v3fC7Zohoaf0c1w8vKtVMYx2PoRuikipQnqg0GiqdY5xpzlIPbEOW0t528+09AoerilvLPWf1 kcPOL8eLnjwwKcCpmuydhvmXp/sU2pj86SBiTaaa2wDMZRBxKQQwGnK0UqsA12WeFa3HA4WQn EtYPcB+0ppxhB46/xAX28Mc6DbbvpKGBWVBmCDtJK2EX7ULmXlWcOiQPDd7QPyu73uDUSV74T AHcrfQLZhM6zfZ5BlA77lgKznrt4PJ+itHkwFvZmM21Eq/Uz6YD343SSdJrI7JvqzciiPeB+7 z7DSsMdeqN4nkF9krJEJtU8O/SRCMy3yRZZkv+KyWqxgdpRzjuz5KiRtmua425dXxIodX+wir 1gyoRATmGThwp8C0fQ5xiYr9gAM= X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20231207_142909_123001_DB7D18EF X-CRM114-Status: GOOD ( 13.94 ) 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-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi Umang, Am 07.12.23 um 09:41 schrieb Umang Jain: > Hi Stefan > > On 12/7/23 3:04 AM, Stefan Wahren wrote: >> Hi Umang, >> >> Am 06.12.23 um 06:27 schrieb Umang Jain: >>> Hi Stefan >>> >>> On 12/6/23 1:02 AM, Stefan Wahren wrote: >>>> Hi Umang, >>>> >>>> Am 05.12.23 um 09:41 schrieb Umang Jain: >>>>> Drop vchiq_log_error() macro which wraps dev_dbg(). Introduce the >>>>> usage >>>>> of dev_err() directly. >>>> sorry, i missed to review the last change. So the change won't be that >>>> trivial. >>> >>> All the changes you said are valid and I have taken a note of it. >>> >>> However, I think it would best that if we can address them in a >>> separate series on top of this (and I will do it). It would seem to be >>> streamlined on top of this. >> i would address the comments as separate patches and then finally >> convert the trivial rest with such a patch within one series. This >> avoids unnecessary changes and reviewes. But that's just my opinion. >> > > Series is merged by Greg KH (Thank you). > > Do you have more comments on 3/4 and 4/4 or are they waiting on your > review queue. Let me know so that I can prepare a follow up series in > -one- go. i was busy. I will look at the rest tomorrow. Best regards > > thanks! >> Best regards > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel