On Wed, Jun 25, 2014 at 04:50:31PM +0200, Matthias Beyer wrote:
> On 23-06-2014 13:44:22, josh@xxxxxxxxxxxxxxxx wrote:
> > On Mon, Jun 23, 2014 at 11:42:33AM +0200, Matthias Beyer wrote:
> > > Signed-off-by: Matthias Beyer <mail@xxxxxxxxxxxxxxxx>
> >
> > As with the previous line-wrapping patch, this doesn't seem like a net
> > improvement; the lines seem far more readable un-wrapped.
>
> I don't know. Previous patches which were similar were added.
> Readability is really much opinion-based.
>
> So should I rebase this patchset and remove the line-length patches?
>
> Are there more opinions on this?
I'm not the arbiter of kernel formatting; it's up to Greg (as the
maintainer of the staging tree) if he accepts the patches or not.
However, I've submitted a patch to checkpatch that would silence this
warning by default when not running with --strict, which should
de-emphasize it in favor of other, more objectively clear cleanups.
> > > drivers/staging/bcm/DDRInit.c | 31 +++++++++++++++++++++----------
> > > 1 file changed, 21 insertions(+), 10 deletions(-)
> > >
> > > diff --git a/drivers/staging/bcm/DDRInit.c b/drivers/staging/bcm/DDRInit.c
> > > index 423bfd9..4564f40 100644
> > > --- a/drivers/staging/bcm/DDRInit.c
> > > +++ b/drivers/staging/bcm/DDRInit.c
> > > @@ -1159,7 +1159,8 @@ int download_ddr_settings(struct bcm_mini_adapter *Adapter)
> > > {
> > > struct bcm_ddr_setting *psDDRSetting = NULL;
> > > ULONG RegCount = 0;
> > > - unsigned long ul_ddr_setting_load_addr = DDR_DUMP_INTERNAL_DEVICE_MEMORY;
> > > + unsigned long ul_ddr_setting_load_addr =
> > > + DDR_DUMP_INTERNAL_DEVICE_MEMORY;
> > > UINT value = 0;
> > > int retval = STATUS_SUCCESS;
> > > bool bOverrideSelfRefresh = false;
> > > @@ -1283,18 +1284,22 @@ int download_ddr_settings(struct bcm_mini_adapter *Adapter)
> > > }
> > > /* total number of Register that has to be dumped */
> > > value = RegCount;
> > > - retval = wrmalt(Adapter, ul_ddr_setting_load_addr, &value, sizeof(value));
> > > + retval = wrmalt(Adapter, ul_ddr_setting_load_addr, &value,
> > > + sizeof(value));
> > > if (retval) {
> > > - BCM_DEBUG_PRINT(Adapter, DBG_TYPE_PRINTK, 0, 0, "%s:%d\n", __func__, __LINE__);
> > > + BCM_DEBUG_PRINT(Adapter, DBG_TYPE_PRINTK, 0, 0,
> > > + "%s:%d\n", __func__, __LINE__);
> > >
> > > return retval;
> > > }
> > > ul_ddr_setting_load_addr += sizeof(ULONG);
> > > /* signature */
> > > value = (0x1d1e0dd0);
> > > - retval = wrmalt(Adapter, ul_ddr_setting_load_addr, &value, sizeof(value));
> > > + retval = wrmalt(Adapter, ul_ddr_setting_load_addr, &value,
> > > + sizeof(value));
> > > if (retval) {
> > > - BCM_DEBUG_PRINT(Adapter, DBG_TYPE_PRINTK, 0, 0, "%s:%d\n", __func__, __LINE__);
> > > + BCM_DEBUG_PRINT(Adapter, DBG_TYPE_PRINTK, 0, 0,
> > > + "%s:%d\n", __func__, __LINE__);
> > > return retval;
> > > }
> > >
> > > @@ -1303,17 +1308,23 @@ int download_ddr_settings(struct bcm_mini_adapter *Adapter)
> > >
> > > while (RegCount && !retval) {
> > > value = psDDRSetting->ulRegAddress;
> > > - retval = wrmalt(Adapter, ul_ddr_setting_load_addr, &value, sizeof(value));
> > > + retval = wrmalt(Adapter, ul_ddr_setting_load_addr, &value,
> > > + sizeof(value));
> > > ul_ddr_setting_load_addr += sizeof(ULONG);
> > > if (!retval) {
> > > - if (bOverrideSelfRefresh && (psDDRSetting->ulRegAddress == 0x0F007018))
> > > + if (bOverrideSelfRefresh
> > > + && (psDDRSetting->ulRegAddress
> > > + == 0x0F007018))
> > > value = (psDDRSetting->ulRegValue | (1<<8));
> > > else
> > > value = psDDRSetting->ulRegValue;
> > >
> > > - if (STATUS_SUCCESS != wrmalt(Adapter, ul_ddr_setting_load_addr,
> > > - &value, sizeof(value))) {
> > > - BCM_DEBUG_PRINT(Adapter, DBG_TYPE_PRINTK, 0, 0, "%s:%d\n", __func__, __LINE__);
> > > + if (STATUS_SUCCESS != wrmalt(Adapter,
> > > + ul_ddr_setting_load_addr,
> > > + &value,
> > > + sizeof(value))) {
> > > + BCM_DEBUG_PRINT(Adapter, DBG_TYPE_PRINTK, 0, 0,
> > > + "%s:%d\n", __func__, __LINE__);
> > > break;
> > > }
> > > }
> > > --
> > > 2.0.0
> > >
>
> --
> Mit freundlichen Grüßen,
> Kind regards,
> Matthias Beyer
>
> Proudly sent with mutt.
> Happily signed with gnupg.
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
