diff mbox series

[v2,01/15] libpdbg: Convert printf to DEBUG statements

Message ID 20181109071011.253734-2-amitay@ozlabs.org
State Superseded
Headers show
Series Device tree path based targeting | expand

Checks

Context Check Description
snowpatch_ozlabs/apply_patch success master/apply_patch Successfully applied
snowpatch_ozlabs/snowpatch_job_snowpatch-pdbg success Test snowpatch/job/snowpatch-pdbg on branch master

Commit Message

Amitay Isaacs Nov. 9, 2018, 7:09 a.m. UTC
Signed-off-by: Amitay Isaacs <amitay@ozlabs.org>
---
 libpdbg/fake.c | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

Comments

Alistair Popple Nov. 13, 2018, 1:58 a.m. UTC | #1
Reviewed-by: Alistair Popple <alistair@popple.id.au>

On Friday, 9 November 2018 6:09:57 PM AEDT Amitay Isaacs wrote:
> Signed-off-by: Amitay Isaacs <amitay@ozlabs.org>
> ---
>  libpdbg/fake.c | 12 +++++++-----
>  1 file changed, 7 insertions(+), 5 deletions(-)
> 
> diff --git a/libpdbg/fake.c b/libpdbg/fake.c
> index 58e7e18..a816ff5 100644
> --- a/libpdbg/fake.c
> +++ b/libpdbg/fake.c
> @@ -13,20 +13,22 @@
>   * see the license for the specific language governing permissions and
>   * limitations under the license.
>   */
> -#include "operations.h"
>  #include <stdio.h>
>  #include <inttypes.h>
> 
> +#include "libpdbg.h"
> +#include "operations.h"
> +
>  static int fake_fsi_read(struct fsi *fsi, uint32_t addr, uint32_t *value)
>  {
>  	*value = 0xfeed0cfa;
> -	printf("fake_fsi_read(0x%04" PRIx32 ", 0x%04" PRIx32 ")\n", addr, 
*value);
> +	PR_DEBUG("fake_fsi_read(0x%04" PRIx32 ", 0x%04" PRIx32 ")\n", addr,
> *value); return 0;
>  }
> 
>  static int fake_fsi_write(struct fsi *fsi, uint32_t addr, uint32_t value)
>  {
> -	printf("fake_fsi_write(0x%04" PRIx32 ", 0x%04" PRIx32 ")\n", addr, 
value);
> +	PR_DEBUG("fake_fsi_write(0x%04" PRIx32 ", 0x%04" PRIx32 ")\n", addr,
> value); return 0;
>  }
> 
> @@ -44,13 +46,13 @@ DECLARE_HW_UNIT(fake_fsi);
>  static int fake_pib_read(struct pib *pib, uint64_t addr, uint64_t *value)
>  {
>  	*value = 0xdeadbeef;
> -	printf("fake_pib_read(0x%08" PRIx64 ", 0x%08" PRIx64 ")\n", addr, 
*value);
> +	PR_DEBUG("fake_pib_read(0x%08" PRIx64 ", 0x%08" PRIx64 ")\n", addr,
> *value); return 0;
>  }
> 
>  static int fake_pib_write(struct pib *pib, uint64_t addr, uint64_t value)
>  {
> -	printf("fake_pib_write(0x%08" PRIx64 ", 0x%08" PRIx64 ")\n", addr, 
value);
> +	PR_DEBUG("fake_pib_write(0x%08" PRIx64 ", 0x%08" PRIx64 ")\n", addr,
> value); return 0;
>  }
diff mbox series

Patch

diff --git a/libpdbg/fake.c b/libpdbg/fake.c
index 58e7e18..a816ff5 100644
--- a/libpdbg/fake.c
+++ b/libpdbg/fake.c
@@ -13,20 +13,22 @@ 
  * see the license for the specific language governing permissions and
  * limitations under the license.
  */
-#include "operations.h"
 #include <stdio.h>
 #include <inttypes.h>
 
+#include "libpdbg.h"
+#include "operations.h"
+
 static int fake_fsi_read(struct fsi *fsi, uint32_t addr, uint32_t *value)
 {
 	*value = 0xfeed0cfa;
-	printf("fake_fsi_read(0x%04" PRIx32 ", 0x%04" PRIx32 ")\n", addr, *value);
+	PR_DEBUG("fake_fsi_read(0x%04" PRIx32 ", 0x%04" PRIx32 ")\n", addr, *value);
 	return 0;
 }
 
 static int fake_fsi_write(struct fsi *fsi, uint32_t addr, uint32_t value)
 {
-	printf("fake_fsi_write(0x%04" PRIx32 ", 0x%04" PRIx32 ")\n", addr, value);
+	PR_DEBUG("fake_fsi_write(0x%04" PRIx32 ", 0x%04" PRIx32 ")\n", addr, value);
 	return 0;
 }
 
@@ -44,13 +46,13 @@  DECLARE_HW_UNIT(fake_fsi);
 static int fake_pib_read(struct pib *pib, uint64_t addr, uint64_t *value)
 {
 	*value = 0xdeadbeef;
-	printf("fake_pib_read(0x%08" PRIx64 ", 0x%08" PRIx64 ")\n", addr, *value);
+	PR_DEBUG("fake_pib_read(0x%08" PRIx64 ", 0x%08" PRIx64 ")\n", addr, *value);
 	return 0;
 }
 
 static int fake_pib_write(struct pib *pib, uint64_t addr, uint64_t value)
 {
-	printf("fake_pib_write(0x%08" PRIx64 ", 0x%08" PRIx64 ")\n", addr, value);
+	PR_DEBUG("fake_pib_write(0x%08" PRIx64 ", 0x%08" PRIx64 ")\n", addr, value);
 	return 0;
 }