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 phobos.denx.de (phobos.denx.de [85.214.62.61]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id CAE07C6FA8E for ; Wed, 21 Sep 2022 13:22:57 +0000 (UTC) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id 619A484CA2; Wed, 21 Sep 2022 15:22:47 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=quarantine dis=none) header.from=thorsis.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de Authentication-Results: phobos.denx.de; dkim=pass (2048-bit key; unprotected) header.d=thorsis.com header.i=@thorsis.com header.b="Hq4GwAVa"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id 7924084CA8; Wed, 21 Sep 2022 15:22:30 +0200 (CEST) Received: from mail.thorsis.com (mail.thorsis.com [92.198.35.195]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) (No client certificate requested) by phobos.denx.de (Postfix) with ESMTPS id 896AF84C96 for ; Wed, 21 Sep 2022 15:22:25 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=quarantine dis=none) header.from=thorsis.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=ada@thorsis.com Received: from adahl by ada.ifak-system.com with local (Exim 4.92) (envelope-from ) id 1oazgG-0001rI-BT; Wed, 21 Sep 2022 15:22:24 +0200 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=thorsis.com; s=default; t=1663766544; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:content-transfer-encoding:content-transfer-encoding; bh=lXKkJ7blX6HNE40MUZYWX3DTpmSqvcZ4jY5x08rMLLY=; b=Hq4GwAVaxtjWSyV3OlEVZ/VKdvu/GGGgA/kCEl9+vXq2VZFzJ80LfdamBuWk1MKwiqtIdt mup2O04g1Z1/xs86FiPNYb2c+sK/SX3cjyLrW0qQL2Gf38210aEIMQ+l4Z6HMy9MeUcdz4 ER2PJHHmPMg3ZizEfQOWYD34OPD87bkEFUDju0OiqgV5feRLnzAvzqziNRblq2tnQnziWQ POA5e+pnDSvgbXihCWn5wX64TuVTIwkYmtyDFWcQnlqbvyNOWBMy4I5+TIXd/+1ewu7EDX wIX7G+pBFfy1pjECy61N+YzzCuuZXXfuVz0rcYVGnEBa0eRxTL6QnXEKxJBLZQ== From: Alexander Dahl To: u-boot@lists.denx.de Cc: Michal Simek , Simon Glass Subject: [PATCH v2 0/8] Use logging feature instead of FPGA_DEBUG Date: Wed, 21 Sep 2022 15:22:08 +0200 Message-Id: <20220921132216.7089-1-ada@thorsis.com> Content-Transfer-Encoding: 8bit X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.39 Precedence: list List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" X-Virus-Scanned: clamav-milter 0.103.6 at phobos.denx.de X-Virus-Status: Clean Hei hei, while working on FPGA support for a new device I discovered debug logging in some FPGA drivers is still done as in the old days. Bring that to what I thougt would be the currently preferred approach. Notes: Adding those Kconfig symbols in patch 4 is just to be able to build those two old drivers. All drivers touched were build tested with sandbox64_defconfig and GCC8 on Debian GNU/Linux 10 (buster). Lines with other possibly questionable output were not touched, only what seemed to be designated debug output, and only for FPGA drivers having that ancient FPGA_DEBUG / PRINTF macros, so there's room for future improvements. Changelog: v1 -> v2: - Rebased on master - Added patch to introduce new FPGA uclass in front of the other patches - Use that new uclass as log category - Slightly reworded cover letter Greets Alex Alexander Dahl (8): dm: fpga: Introduce new uclass fpga: altera: Use logging feature instead of FPGA_DEBUG fpga: cyclon2: Use logging feature instead of FPGA_DEBUG fpga: Add missing Kconfig symbols for old FPGA drivers fpga: ACEX1K: Use logging feature instead of FPGA_DEBUG fpga: spartan2: Use logging feature instead of FPGA_DEBUG fpga: spartan3: Use logging feature instead of FPGA_DEBUG fpga: virtex2: Use logging feature instead of FPGA_DEBUG drivers/fpga/ACEX1K.c | 22 +++++++++------------- drivers/fpga/Kconfig | 12 ++++++++++++ drivers/fpga/altera.c | 13 ++++++------- drivers/fpga/cyclon2.c | 24 ++++++++++-------------- drivers/fpga/spartan2.c | 34 +++++++++++++++------------------- drivers/fpga/spartan3.c | 34 +++++++++++++++------------------- drivers/fpga/virtex2.c | 37 +++++++++++++++---------------------- include/dm/uclass-id.h | 1 + 8 files changed, 83 insertions(+), 94 deletions(-) base-commit: 12ed6d4911ced1df099a365e0a994b54211b60f3 -- 2.30.2