Re: [PATCH 6/6] scsi: sd: Do not split error messages

Bart Van Assche
2024-08-05T23:38:35Z

On 7/30/24 4:47 PM, Damien Le Moal wrote:
> On 7/31/24 06:00, Bart Van Assche wrote:
>> @@ -2856,8 +2857,8 @@ sd_read_capacity(struct scsi_disk *sdkp, struct queue_limits *lim,
>> */
>> if (sdp->fix_capacity ||
>> (sdp->guess_capacity && (sdkp->capacity & 0x01))) {
>> - sd_printk(KERN_INFO, sdkp, "Adjusting the sector count "
>> - "from its reported value: %llu\n",
>> + sd_printk(KERN_INFO, sdkp,
>> + "Adjusting the sector count from its reported value: %llu\n",
>> (unsigned long long) sdkp->capacity);
>
> Can you fix the alignment of the format string while at it ? No need for that
> extra tab, removing it will make the line shorter.

Sure, I will fix the alignment of the sd_printk() arguments. Thanks for
the reviews!

Bart.