Re: [PATCH] megaraid_sas: Add support for MegaRAID Aero controllers

2018-11-09 Thread Shivasharan Srikanteshwara
On Fri, Nov 9, 2018 at 7:43 AM Martin K. Petersen
 wrote:
> Shivasharan,
>
> > This patch adds support for MegaRAID Aero controller PCI IDs.
> > Throw a warning message when a Configurable secure type controller is
> > encountered.
>
> > + dev_warn(>dev, "Adapter is in configurable secure 
> > mode\n");
>
> Why warn and not info?

Hi Martin,
Agree that a dev_info would be the better choice here instead of warn.
Will send out v2 of this patch with the change.

Thanks,
Shivasharan
>
> --
> Martin K. Petersen  Oracle Linux Engineering


Re: [PATCH] megaraid_sas: Add support for MegaRAID Aero controllers

2018-11-08 Thread Martin K. Petersen


Shivasharan,

> This patch adds support for MegaRAID Aero controller PCI IDs.
> Throw a warning message when a Configurable secure type controller is
> encountered.

> + dev_warn(>dev, "Adapter is in configurable secure 
> mode\n");

Why warn and not info?

-- 
Martin K. Petersen  Oracle Linux Engineering