lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Wed, 27 Jul 2022 10:57:56 -0700 From: Bart Van Assche <bvanassche@....org> To: Stanley Chu <stanley.chu@...iatek.com>, linux-scsi@...r.kernel.org, linux-kernel@...r.kernel.org, martin.petersen@...cle.com, avri.altman@....com, alim.akhtar@...sung.com, jejb@...ux.ibm.com Cc: peter.wang@...iatek.com, chun-hung.wu@...iatek.com, alice.chao@...iatek.com, powen.kao@...iatek.com, mason.zhang@...iatek.com, qilin.tan@...iatek.com, lin.gui@...iatek.com, eddie.huang@...iatek.com, tun-yu.yu@...iatek.com, cc.chou@...iatek.com, chaotian.jing@...iatek.com, jiajie.hao@...iatek.com Subject: Re: [PATCH v2 2/5] scsi: ufs: ufs-mediatek: Provide detailed description for UIC errors On 7/26/22 20:44, Stanley Chu wrote: > diff --git a/drivers/ufs/host/ufs-mediatek.h b/drivers/ufs/host/ufs-mediatek.h > index aa26d415527b..9017ab8f9867 100644 > --- a/drivers/ufs/host/ufs-mediatek.h > +++ b/drivers/ufs/host/ufs-mediatek.h > @@ -26,6 +26,44 @@ > #define REG_UFS_DEBUG_SEL_B2 0x22D8 > #define REG_UFS_DEBUG_SEL_B3 0x22DC > > +/* > + * Details of UIC Errors > + */ > +static const u8 *ufs_uic_err_str[] = { > + "PHY Adapter Layer", > + "Data Link Layer", > + "Network Link Layer", > + "Transport Link Layer", > + "DME" > +}; Why type u8 for strings instead of char? Please define arrays in .c files. Otherwise one copy of the array will be included in each source file this header file is included in. Thanks, Bart.
Powered by blists - more mailing lists