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 25337C07545 for ; Wed, 25 Oct 2023 11:30:35 +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=OwfTicPQm4Y5IMiNu5P9xez8BrF16YH4YxtvW3rkmig=; b=EsON09fTqDOKIg yKt8+4Q1XUZdp9brMjEY7zDdIeEuLqC24vNwt+xzPdLL1GM8gc1SDvQHmvgcL84ct/U05BpkYOBXB +I1G3FUCDPFoPirKO95NTkkICRp3DX+dPSA1HanATTA0/wA2ipXNX0Cx4tRpZLHtkyNwPAvDMqgc1 2rvt6K/10hFhPwiqi4BQx/wgP8D1p9yusnhjDV0WH8Zgx1P9IMrY9pwD7RjdcT3kA3FatNlFx1BSD S/RknCNdHcZNsHvKkHcNoOj3GDuloBLXY6pr6vl5EDnPZ5zcWuMDDqSSdTVtY5aeSOEJTfRIohO0Y vbnLbluahmFQk93/4dMg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qvc5Q-00CC0D-1w; Wed, 25 Oct 2023 11:30:08 +0000 Received: from mout.gmx.net ([212.227.17.20]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qvc5N-00CBwh-2x; Wed, 25 Oct 2023 11:30:07 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=gmx.net; s=s31663417; t=1698233375; x=1698838175; i=wahrenst@gmx.net; bh=wzqK2t8zSwabKpehgNyCy4Qys97N7Wvcq9t1ZXelOTI=; h=X-UI-Sender-Class:Date:Subject:To:Cc:References:From: In-Reply-To; b=mqL21FPzXaG9UDjo9qx0YFtjUaeBhzkTqj9y8Lk2Esek0fapJaN5kfb6jkG9o7Bh M6cWqQppCdkxja0PT4JtKW9N+U5VoRwYEEDhU3Ocy1YZe+dUhMgtCdUC6VbN+zHYc H2zt7BEntQO6dZYfSQW55JMe2Tw8jTDo3T3FQe5FlcvGChbc0utLOv302lfFP2cz3 YcyEWZta/5/iyZBejuoti7JTiMKA8EZ+f2YIMLbIZCms/kcWJ9P6NJbShKUc0gTav Gc50eo61BNQlzggHDtp3n276W5FD5qZC+uY2YgeY8aiZpDHbwVABb/nhLVKTG5dbC G4bKbODAR9ixtyWVbQ== X-UI-Sender-Class: 724b4f7f-cbec-4199-ad4e-598c01a50d3a Received: from [192.168.1.129] ([37.4.248.43]) by mail.gmx.net (mrgmx105 [212.227.17.168]) with ESMTPSA (Nemesis) id 1M3DJl-1qumCB34jl-003gar; Wed, 25 Oct 2023 13:29:34 +0200 Message-ID: Date: Wed, 25 Oct 2023 13:29:32 +0200 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH 0/1] staging: vc04_services: Use %p4cc to print fourcc 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, Phil Elwell , Dave Stevenson Cc: Stefan Wahren , Greg Kroah-Hartman , Dan Carpenter , Kieran Bingham , Laurent Pinchart , "Ricardo B . Marliere" , Sakari Ailus References: <20231025060717.71895-1-umang.jain@ideasonboard.com> From: Stefan Wahren In-Reply-To: <20231025060717.71895-1-umang.jain@ideasonboard.com> X-Provags-ID: V03:K1:IrkI2/n4fOM+fWg3bTBcGtjAWiONXUffbYvw5yNKS/T6R+tugQl mQlWd3hkuEj71C3imhFekLckYymOk1U3174Sww0busu2VJRbu2Jgq0jVV2HdBiENImzlDcM SgDBaxuWlkMMFNa9MixT9q/CkoLiW20nJibcMLffT0D1sGp7n69qmMFOPs7pQOZ894OCnyt s+acDm6JsJ9dR1P/CytvQ== UI-OutboundReport: notjunk:1;M01:P0:dhXhY+7uJ1M=;3lbfaKuJ0gdva1ufQA3NnXtsjJ7 MIkan6w2pgVUhI/Th1fWZkfJTgS4MtypA8KmcfUmfC6+KJNLrPemWFP7F/fh7EWqDkH16CH2E lT96nHmUVky4e79Apz7jyHC2mwGjWheK6eui9Rq5biZOAQlmM1EdZExBc+Ndr4M5H3MLjqNPm ad4i6g5V+Ip7vBjQg9ZRViy4xjIVhYvzqEeMauGArqQy+Nj9GaD77uWlrBkNxzm9bunyb6tsD IIFIqlXOGnGjrIFiDYCYkyA8ecCHi+OPICdWuNwPkhHAaI6Arr3ZSbLZfhtJ/E4vqPspuB57w CEL3/O1JQHXtSZMsEvObmH00Fm8M+7SPGHWU1LcysuZF5EAp9PY+ApSyeDo8f/q2Dmf+tCWll QOW3J1kq9QSybSOyixrSe6zxGZxXFIoEZQ9S5cQrtW2HjI/4l0uwqtIPs8n/dj1CVB7sXPBGW UJQsUTNYl/uup8WI7o14yX62eag9bK7wBWiaFJIkBOFBUr/Vtqlrre8E+mD7IfBv3+M8qVlrC p0hZ9i5bDUe2pvaVvqUchIYCtxy4Y7Kh1zBwqm8dXS2N99FW+aJFroxpRrysdUHxyH+NnyjxL LWsTZZL/9oO0vS4590b6Ot/dILD2PIBBVtckIjiV0EKn2FKzVCwUWLEIK28Pwzq7hYJUli3e2 pIbcQssi8p83jZHgA3AOMoJRIB43y1qWsNt5v8Qhzw4mAtKbRuBOO+hJdT+nHSa5adHvIgUWV HxXJVLadWtCfqS8ADzgR31Ha5jKT4+adY1UxbIKLlFMGrKeG9X7g3LHuHUm7y1fJvPWUBIUfF 8YsEHABulT3MevfKf7yPfKQYbB0LSUh8BCfVmxTRB+TNWsfQ9o0aq4/AZbUdxalUZQMzIDwjp G+/8qRy4hMeYoM5QxnlnpKJ6VMUhnHCkexlTrZqLNtaAYjWLalxzXLT0hr+ZzfOa0dExpzm03 Br/rd+TV6sWWQeOntP8j7VfcNRQ= X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20231025_043006_325871_A4EC13D1 X-CRM114-Status: GOOD ( 12.21 ) 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, [add Raspberry Pi guys] Am 25.10.23 um 08:07 schrieb Umang Jain: > The following patch drop VCHIQ_FOURCC_AS_4CHARS macro in favour of %p4cc > format modifier to print FourCC codes in the logs. > > *Before this patch* > `mailbox: vchiq_core_msg debug: Sent Msg DATA(5) to AUDS s:1 d:62 len:20` > > *After this patch* > bcm2835_vchiq 3f00b840.mailbox: vchiq_core_msg debug: Sent Msg DATA(5) to SDUA little-endian (0x41554453) s:1 d:62 len:20 > > The inversion of AUDS to SDUA as per usage of %p4cc > Does it hamper readability ? Feedback is appreciated. looks good to me. Any concerns Phil or Dave? > > As documented in the commit message, the 'entity' char array length is > increased to hold more characters for the log output. Not doing so, > causes kernel stack corruption at runtime. > > Based on top of: > - [PATCH v2 0/8] staging: vc04: Drop custom logging based on printk > > Umang Jain (1): > staging: vc04_services: Use %p4cc format modifier to print FourCC > codes > > .../interface/vchiq_arm/vchiq_arm.c | 20 +++++----- > .../interface/vchiq_arm/vchiq_core.c | 40 +++++++++---------- > .../interface/vchiq_arm/vchiq_core.h | 6 --- > .../interface/vchiq_arm/vchiq_dev.c | 7 ++-- > 4 files changed, 33 insertions(+), 40 deletions(-) > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel