Re: [Freedreno] [PATCH] fix double ;;s in code
On Sat, Feb 24, 2018 at 09:52:27AM +0100, Pavel Machek wrote: > Hi! > > > > diff --git a/drivers/soc/imx/gpc.c b/drivers/soc/imx/gpc.c > > > index 53f7275..cfb42f5 100644 > > > --- a/drivers/soc/imx/gpc.c > > > +++ b/drivers/soc/imx/gpc.c > > > @@ -348,7 +348,7 @@ static int imx_gpc_old_dt_init(struct device *dev, > > > struct regmap *regmap, > > > if (i == 1) { > > > domain->supply = devm_regulator_get(dev, "pu"); > > > if (IS_ERR(domain->supply)) > > > - return PTR_ERR(domain->supply);; > > > + return PTR_ERR(domain->supply); > > > > > > ret = imx_pgc_get_clocks(dev, domain); > > > if (ret) > > > > > > > Considering the controversy how the changes should be merged, I'm going > > to send a separate patch just for IMX GPC driver with a reported-by-you > > tag. Thanks for catching this. > > That works for me. > > Alternatively... Andrew Morton merged the patch to his -mm tree > (thanks!), so you don't need to take any action, and it will be > eventually fixed. Okay, I'm dropping my patch from IMX tree. Shawn ___ Freedreno mailing list Freedreno@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/freedreno
Re: [Freedreno] [PATCH] fix double ;;s in code
On Tue, 20 Feb 2018 10:03:56 +0200 Jani Nikulawrote: > On Mon, 19 Feb 2018, Pavel Machek wrote: > > On Mon 2018-02-19 16:41:35, Daniel Vetter wrote: > >> Yeah, pls split this into one patch per area, with a suitable patch > >> subject prefix. Look at git log of each file to get a feeling for what's > >> the standard in each area. > > > > Yeah I can spend hour spliting it, and then people will ignore it > > anyway. > > > > If you care about one of the files being modified, please fix the > > bug, ";;" is a clear bug. > > > > If you don't care ... well I don't care either. > > Look, if this causes just one conflict down the line because it touches > the kernel all over the place, then IMO it already wasn't worth > it. Merge conflicts are inevitable, but there's no reason to make life > harder just to cater for a cleanup patch. It's not that important. > > Had it been split up, the drm parts would've been merged already. I just stage patches like this behind linux-next. So if your stuff in in -next, you'll never notice a thing. ___ Freedreno mailing list Freedreno@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/freedreno
Re: [Freedreno] [PATCH] fix double ;;s in code
On Tue, Feb 20, 2018 at 3:03 AM, Jani Nikulawrote: > On Mon, 19 Feb 2018, Pavel Machek wrote: >> On Mon 2018-02-19 16:41:35, Daniel Vetter wrote: >>> Yeah, pls split this into one patch per area, with a suitable patch >>> subject prefix. Look at git log of each file to get a feeling for what's >>> the standard in each area. >> >> Yeah I can spend hour spliting it, and then people will ignore it >> anyway. >> >> If you care about one of the files being modified, please fix the >> bug, ";;" is a clear bug. >> >> If you don't care ... well I don't care either. > > Look, if this causes just one conflict down the line because it touches > the kernel all over the place, then IMO it already wasn't worth > it. Merge conflicts are inevitable, but there's no reason to make life > harder just to cater for a cleanup patch. It's not that important. > > Had it been split up, the drm parts would've been merged already. FWIW, the amdgpu and scheduler changes have already been fixed for -next. Alex > > BR, > Jani. > > -- > Jani Nikula, Intel Open Source Technology Center > ___ > amd-gfx mailing list > amd-...@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/amd-gfx ___ Freedreno mailing list Freedreno@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/freedreno
Re: [Freedreno] [PATCH] fix double ;;s in code
Daniel Vetterwrites: > On Sun, Feb 18, 2018 at 11:00:56AM +0100, Christophe LEROY wrote: >> Le 17/02/2018 à 22:19, Pavel Machek a écrit : >> > >> > Fix double ;;'s in code. >> > >> > Signed-off-by: Pavel Machek >> >> A summary of the files modified on top of the patch would help understand >> the impact. >> >> A maybe there should be one patch by area, eg one for each arch specific >> modif and one for drivers/ and one for block/ ? > > Yeah, pls split this into one patch per area, with a suitable patch > subject prefix. Look at git log of each file to get a feeling for what's > the standard in each area. This part is actually pretty annoying. I hacked up a script (below) which seems to do a reasonable job in most cases. For this patch it gives: $ for f in $(git ls-files -m); do ./guess-prefix.py $f; done ARC: ARC: ARM: arm64: ptrace: KVM: PPC: powerpc/powernv: x86/efi: block/sed-opal: clocksource: mips-gic: clocksource/drivers/sun5i: drm/amd/display: drm/amd/powerplay: drm/msm/mdp5: drm: iommu/vt-d: md: soc: imx: gpc: I think those are correct except for: - "drm:" for "drivers/gpu/drm/scheduler" which has only a single commit. - "md:" for "drivers/md/raid1.c" which is tricked by inconsistent usage of "md: raid1:" and "md/raid1:". But that seems like a reasonable hit rate. Another approach would be to have a file that defines for each subsystem what the preferred style is, but that is likely to be a PITA to maintain. cheers #!/usr/bin/python3 import sys import re from subprocess import check_output from collections import Counter if len(sys.argv) != 2: print('Usage: %s ' % sys.argv[0], file=sys.stderr) sys.exit(1) fname = sys.argv[1] cmd = ['git', 'log', '--format=%s', '-n', '100', fname] output = check_output(cmd).decode('utf-8') ignore = ['License', 'Merge'] # Ordered list of patterns patterns = [ # Common style "foo/bar/baz: Fix the foo" re.compile('^([\w\-_]+: )+'), # Less common "foo bar baz: Fix the foo" re.compile('^([\w\-_]+:? )+: ') ] words = [] for line in output.splitlines(): prefix = line.split()[0] for patt in patterns: match = patt.search(line) if match: prefix = match.group(0) break if prefix in ignore: continue words.append(prefix) # Warn if we didn't find many examples if len(words) < 5: print("Warning: only found %d previous commits to guess from for" % len(words), fname, file=sys.stderr) counts = Counter(words) print(counts.most_common(1)[0][0]) ___ Freedreno mailing list Freedreno@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/freedreno
Re: [Freedreno] [PATCH] fix double ;;s in code
On Mon 2018-02-19 16:41:35, Daniel Vetter wrote: > On Sun, Feb 18, 2018 at 11:00:56AM +0100, Christophe LEROY wrote: > > > > > > Le 17/02/2018 à 22:19, Pavel Machek a écrit : > > > > > > Fix double ;;'s in code. > > > > > > Signed-off-by: Pavel Machek> > > > A summary of the files modified on top of the patch would help understand > > the impact. > > > > A maybe there should be one patch by area, eg one for each arch specific > > modif and one for drivers/ and one for block/ ? > > Yeah, pls split this into one patch per area, with a suitable patch > subject prefix. Look at git log of each file to get a feeling for what's > the standard in each area. Yeah I can spend hour spliting it, and then people will ignore it anyway. If you care about one of the files being modified, please fix the bug, ";;" is a clear bug. If you don't care ... well I don't care either. drivers/gpu/ has four entries, i guess that's something for you. Pavel > > > diff --git a/drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c > > > b/drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c > > > index 61e8c3e..33d91e4 100644 > > > --- a/drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c > > > +++ b/drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c > > > @@ -718,7 +718,7 @@ static enum link_training_result > > > perform_channel_equalization_sequence( > > > uint32_t retries_ch_eq; > > > enum dc_lane_count lane_count = > > > lt_settings->link_settings.lane_count; > > > union lane_align_status_updated dpcd_lane_status_updated = > > > {{0}}; > > > - union lane_status dpcd_lane_status[LANE_COUNT_DP_MAX] = {{{0}}};; > > > + union lane_status dpcd_lane_status[LANE_COUNT_DP_MAX] = {{{0}}}; > > > hw_tr_pattern = get_supported_tp(link); > > > diff --git a/drivers/gpu/drm/amd/powerplay/amd_powerplay.c > > > b/drivers/gpu/drm/amd/powerplay/amd_powerplay.c > > > index 4c3223a..adb6e7b 100644 > > > --- a/drivers/gpu/drm/amd/powerplay/amd_powerplay.c > > > +++ b/drivers/gpu/drm/amd/powerplay/amd_powerplay.c > > > @@ -162,7 +162,7 @@ static int pp_hw_init(void *handle) > > > if(hwmgr->smumgr_funcs->start_smu(pp_handle->hwmgr)) { > > > pr_err("smc start failed\n"); > > > hwmgr->smumgr_funcs->smu_fini(pp_handle->hwmgr); > > > - return -EINVAL;; > > > + return -EINVAL; > > > } > > > if (ret == PP_DPM_DISABLED) > > > goto exit; > > > diff --git a/drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c > > > b/drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c > > > index 3e9bba4..6d8e3a9 100644 > > > --- a/drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c > > > +++ b/drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c > > > @@ -680,7 +680,7 @@ struct msm_kms *mdp5_kms_init(struct drm_device *dev) > > > } else { > > > dev_info(>dev, > > >"no iommu, fallback to phys contig buffers for > > > scanout\n"); > > > - aspace = NULL;; > > > + aspace = NULL; > > > } > > > pm_runtime_put_sync(>dev); > > > diff --git a/drivers/gpu/drm/scheduler/gpu_scheduler.c > > > b/drivers/gpu/drm/scheduler/gpu_scheduler.c > > > index 2c18996..0d95888 100644 > > > --- a/drivers/gpu/drm/scheduler/gpu_scheduler.c > > > +++ b/drivers/gpu/drm/scheduler/gpu_scheduler.c > > > @@ -461,7 +461,7 @@ void drm_sched_hw_job_reset(struct drm_gpu_scheduler > > > *sched, struct drm_sched_jo > > > { > > > struct drm_sched_job *s_job; > > > struct drm_sched_entity *entity, *tmp; > > > - int i;; > > > + int i; > > > spin_lock(>job_list_lock); > > > list_for_each_entry_reverse(s_job, >ring_mirror_list, > > > node) { -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html signature.asc Description: Digital signature ___ Freedreno mailing list Freedreno@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/freedreno
Re: [Freedreno] [PATCH] fix double ;;s in code
On Mon, 19 Feb 2018, Pavel Machekwrote: > On Mon 2018-02-19 16:41:35, Daniel Vetter wrote: >> Yeah, pls split this into one patch per area, with a suitable patch >> subject prefix. Look at git log of each file to get a feeling for what's >> the standard in each area. > > Yeah I can spend hour spliting it, and then people will ignore it > anyway. > > If you care about one of the files being modified, please fix the > bug, ";;" is a clear bug. > > If you don't care ... well I don't care either. Look, if this causes just one conflict down the line because it touches the kernel all over the place, then IMO it already wasn't worth it. Merge conflicts are inevitable, but there's no reason to make life harder just to cater for a cleanup patch. It's not that important. Had it been split up, the drm parts would've been merged already. BR, Jani. -- Jani Nikula, Intel Open Source Technology Center ___ Freedreno mailing list Freedreno@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/freedreno